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 overlayheight: 99%;
it should be100%
. Also, for the.numbers
, I suggest removingjustify-content: space-between;
and changing margin-right on spans to thismargin-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 👾
Marked as helpful
@fullstackdeveloper3
Posted
@kens-visuals thanks for all these advise, but about the justify-content: space-between;
I replaced it with margin-right: 90px;
and it didn't look good so I return to the justify-content: space-between
, thank you a lot bro cause you help me always.
@fullstackdeveloper3 no worries bro. No pressure, If you like the way it is, don't listen to anybody, that's your project and before anyone else you have to like what you've created so keep it up ✌🏻
Marked as helpful
@fullstackdeveloper3
Posted
@kens-visuals thanks master.