Aviral Sharma• 220
@aviralsharma07
Posted
Hey, Congratulations on creating this project. Here are my few suggestions for you.
- Use semantics HTML Tags like <article> to wrap card content then <div>
- You don't need to wrap the <img> inside an <div> it is not required.
- Write your CSS in a separate file named styles.css and link it to HTML in the <head> section.
- Use max-width: 25ch instead of width for .text-class
- There is no need for a Flex basis multiple times in this project. Refer to this to learn Flexbox: https://css-tricks.com/snippets/css/a-guide-to-flexbox/
- Use margin-top for spacing between <p> and heading not padding-top.
Hope it helps. Keep up the good work!
Marked as helpful
0
Ozoeze Ugo• 10
@Ugonadia
Posted
@aviralsharma07 Thank you
1
