P
Andrew Teece• 380
@andrewteece
Submitted
Any recommendations and or suggestions are always appreciated.
Lamar
@Everezze
All comments
- Lamar• 495
@Everezze
Posted
Hi Andrew, good job on the challenge!
I might be picky but something to change would be the ratings touching the edges of It's container to get them aligned with the other contents in the component (space-between instead of space-around). A personal challenge(not a requirement in the project iirc?) would be handling when the user click 2 times on the same rating to remove it without choosing another one.
Apart from that, that's great! Keep going
Marked as helpful
0 - varun• 10
@vsharma12
Submitted
Can i make it better? please review it, it would be helpful!! Thanks :)
Lamar• 495@Everezze
Posted
Hi, You need to change the font-size it's too big, and some elements are also bold like title, the button. Instead of setting the card for different width you could set a max-width so for the desktop it will never be longer than that. The annual plan section have a grayish background that you need to set. The transition time on your button might be too long but I'm not sure about this one.
Marked as helpful
1 - Angel Duarte• 40
@angelostd
Submitted
Any feedbback is always welcome!~ :D
(I think the HTML validations is fixed, but the report doesn't update).
Lamar• 495@Everezze
Posted
Hi angelo congrats for completing the challenge ! Like @Fluffy said the cards are a little bit bigger so you might reduce their width a little bit. Also I would add that it needs space between your anchors and the text, you can achieve that by just setting a margin top or bottom depending on which html element you're setting it on.
Marked as helpful
0
