Skip to content
OverviewSolutions
10
Comments
5
Saif Shalan
@SafeNSound95

All comments

  • Saif Shalan•190
    @SafeNSound95Posted about 1 year ago

    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
  • Yateesh Gowda•40
    @keenyateesh19Submitted about 1 year ago

    Results Summary Component - Feedback Welcome!

    👋 Hello Community!

    I've just completed the Results Summary Component project. It's a responsive HTML and CSS design for displaying summary information. I'd love your input and suggestions to make it even better.

    🌟 Feedback Welcome:

    Your insights are invaluable in refining this design. Thanks for your help!

    Best, Yateesh

    Responsive 3 Card Preview Component

    1
    Saif Shalan•190
    @SafeNSound95Posted about 1 year ago

    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
    @SafeNSound95Posted about 1 year ago

    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
    @SafeNSound95Posted about 1 year ago

    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
    @SafeNSound95Posted about 1 year ago

    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