Space tourism multi-page and responsive website

Solution retrospective
I think I can improve the handling of responsive and flexible images. Also, any general feedback is welcome.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @emestabillo
Hey @hebrerillo, this looks visually solid, congrats!
The only thing that I noticed is the use of semantics, especially headings, on each page. We're missing the
h1
s here and the heading levels doesn't seem to be in order (ex. h5 is mentioned first before h2). Useclass
es to style your headings (or anything for that matter), instead of looking at 'how large' they look on the design and assigning a heading tag. Here's a resource regarding the headings.Hope this helps!
Marked as helpful - @emestabillo
@hebrerillo seems better, yes. It's best-practice, however, not to skip heading levels as stated in the resource I mentioned above. You can reuse the heading tag once it's been mentioned once, in order.
For example, in the technologies page, you have
h1> h4 > h3
. The numbers are not in order. Ideally, it would beh1> h2 > h3
. You can then reuse h2 and h3 elsewhere in the page as long as it has been mentioned in order once, ex.h1> h2 > h3 > h2 > h3 > h2
. This semantic is common in designs that have multiple sections, since each of the sections will usually need the same headings.Another side note about the h1 on the homepage. I would keep the entire phrase 'SO, YOU WANT TO TRAVEL TO SPACE' in one h1 tag. I would write it as
<h1>SO, YOU WANT TO TRAVEL TO <span>SPACE</span></h1>
.Hope this helps!
Join 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