@FLCHRIS
Posted
Hi there!, congrats on completing this challenge!! π₯³π
I have some tips to improve your solution. π―
HTML:
- Header elements implements six levels
h1-h6
, withh1
being the most important andh6
being the least important. avoid omitting header levels, your page should start with the headerh1
and successively withh2...h6
. More info HERE - Your main content should be conteined in the
main
tag. More info HERE - Your
img
tag, should havealt
attribute.
SCSS:
Your reset is good, but you can improve it by using some popular resets like the following π:
I can see that your variables are in the partials folder, but your reset is inside your styles.scss file. If you want to organize your files I recommend the following (it's my way of doing it, you can find another way that suits you):
-- scss
---- components
------ _button.scss
------ _nav.scss
---- utilities
------ _variables.scss
---- layout
------ _footer.scss
------ _main.scss
---- base
------ _reset.scss
---- index.scss
Those are some example files, I hope and they are clear. π
I hope I've helped you
Happy coding! βοΈ
Marked as helpful
@Feernandou
Posted
@FLCHRIS Thank you very much for your suggestions. It is very kind of you. I will take into account everything you recommended.
@FLCHRIS
Posted
@Feernandou
I'm glad I helped you π Keep it up!