Skip to content
OverviewSolutions
32
Comments
3
Lamar
@Everezze

All comments

  • Lamar•515
    @EverezzePosted 10 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 helpful
    0
  • varun•10
    @vsharma12Submitted about 3 years ago

    Can i make it better? please review it, it would be helpful!! Thanks :)

    PURE HTML AND CSS

    3
    Lamar•515
    @EverezzePosted about 3 years ago

    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
    @angelostdSubmitted about 3 years ago

    Any feedbback is always welcome!~
    :D

    (I think the HTML validations is fixed, but the report doesn't update).

    Only HTML and CSS

    2
    Lamar•515
    @EverezzePosted about 3 years ago

    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