Design comparison
Solution retrospective
I am not sure if this is the best practice of SASS so please feel free to leave a feedback
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hi Mortada, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader. You can read more in MDN.
prefer to use
max-width
on the element you want to make the line break!To use max-width on divs that have background-colors, we have to use
max-width
only on the content div and let the one with thebackground-color
stretch!The rest is great!
I hope it helps... 👍
Marked as helpful1
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