@Yazdun
Posted
Hello š this looks great ! Here are my suggestions :
- Try using 7-1 pattern to structure your sass. They even offer a boilerplate folder architecture which makes your life much easier.
- On smaller device, cards take full view width and height, consider giving some paddings to parent element.
- Each
html
page must have onemain
tag which wraps the whole page's content, also thismain
tag must have alevel one heading
which tells what is page about, you can style thish1
to be hidden. This will solve your ACCESSIBILITY issue, don't forget to create new report though - Instead of
article
andsection
, Usediv
and It may solve your HTML VALIDATION issue.
ā I hope this was helpful for you
Marked as helpful
@LoganWillaumez
Posted
Hello @Yazdun , thanks for your feedback !
Oh perfect, it's exactly what I needed for improve my SCSS skills š
Thanks for other responses, I corrected my code and all is good now.
Have a nice day!