HTML, CSS, mobile first

Community feedback

• 

  P

  Ken•4,915

  @kens-visuals

  Posted over 3 years ago

  Hey @fullstackdeveloper3 👋🏻

  I have some suggestion for the layout of the project and some other minor things.

  • Frankly, I'm not sure what this is data-lt-installed="true" and Google didn't have any useful answer, but I think that line is causing an accessibility issue, try removing it and generating a new report to see if the issue goes away.
  • For the layout, I'd suggest giving .component width: 100%; and .container width: 80%;. On mobile size, the image overlay height: 99%; it should be 100%. Also, for the .numbers, I suggest removing justify-content: space-between; and changing margin-right on spans to this margin-right: 90px;

  I hope this was helpful 👨🏻‍💻 you did a great job by making it responsive, those suggestions will make the desktop version to look more like the original design. Cheers 👾

  23

  Marked as helpful
• 

  Matthew•20

  @Krisix12

  Posted over 3 years ago

  You linked the mobile photo to desktop. Have to do it responsive so with css change the desktop photo to the mobile.

  11

  Marked as helpful