Design comparison
SolutionDesign
Community feedback
- @Finney06Posted over 1 year ago
Hello there ๐. Good job on completing the challenge !
Here are some suggestions regarding your code that may be of interest to you.
HTML ๐ท๏ธ:
To clear the Accessibility report:
- Wrap the page's whole main content in the
<main>
tag. - Use HTML5 semantic elements such as
<header>
,<nav>
,<main>
,<aside>
, and<footer>
to define these sections.
I hope you find it helpful!๐ Above all, the solution you submitted is ๐. ๐Happy coding!
1 - Wrap the page's whole main content in the
- @visualdennissPosted over 1 year ago
Good work! Your API requests seem to be working without any issues.
It looks like the dice button has a default border applied, you can remove that with simply adding border: none; to your button for a nicer look.
Hope you find this feedback helpful!
1
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