@loez97
Latest solutions
Latest comments
- @stuartbradley
I like your css code. It separated nicely and commented well. Same with the html, it is commented well.
The only thing i could raise is the background colour is not the same, but apart from that. Its a well designed page
- @LouisVerchWhat are you most proud of, and what would you do differently next time?
Getting more and more comfortable with HTML. Root CSS template is good as well. It might have been better to use grid for the social cards
What challenges did you encounter, and how did you overcome them?I had issues with flexbox divs especially the . I experimented before finding the right solution
What specific areas of your project would you like help with?Is my code okay or does it need to be modified ?
@stuartbradleyThe HTML and CSS is very clean and good use of variable within the CSS file. Something I'm going to add to my CSS files going forward.
- @Gentlefrank@stuartbradley
Your html is well commented. You could however work on your structuring and formatting of your code. There are a couple of instances where there are line breaks between tags that don't need to be there and hinder readability.
Its good you are including media queries, however they dont seem to work as intended. Using browser tools, I changed the width of the page and the card outline stays intact. However the text and images fall out of line of the card and it becomes skewed.
Marked as helpful - @venkatesh-khatake@stuartbradley
.
