Looking to hire developers?
@KristianDunne
@NiteArie
Submitted
Would love hearing some small improvement tips to further make the website better. Thanks in advance! Really appreciate the feedback. Problem: When the screen max-width is around 900px, the page layout will become a 1 column layout. I think that this l1-column is not that great for that width because it makes the form too fat, would love hearing some other idea for this specific scenario
@KristianDunne
Posted
I like how you used the background-image property on your input fields to show the error icon! I might try this one my implementation. Overall it's pretty much a perfect match to the designs.
Great work!
@dianailan
Submitted
Hi how can I make background pattern images to be responsive and become smaller with screen but don't change position relative to card?
@KristianDunne
Posted
Hi Diana,
You need to set the bg-patterns as the bodys background through CSS instead of using them as HTML img elements. Check out how I did it here:
https://github.com/KristianDunne/profile-card-component/blob/main/style.css
Hope this helps!
@aymeric131
Submitted
What i should improve in my code ?
@KristianDunne
Posted
I think markup is clean and concise. You could fix the spacing between the elements particularly in the footer and reduce the border width to closer match the design.
Besides that its nice and responsive, great work!
@ImranAvenger
Submitted
Hi there, after more hardworking I may be successful. This project took a lot of time, anyway, finally, I'm so happy. ☺ And I've submitted it here, now it's the turn of you to judge it, and give me feedback, please! Thank you in advance.
@KristianDunne
Posted
It works well on mobile and desktop however as the screen width starts to get better the content inside you section_1 div is fixed too far to the left of the screen. Besides that great work!
@deborahtrota
Submitted
Any feedback or suggestions will be appreciated :)
@KristianDunne
Posted
Your site is not responsive on mobile or ultrawide views so it would be good to fix that next. It is also worth adding some padding to the social icons to give them some space from their borders to closer match the design. Keep up the work!
@aaronpaulgz
Submitted
Any feedback is welcome :D Tried to did it quickly to discover if I had improved my skills :D
@KristianDunne
Posted
The component is not centred on desktop and it needs some padding on mobile. Once you fix these I think it will be great!
@limtedtorni000
Submitted
Any feedback or suggestions will be appreciated :)
@KristianDunne
Posted
Wow, this is a pixel-perfect match to the design!
Its slick, responsive, your markup is minimal and use of SASS is great.
Great work keep it up!
@daudi13
Submitted
All feedbacks are welcome :-)
@KristianDunne
Posted
@daubi13 Your mobile implementation is great but I have a few suggestions that would improve the desktop version.
boxesmax-width to your .container-boxline-height on your list itemsYou should also capitalize the sign up button label and remove its opacity when it is not being hovered over.
Keep up the good work!
@MarcusMourning
Submitted
@KristianDunne
Posted
Great implementation,
Just missing the bold text on the active accordion items title.
Nice work Marcus!
@cyishere
Submitted
Any suggestion about how to deal with the filter buttons on the desktop or mobile?
@KristianDunne
Posted
Nice implementation CY!
You should look at using styled-components or CSS modules alongside your components to have the styles scoped. Would be much nicer than having to go back to a global stylesheet.
Great work!
@SirenaAlyceDev
Submitted
I could not figure out how to get the background image circles positioning correct. Can anyone help with that? I also noticed that some of my bootstrap classes and stylings aren't showing when deployed via Vercel. Can anyone help with that also?
@KristianDunne
Posted
Hey @SirenaAlyceDev,
Because you installed bootstrap via npm you will have to tell vercel to install your node_modules.
Hope this helps.
@lnirjar
Submitted
@KristianDunne
Posted
It looks nice and works well on mobile but the card__body grows as the height of the page increases which is down to the flex-grow: 1.
Maybe revisit this and check against larger screen sizes, but otherwise good work!
@RenszCamacho
Submitted
Hi guys, this is my second challenge. I added some extra features and changed the design slightly. I'd like to hear your thoughts, what kinds of problems you find, and how you would approach them.
Cheers.
@KristianDunne
Posted
Hey Rensz,
Looks good this and really like what you have done with the animations to add a bit of something extra to it.
Good work!
