@Kamasah-Dickson
Posted
Your solution is responsive on mobile good work Keep codingπ
Here's my solution for this challenge, feel free to write your suggestions. Thank You.
@Kamasah-Dickson
Posted
Your solution is responsive on mobile good work Keep codingπ
@NaveenGumaste
Posted
Hay ! Monica Good Job on challenge
These below mentioned tricks will help you remove any Accessibility Issues
-> Add Main tag after body <main class="container"></main>
-> Always use the "alt attribute" and write what img is , and if the img is for decorative then leave it empty but always add it with img tag.
-> Learn more on accessibility issues
If this comment helps you then pls mark it as helpful!
Have a good day and keep coding π!
@vanzasetia
Posted
Hi There! π
Congratulations on completing this challenge! π
I notice that you have put text content inside all social media links but you make them display: none;
. If you make the text display: none;
, it won't be accessible by anyone. Instead, I would suggest having a visually hidden text, the sr-only
styling.
Some feedback:
alt
attribute. You can leave them empty if the images are decorative.nav
is commonly used for links that are used to navigate the user in the website.:focus-visible
styling to any interactive elements (button
, links, input
, textarea
). This will make the users can navigate this website using keyboard (by using Tab
key) easily.max-width
for the container
instead of width
so that it allows to shrink.I hope this information is useful! Happy coding! π
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