Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Saif Shalanā€¢ 190

    @SafeNSound95

    Posted

    Great job.

    a couple of improvements: text-transform: uppercase for the header, a lot more line-height for the paragraph text, a little bit less padding for the buttons, and finally use outline instead of border for the hover state for the button, as border will make it jump when hovered while outline will produce a much more elegant effect.

    Marked as helpful

    0
  • Saif Shalanā€¢ 190

    @SafeNSound95

    Posted

    Congratulations, great work.

    I'd only suggest having more padding on the sides of each card to allow the content to breathe more, but other than it's perfect.

    0
  • Saif Shalanā€¢ 190

    @SafeNSound95

    Posted

    Very good eye for details.

    a couple of notes though:

    I think centering the card component like the original design intended would look better, as well as pushing the attribution to the bottom.

    a slight increase of padding-left and padding-right would allow the card to breathe more as well.

    Marked as helpful

    1
  • Saif Shalanā€¢ 190

    @SafeNSound95

    Posted

    congratulations! very good job. but maybe some padding-bottom or margin-bottom to the <p> element would make it more appealing and closer to the design.

    I'd also lessen the effect of the box shadow. but these are very little things, the overall design is very close to the original so congrats again.

    Marked as helpful

    0
  • Saif Shalanā€¢ 190

    @SafeNSound95

    Posted

    good job, but I'd add more padding between the QR image and the white box. also it's better to adhere to the design guide provided in the challenge.

    Marked as helpful

    1