Submitted about 5 years ago
first challenge with bootstrap 4
@MOHAMMED-ABD-RAZAQ
Design comparison
SolutionDesign
Community feedback
- @mattstuddertPosted about 5 years ago
Hi Mohammed, great work on this challenge and congratulations on submitting your first solution! What specifically would you like to get feedback on?
As far as your HTML goes, I've got a couple of suggestions:
- I'd recommend taking a look into using HTML5 structural elements to replace some of your
div
tags. Here's a great MDN article of adding structure to the page. - You're currently using
h2
andh5
heading elements, but noh1
,h3
orh4
. I'd recommend always having a singleh1
on the page. This marks out the main page title. Also, I'd recommend not skipping heading levels unnecessarily, as this can cause accessibility issues for people using screen readers.
I hope these tips help. Let me know if you have any questions! 🙂
3 - I'd recommend taking a look into using HTML5 structural elements to replace some of your
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