mrah898• 20
@mrah898
Submitted
P
Austin Tanksley
@austin-tanksley
All comments
- PAustin Tanksley• 40
@austin-tanksley
Posted
Hi Mrah, hope you are doing well. It looks like you were on the right path with this. Though there are a couple things that could be improved. Firstly I would make sure to add a border and a box shadow to the card. A quick google search will help with setting up the box shadow parameter. I can't seem to see your source code, but I would make sure that your alignments and justifies are matching the design. Spacing looks close but could be improved a bit.
0 - Istifanus Bako• 30
@istiefanus
Submitted
PAustin Tanksley• 40@austin-tanksley
Posted
Hello Istifanus, this looks great! I think the only think I would change would to make the heading link change on hover. For this you can add a css selector for the link like so a:hover{} Nice Work!
0 - Jandrie Klopper• 10
@jandrieklopper
Submitted
PAustin Tanksley• 40@austin-tanksley
Posted
Hi Jandrie, great work on this! A couple adjustments I would make would be to make the body take up the full height of the preview pane, for this use 100vh. Another thing would be to adjust the font size of the heading to 22px to match the original design. And lastly, it is looking like the box shadow is missing from your card. Over all though it is very close. the sizing of the card looks accurate, and spacing feels in the ballpark. Nice job!
Marked as helpful
0
