Responsive Stats Preview card component
Design comparison
Solution retrospective
Any feedback about my solutions is welcome :D
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hello Abieb_Guardian, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:
To align some content in the center of the screen, always prefer to use
display: flex;
it will make the layout more responsive!Example:
body { margin: 0; padding: 0; display: flex; align-items: center; flex-direction: column; justify-content: center; min-height: 100vh; }
The links must have an aria-label or sr-only text that tells where the link navigates the user. For example: Visit our Facebook. For images, you should set aria-hidden=” true” to be ignored by screen readers and to avoid redundancy and repetition.
The remainder is excellent.
I hope it's useful. 👍
Marked as helpful0@AbibGuardian50Posted almost 2 years ago@AdrianoEscarabote hi Adriano Thanks for your advice! i will try to implement it in my next project,thank you and have a great day
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