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
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!
Hi how can I make background pattern images to be responsive and become smaller with screen but don't change position relative to card?
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!
What i should improve in my code ?
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!
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.
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!
Any feedback or suggestions will be appreciated :)
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!
Any feedback is welcome :D Tried to did it quickly to discover if I had improved my skills :D
The component is not centred on desktop and it needs some padding on mobile. Once you fix these I think it will be great!
All feedbacks are welcome :-)
@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!
Great implementation,
Just missing the bold text on the active accordion items title.
Nice work Marcus!
Any suggestion about how to deal with the filter buttons on the desktop or mobile?
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!
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?
Hey @SirenaAlyceDev,
Because you installed bootstrap via npm you will have to tell vercel to install your node_modules.
Hope this helps.
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!
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.
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!
