asterodeia• 20
@Asterodeiawd
Submitted
ioan
@JohanPeraldi
All comments
- ioan• 50
@JohanPeraldi
Posted
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:
- 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%);
- Add a box-shadow to your container;
- 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 - Ameya kawade• 30
@Cool-developer-975
Submitted
Any Suggestions or improvements?
ioan• 50@JohanPeraldi
Posted
It looks pretty good to me! I can only see two things to make your solution even closer to the design that was provided:
- You can add a box-shadow around the card to make it stand out better, and
- 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
