Design comparison
Solution retrospective
Finished this project with Responsiveness ! Used HTML, CSS and Bit of Js for email validation .
Any feedbacks are appreciated.
Community feedback
- @GerbenDolPosted over 4 years ago
Hey Sijan! Your solution is looking really good, very well done!
A few things I'd take a look at:
- Maybe give the image a max-width. On my smaller laptop screen it takes up a lot of room
- I feel like your form is a bit on the small side
- When I submit your form I see the error appear but right after that it seems to reload the page
- It looks like your social icons aren't centered, it's especially noticeable on hovering the Facebook icon
Anyways, really nice solution! Keep it up. 😁
1@sijandahalPosted over 4 years ago@GerbenDol Yes, I have problem with the page automatically reloading and I couldn't figure it out.Would you please mind looking at that ? and talking about the image I have given it the width of 100% and max-width as well, which I thought of making it responsive.
0@GerbenDolPosted over 4 years ago@sijandahal I think doing some Googling will help you fix the issue with no trouble. Perhaps try adding
return false
to the form instead of justreturn
. 🤓I see you set a max-width of 1000 pixels on the image, but I think that might be too much. Of course this is also up to you as a developer to decide!
Sorry for the late response. 🙂
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord