Design comparison
SolutionDesign
Solution retrospective
What can i improve?
Community feedback
- @pikapikamartPosted almost 3 years ago
Hey, awesome work on this one. Though the desktop layout is stretched and the text-content are not aligned on the left side like what it should be, but still points for this one. On the responsive state, when I resize my screen, I don't know why the site seems sluggish and if you go to like 780px, you will notice that the layout shrunk and the text are overflowing the container. The mobile state looks great though.
Some suggestions on the site would be:
- Making the overlay a bit darker would be really nice so that it will match the design.
- When using
img
tag, you don't need to add words that relates to "graphic" such as "img" and others, sinceimg
is already an image so no need to describe it as one. If you think an image is meaningful, make sure to use a descriptive/meaningfulalt
value. - Making the alignment for the text-content on the left would be really nice:>
- The
insights
text being purple is missing, might want to look on that one. - You could used
ul
for the.stats
selector since those are list of items that are inside it. - Each item inside the
.stats
are not really heading text at all. By making it one, those text doesn't really give information on what the section would contain as heading tag does. Usingp
tag on all of them would be great. - Lastly, just addressing the responsive state of the layout^^
Aside from those, great job again on this one.
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord