Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive Landing Page, Using React & Bootstrap

#bootstrap#react#jss
C.J. Cline 250

@SiegeTank-90

Desktop design screenshot for the Space tourism multi-page website coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


Thanks to F.E.M. for putting up the site and setting up these free challenges. Feed back is welcome on any parts, but in particular I'm looking for anything, that would be more likely done in a working environment, if this had been done to work with a team of developers what kind of change would have been made?

Community feedback

@zineb-Bou

Posted

Hi there, congratulation on completing this challenge, taking a quick look at your solution I think there are a couple of things that need some improvement

  1. On some pages, there is a horizontal scroll bar, setting an explicit width to any container or element may cause this problem. try to fix that.
  2. The background images are a bit more zoomed out.
  3. On the Destination page (desktop version), on the planets navigator, options are overlayed on top of each other.
  4. Try to use the right Html semantic elements, for instance in navigators, instead of using <h5> inside a <div>, <ul> is more adequate for that.
  5. Headings in HTML [<h1> ..... <h6>] are used to construct a table of contents for a document, so avoid using heading elements to resize text. Instead, use the CSS font-size property.

Happy coding

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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