Hi, this is a really good start!
I have a few improvement suggestions for you, but all pretty easy to fix
- you don't need to set min-width 100vw on the body. Body is 100% wide anyway and this can cause you problems on larger pages with scrollbars
- it's invalid html to have the whole card in a header element
- at the moment your solution breaks on mobile landscape. It's probably due to a height instead of min-height somewhere
- the stats shouldn't be in separate paragraphs, like 80k followers needs to be read as one sentence or list item. Instead, I suggest you place both parts inside one paragraph tag, but then use spans with classes on them to style them display block and change the things like font weight etc to get them looking right.
That's all though really, very minor changes overall, so very well done on this! 👍
@ajayv1
Posted
Hi Grace,
I was stuck at the background image of the body and checked your solution, Your code is real inspiration to me, the screen reader for the seo, I would have never thought of it.
Getting comments from you feel like I am on the right path. I sincerely appreciate it and will add all the suggestions you gave.
Thanks.
My pleasure. And really well done on this :)
@ajayv1 I checked again and it's these things that would fix the landscape mobile issue:
body {
/* overflow: hidden; */
}
.card-component {
padding: 1rem; // or similar
}
@ajayv1
Posted
Thanks, Grace.
It fixed the landscape issue. I also added the other suggestions you gave.
