Skip to content
Submitted over 1 year ago

Article Component (Flex CSS) 💨

ezecoder•870
@3eze3
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Article preview component
  • HTML
  • CSS
  • JS
1newbie

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•960
    @Tryt4nPosted over 1 year ago

    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
    ezecoder•870
    @3eze3Posted over 1 year ago

    @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

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