Design comparison
SolutionDesign
Solution retrospective
Hey guys,
All suggestions for improvement are welcome. Code is written in SCSS. You can also find the SCSS-file in my repository.
Thanks, I am grateful for any help π€
Community feedback
- @fernandolapazPosted over 1 year ago
Hi ππ», just a couple of things that may interest you:
- It would be better to capitalize the headings with CSS (using
text-transform: uppercase;
) and not in HTML. Some screen readers will read out uppercased text as individual letters.
- Remember that
<button>
should be used for any interaction that performs an action on the current page and<a>
should be used for any interaction that navigates to another view. Using both doesn't seem appropriate, links are suitable in this case. And then you can style it with CSS to look like buttons.
I hope itβs useful π€π»
Nice solution.
Regards,
Marked as helpful1 - It would be better to capitalize the headings with CSS (using
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