Skip to content
Submitted 5 months ago

Basic HTML,CSS and JS

Aaliyan10•190
@Aaliyan10
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

What specific areas of your project would you like help with?

I added a simple div banner in the share part. how do I add the shape that was mentioned in the design. And also how to change the color of the share button.

Community feedback

  • AinaKhanDurrani•30
    @AinaKhanDurraniPosted 5 months ago

    Consider using <article> for the main content to provide more semantic meaning, especially since it's an article preview component. Ensure all images have descriptive alt attributes, e.g., <img src="images/drawers.jpg" alt="Wooden drawers">.

    For JavaScript, consider using more descriptive variable names and avoiding magic numbers (like 2 for toggling). For better readability, split the JavaScript code into functions. For example, create functions to handle showing and hiding the share options.

    0
    Aaliyan10•190
    @Aaliyan10Posted 5 months ago

    @AinaKhanDurrani thank you for your feedback. Could you please check the parts I wanted help in and guide me with them as well?

    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