HTML, CSS, Flexbox and grid

Community feedback

• 

  Agata Liberska•3,995

  @AgataLiberskaPosted over 3 years ago

  Hi @StudioWebpic, well done on your first project, it looks great!

  Two small things I noticed: You're using a media query for max-width of 1000px. At that point the card is 800px wide and still in column layout and there's quite a lot of unused space, which makes it look a bit off balance. What you could to is go a bit lower on your media query (so it goes from column to grid earlier) or / and set a max-width for your card div so it doesn't stretch too much before it switches to grid.

  And the second thing - you've used a <ul> for one of your boxes and removed the list style, which is great, but you've still got the padding on the left which is a default list style. I would personally remove it here, because it can wrap your text too much and make it hard to read. But if that was intentional - sorry!

  Hope this helps :)

  1

  ---

  

  Studio Webpic•40

  @StudioWebpicPosted over 3 years ago

  @AgataLiberska Thx you a lot for you feedback <3

  0