Responsive page using HTML, CSS, and SCSS.
Design comparison
Solution retrospective
Hello there! That is my first project here, I would like feedback. Is it good? How can I improve it? What could be better?
Community feedback
- @brodiewebdtPosted almost 3 years ago
Between 450px and 768px, the design distorts. Either side of those points it looks good.
Wrap your container div in a Main tag. Change the H3 tag to an H2 and it will clear the accessibility warnings. You may have to re-style the H2 to look like the design.
Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/
Hope this helps.
Marked as helpful0 - @meladcodesPosted almost 3 years ago
Hi, Well done on your first project! in order to avoid accessibility and HTML issues in the future:
- wrap everything in your body in <main>
- start your headings with h1 and move up by one level each time
Hope this helps!
Marked as helpful0@meladcodesPosted almost 3 years ago@anoshaahmed your comments have helped me so I'm trying to help others, is there something wrong? If so, then I apologise.
0@anoshaahmedPosted almost 3 years ago@meladcodes at least use your own words. you literally copy-pasted my exact words. and u should probably fix ur own work before advising others
0@meladcodesPosted almost 3 years ago@anoshaahmed I never said I'm the perfect coder who makes no mistakes, I'm a beginner myself. If you give so much value to a comment then I will delete it.
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