@vanzasetia
Posted
Hello, Michelle! π
Good effort on this challenge! π Your solution looks great! π
However, I recommend refactoring the CSS by following this guide.
- First, you don't need absolute positioning. It works well here, however, in a large project (like landing page challenges), it would be a pain to position every element with absolute positioning, especially in making the layout responsive. So, to make the card in the middle of the page you can use flexbox instead of
margin
andposition
properties and usepadding
to control the position of the element inside the card. - Second, the element that wraps all the cards only needs a
max-width
to prevent the cards from becoming too large. - Third, every card doesn't need
width
andheight
. Forwidth
, since it's a block element it will for sure havewidth: 100%
so it will follow themax-width
of the parent element. Forheight
, let the element inside it controls the height of it. After that, addpadding
to prevent the element inside it from touching the edges of the card.
For the HTML, I recommend leaving the alt
empty for all icons because they are only for decoration purposes.
That's it! Hope this helps.
@MichelleMonrreal
Posted
@vanzasetia Thanks so much this definitely helps! Really appreciate it!
@vanzasetia
Posted
@MichelleMonrreal You're welcome! π