Skip to content
OverviewSolutions
1
Comments
2
ioan
@JohanPeraldi

All comments

  • ioan•50
    @JohanPeraldiPosted over 1 year ago

    That’s pretty close, well done! I can see three small things you can do to make your solution even closer to the design that was provided:

    1. Change the colour of the h2 title (which could/should? be an h1) to the dark-blue colour provided in the style-guide.md -> hsl(218, 44%, 22%);
    2. Add a box-shadow to your container;
    3. Decrease the spacing between the QR code image and the container, almost by half. Either remove the padding or give your imgBox a 100% width instead of 90%. Other than that it looks good!
    Marked as helpful
    2
  • ioan•50
    @JohanPeraldiPosted over 1 year ago

    It looks pretty good to me! I can only see two things to make your solution even closer to the design that was provided:

    1. You can add a box-shadow around the card to make it stand out better, and
    2. The title text is a dark blue colour (refer to the style-guide.md file) and you seem to have left it with the default black colour.

    Other than that I can't see anything else to point out.

    Marked as helpful
    1