Design comparison
SolutionDesign
Community feedback
- @HamidAliZainPosted about 2 years ago
replace the navbar position fixed to absolute so it looks pretty and very awesome
0 - @kirtymeenaPosted about 2 years ago
Hi soheil ramezani
This was a tough challenge, but you did a great job. On the destination, crew, and technology page, the background image is not covering the entire page. Try removing the CSS in the class destination.
.destination { height: 100%; }
For crew as well you can remove the below class from the CSS file and it will work fine.
.crew{ height: 100%; }
There are some accessibility errors. to remove it you can add an alt text in img. tag
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