Skip to content
Submitted about 1 year ago

Article Preview Component using flexbox

WasiArshad•30
@WasiArshad
A solution to the Article preview component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

What are you most proud of, and what would you do differently next time?

This project put me the extra pressure and that's leads me to work hard get to know many new things in the different ways.

What challenges did you encounter, and how did you overcome them?

I was unable to show popup button seamlessely but later on I tried and learn from other third party websites.

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

I can me more helpful in the HTML and CSS .

Code

Please log in to post a comment

Log in with GitHub

Community feedback

  • P
    visualdennis•8,395
    @visualdenniss

    Hello there,

    great job completing this successfully! Few suggestions for further improvements:

    • Add a little bit more gap between 'Share' and button icons.
    • No need to choose headers based on their size, e.g. h5 to get a smaller. Headings on a page should follow an order, h1 , h2 etc. and use CSS to change their style.
    • Consider using media queries to add responsiveness for mobile view.
    • Variable naming in JS looks a little confusing, as one would except 'buttons' would be multiple buttons, but you are just selecting one button, so better to name as 'button' etc.

    Hope these recommendations are helpful!

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