Responsive solution using Flexbox only, no frameworks.

Community feedback

• 

  Joseph•770

  @Jos02378Posted about 3 years ago

  hey @JSaporski , great job on this solution!

  Some suggestions for you:

  1. For your stats, instead of splitting the h2 and the span tag, you can use <h2>10k+ <br><span>companies</span></h2> instead.
  2. I don't know how you manage to get the image to look like that without styling it but you can use an overlay for the image using a div just like in this video overlay video.
  3. You can try to follow a CSS naming convention called BEM. You can watch this video for more information Watch the video.
  4. You can try to use relative units like rem and em in the future. Here is a link to an article that explains when to use which unit see the article.

  I hope this helps, good luck!

  Marked as helpful

  0

  ---

  

  João Guilherme Saporski•210

  @JSaporskiPosted about 3 years ago

  @Jos02378 Oh! Thank you! I'm sure your tips are valuable, I'll use them in future projects too. On the image, I used Figma to manipulate the colors with a purple layer behind it, and then I merged the purple layer with the image, but your tip to use a div instead makes it much easier, thanks.

  0