@ArtiqqSubmitted about 2 months ago
Nostromito
@NostromitoAll comments
- @ArtiqqSubmitted about 2 months ago@NostromitoPosted about 2 months ago
Looks much better than mine! Great work!
Marked as helpful0 - @Pan1402Submitted 2 months ago@NostromitoPosted 2 months ago
Looks amazing. The only thing I would do is use references from other words to get the font sizing right. Looks perfect other wise.
Marked as helpful1 - @Black-crypto-gifSubmitted 2 months ago@NostromitoPosted 2 months ago
It looks great. the only thing I noticed is the font color inside the rose background wasn't added. Keep it up!
Marked as helpful1 - @vic1500Submitted 2 months agoWhat are you most proud of, and what would you do differently next time?
What I'm most proud off in this project is the naming conventions that I came up with for the css classes
@NostromitoPosted 2 months agoEverything looks really good! I use min-height: 100vh; in the body element on css in order to bring down the container.
0 - @ArtiqqSubmitted 2 months ago@NostromitoPosted 2 months ago
Looks great!
The only thing I did differently was add a media query for the "html & css foundations" and made the font size smaller for mobile use so the text doesn't fold over.
Also making Mr. Hoopers photo smaller might help with the over all size.
don't take too much of my advice though. Still very new to this!
Marked as helpful0 - @Olakeys19Submitted 2 months ago
