Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Flexbox

Varvara 90

@Varvaron

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


My second project here. I had so many probles with text part (i thought it would the easiest one). I would like to hear a piece of advice about it

Only web version is ready for now.

Community feedback

@W1lsonCodes

Posted

Hey Varvara,

I have never really reviewed code before so you are my first.

I have 1 thing that i saw and that is your in the html. I believe you may have forgotten to move the style content from the html to the external css stylesheet.

I really liked your @import method for the fonts. I may have to switch over to that as it does make the code alot cleaner.

Look forward to see you're mobile version!

1

Varvara 90

@Varvaron

Posted

@W1lsonCodes I didn't move the style content purposely. For me it wasn't a part of a challenge, so i didn't want to mix it, i just changed color of links to make it the same as on entire page.

This method is very easy! All you need is to copy an information from Google Fonts, for examle. You will learn it in 5 seconds. Also i'm not sure that this way is the best, i just use it because it's comfortable for me now

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord