MarqCodesFromZero
@MarqCodesFromZeroAll comments
- @W631IJSubmitted 4 months ago@MarqCodesFromZeroPosted 4 months ago
Good job overall!
Just missing some li{ margin-bottom: aPX} on list items.
also dont forget to add *{margin:0; padding:0}
so then you can have more manipulation of the padding and margin of each element. (might help you with the list alignment)
After you add the * property then you can add padding to the li{} so that it aligns with the other elements!
0 - @alzwo1Submitted 4 months ago@MarqCodesFromZeroPosted 4 months ago
That's why he's the GOAT! nice no changes needed.
0 - @yourstrulycreatorSubmitted 4 months agoWhat are you most proud of, and what would you do differently next time?
I have a mastery of basic css styling
What challenges did you encounter, and how did you overcome them?Making the author image and text centered was overcame by change use of display inline-block to inline-flex
What specific areas of your project would you like help with?Just clean code
@MarqCodesFromZeroPosted 4 months agoNot much to talk about besides using the box-shadow property in css to create the shadow on the right side of the Container, adding padding to "learning".
Also make the font colour on the preview title black.
0 - @TheBeyonder616Submitted 4 months agoWhat are you most proud of, and what would you do differently next time?
using semantic html5
What challenges did you encounter, and how did you overcome them?I was unable to use the figma file
What specific areas of your project would you like help with?help using the figma file
@MarqCodesFromZeroPosted 4 months agoPretty accurate...only thing to improve would probably removing the footer from inside the article to make it stay below the qr code. Then in the css, in the attribution class, add margin-top :12px for example to create some space between the footer and the qr code
Marked as helpful1