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 Preview Component using flexbox

WasiArshad• 30

@WasiArshad

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


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 .

Community feedback

P
visualdennis• 8,255

@visualdenniss

Posted

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!

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