Stats Preview Card Component Challenge using Flexbox / pseudo element

Community feedback

• 

  Raymart Pamplona•16,100

  @pikapikamartPosted about 3 years ago

  Hey, awesome work on this one .Desktop layout looks fine, but resizing the screen-width, the layout is being hidden and creating horizontal scrollbar. Mobile state looks great though but your breakpoint of like 1020px is too big for all the mobile state to occupy don't you think. Desktop layout could use more of those.

  Some suggestions would be:

  • Always have a main element to wrap the main content of your page. On this one, the .main-div should be using the main instead of div.
  • For the purple overlay, lots of other suggestions use the mix-blend-mode :>
  • Your .stats-container could have used ul since those are "list" of information about the company website.
  • When wrapping a text-content do not just use div to wrap it, use meaningful element like a p tag if it just a regular text or heading tag if it is an heading.
  • Lastly, just for the mobile breakpoint to be adjusted so that desktop layout could be shown more.

  Aside from those, great job again on this one.

  Marked as helpful

  1