W631IJ• 30
@W631IJ
Submitted
MarqCodesFromZero
@MarqCodesFromZero
All comments
@MarqCodesFromZero
Posted
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- Aleksander Zwoliński• 120
@alzwo1
Submitted
@MarqCodesFromZero
Posted
That's why he's the GOAT! nice no changes needed.
0 - Michael Nwankwo• 20
@yourstrulycreator
Submitted
What 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
@MarqCodesFromZero
Posted
Not 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 - Peter Godspower• 470
@TheBeyonder616
Submitted
What 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
@MarqCodesFromZero
Posted
Pretty 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 helpful
1
