Design comparison
Solution retrospective
Hello there! This is my code solution for the "Huddle Landing Page with Curved Sections" challenge.
I would love to receive some feedback. I'm eager to learn and improve my coding skills, so any constructive criticism or suggestions for improvement would be greatly appreciated.
Thank you in advance for taking the time to review my work, and I look forward to hearing your thoughts!
Community feedback
- @javascriptor1Posted 11 months ago
Hi DEEPALI ,
Excellent work. I just uploaded my solution to this challenge today. Here are my notes which I can share with you about your solution :
1- For anchor links, always use anchor tag <a> and don't use button. Look at your code below:
<button class="cta-button">Get Started For Free</button>
Where does this button take the user?? No where. Buttons are used to take action like subscribe or submit a form. If you are sending the user to a link , use a with href attribute.
2- For contact information in the footer, use the address html tag and wrap all information inside it.
3- The color of the logo on top should be black, not dark cyan. You have to change it inline or via CSS.
4- The hover state effect for social media icons is not correct.
5- You have not applied any checking in JS whether an email is valid or not but relied on built-in browser checking. Do this checking in JavaScript and display the error message as required.
6- For the main, it's supposed to wrap the main part or section of the page as per my understanding. This website is made of different sections and it's not correct to wrap everything in the main element including the footer 😊
7- The Facebook icon is a little aligned to the left in comparison to the above section. This is very subtle and hardly noticeable. These elements are supposed to have the same padding/margin, so they are aligned perfectly on the same vertical line.
Good luck and happy coding.
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