Vanza Setia• 27,855
@vanzasetia
Posted
👋Hi Shahin!
👍 Good job on completing this challenge! I have some feedback on this solution:
- On mobile view (360px * 640px) the
Rated 5 Stars in...sentence is not aligned to the center and become two lines. - The name of the person, should not be heading. They should be on paragraph tags. Heading is commonly used for titling a section.
- Also, you need to change the
main__cardswithulandlifor each card. That way you don't need a heading anymore on each card. - In my opinion the person image is important. So, I recommend to use their name as the
altvalue.
That's it! Hopefully this is helpful!
Marked as helpful
0
Shahin NJ• 1,190
@SJ-Nosrat
Posted
@vanzasetia Hi, thank you for taking an in-depth look at my project.
With regards to your first (bullet) point, how would you go about solving that issue? The problem is from the padding that I placed on them; would you give them a width and a height property?
0
