Responsive article preview card using HTML5, CSS3, Sass & JavaScript
Design comparison
Solution retrospective
Hi Everyone,
I would like to get suggestions on how i used JavaScript to get the work done, is it right how i used? or is there any better way which i can go with?
Thank you... :)
Community feedback
- @NaveenGumastePosted almost 3 years ago
Hello Srikar Gunnam ! Congo π on completing this challenge
Let's look at some of your issues, shall we:
- Warp your card content around the main tag Ex: π
<body> <main class="container"> *all you content here* </main> </body>
- As matt said : 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 alt.
happy Codingπ
0 - Account deleted
Hello there! π
Congratulations on finishing your challenge! π
I have some feedback on this solution:
-
Always Use Semantic HTML instead of
div
like<main>
<header>
<footer>
, etc for more info -
All Images must have alternate text
i hope this is helpful and goodluck!
0@mattstuddertPosted almost 3 years ago@Old1337, just a quick heads up that it's not required for all images to have alternative text. They need the
alt
attribute, but the value can be left blank if it's a decorative image. This will ensure screen readers ignore the image.Images that add context to the content should always have descriptive alternative text π
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