@ApplePieGiraffe
Posted
Hey, Aaron Gonzalez! 👋
Good job on this challenge! 👏
I suggest,
- Using a single
<h1>
tag for the text "Victor Crest 26" rather than an<h1>
and an<h2>
tag since those two elements really aren't two separate headings. You can use the<span>
tag to easily style the "26" differently than the rest of the text. - Another way of positioning the image that I might share would be to put in the
div.card-identifier
and use negative margin to slide it up into the background of the card. That way, you can group the image with the rest of the profile information and not have to use so much padding in thediv.card-identifier
to push the rest of the content down. 😉
Keep coding (and happy coding, too)! 😁
@aaronpaulgz
Posted
@ApplePieGiraffe Thanks, Very cool feedback 😄 I'll implement it as well
Virtual hugs!
@aaronpaulgz
Posted
@ApplePieGiraffe Feedback implemented :D Thanks