Responsive landing page using Sass and JavaScript

Please log in to post a comment
Log in with GitHubCommunity feedback
- @jonathan401
Hey @Ol4bod3 👋. Congrats on completing this challenge 👏🎉. You did a very nice job ☺️. Just a few suggestions though ☺️:
-
You should try making sure you don't skip any heading level. So your heading should go like: h1, h2, h3, h4, h5, h6. So your code should look something like:
<h2>Transform your brand</h2>
. This will fix accessibility issues. -
On line 36 on your index.html file you should use forward slashes in the src attribute on your image tag to provide a relative path to the image. So you should write
<img src='images/icon-arrow-down.svg' alt=''/>
. -
This might not be necessary but you should also try adding a heading in every section
I guess that's all 😁 I hope this helps. Again, congrats on completing the challenge 👏👏🎉 and Happy Coding ☺️!.
-
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