Design comparison
Community feedback
- @MehrshadHeisenberg3Posted over 1 year ago
It's a great design good job!
Just one thing to make it better!
If you look at the original design you can see that the cursir changes to pointer when you hover those items. you can use this for that:
cursor: pointer;
Also you can make different items a link by putting them inside <a></a> tags. Then you can use href="# or whatever link you have" to make them links!
Again, GREAT JOB!
0 - @HassiaiPosted over 1 year ago
Every html must have <h1> to make it accessible. Always begin the heading of the html with <h1> tag wrap the sub-heading of <h1> in <h2> tag, wrap the sub-heading of <h2> in <h3> this continues until <h6>, never skip a level of a heading.
For a responsive content which wont require a media query for this challenge, give the main a fixed max-width value.
Change the min-height value to 100vh.
min-height:100vh
Hope am helpful.
Well done for completing this challenge. 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