Any recommendations and or suggestions are always appreciated.
Lamar
@EverezzeAll comments
- @andrewteeceSubmitted 11 months ago@EverezzePosted 11 months ago
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 helpful0 - @vsharma12Submitted about 3 years ago
Can i make it better? please review it, it would be helpful!! Thanks :)
@EverezzePosted about 3 years agoHi, 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 helpful1 - @angelostdSubmitted about 3 years ago
Any feedbback is always welcome!~
:D(I think the HTML validations is fixed, but the report doesn't update).
@EverezzePosted about 3 years agoHi 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 helpful0