Our reporter did not find any issues in this project! 🎉
Submitted
React in TypeScript and Responsive design with CSS
@kaixuan98
Design comparison
Reports
Avery Chin’s questions for the community
I have been working on web development for quite a while. I wanted to improve in depth where focus on best practices. Do you have any questions about best practices? Which areas of the code that you find it unsure of or violating the best practices? Which area that you think is okay but can be improved with XYZ change?
Community feedback
Hi there, great job the site looks awesome! Regarding your question, I might suggest use of semantic HTML tags as a best practice for accessibility and screen readers. I also noticed that the navmenu and hero banner seem to get smushed unintentionally when resizing down to 600px. It may be worth reviewing how your flex items flow in the document.
All that said, keep up the great work and good luck on your next challenge!!
Marked as helpful
@kaixuan98
Posted
@maximusDecimalusMeridius Thank you for the comment! I look into semantic HTML, it opens up the rabbit hole to accessibility and got me started to thinking about accessibility. I had implemented semantic HTML and some basic accessibility practise with ARIA.
I also went in to fix the sizing issues, where at 600px point, the nav bar switched to the mobile version but does not show the overlay nav bar.
Please log in to post a comment
Log in with GitHubJoin our Slack community
Join over 180,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!