Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Article Component (Flex CSS) 💨

#accessibility#bem#sass/scss#jss
Desktop design screenshot for the Article preview component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello community, I really did not encounter any problems in this project because the interaction really relied on css rather than js because it is just a set of classess with the components, I really liked it, it was something fun to do, and if you have any improvement for my code, both in the design, you can let me know in the comments, Happy coding.

Community feedback

Tryt4n• 940

@Tryt4n

Posted

You can add <time datetime=""> element instead of <span>. For button you can add aria-haspopup="true".

Also your media queries in my opinion are set wrong. About 860px width and below it's overflowing. If popup is open overflow starts around 940px.

Marked as helpful

1

@3eze3

Posted

@Tryt4n Thank you for the recommendations. I appreciate your suggestions to improve the HTML code. I'll take into account your suggestion to use the <time datetime=""> element instead of <span> to represent date and time information. I will also add the aria-haspopup="true" attribute to the button.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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