@romila2003
Posted
Hi @BLADEHEDA,
Congratulations for 🎉 for completing this challenge, the Article preview component looks good and it is great that you used the right semantic. There are some issues/suggestions I want to address:
- I noticed that you took a desktop-first approach however I would strongly suggest you follow a mobile-first approach instead as it is easier for responsiveness, and it is also best practice to do so.
- The
font-family
can be found within the style-guide that is given to you when you download the starter pack. - Instead of using
position: absolute;
to center the card, you can use theflex
property instead e.g.
body {
display: flex;
align-items: center;
justify-content: center;
min-height: 100vh;
flex-direction: column;
}
To make the share-icon functional, you can use the toggle
feature within JS. The toggle feature is like an on/off switch that will show the icon when click and then close the icon when clicked again.
Overall, great attempt and wish you the best for your future projects so keep coding 👍.