
Design comparison
Community feedback
- @AdrianoEscarabotePosted 5 months ago
Hi kirlosbasta, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:
Avoid using the
<br>
tag in your HTML code. While<br>
might seem like a simple way to break lines, it is considered bad practice and can lead to significant accessibility concerns. For users who rely on screen readers, the presence of<br>
can be announced, which disrupts the flow of the content and creates a confusing experience.Instead of
<br>
, you should use semantic HTML to structure your content properly. For example, wrapping text in paragraphs (<p>
) or using<div>
containers for sections provides a cleaner and more accessible solution. This approach improves usability for screen readers and ensures that your content is presented in a meaningful way to all users.For more detailed guidance, refer to the MDN documentation on the
<br>
tag: MDN: Accessibility Concerns of <br>Pro Tip: Accessible web development isn't just a recommendation—it's essential for ensuring inclusivity on the web!
The rest is excellent.
I hope you find it useful. 👍
Marked as helpful0@kirlosbastaPosted 5 months ago@AdrianoEscarabote Thank you adriano for your helpful comment I'll definitely avoid using it next time. 💖
1 - @sing12578Posted 5 months ago
very good
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