Vanilla CSS, HTML5 Semantic Tags, Custom Properties & Flexbox
Design comparison
Solution retrospective
This is an attempt to improve the code:
- Adding responsiveness to the landing page using media queries.
- Layout the container with flexbox
- Semantic tags
- Modularity All feedback is welcome Thank you.
Community feedback
- @correlucasPosted about 2 years ago
πΎHello Faris, congratulations on your solution!
Nice code and nice solution! You did a good job here putting everything together. Iβve some suggestions for you:
To get closer to
overlay effect
on the photo as the Figma Design its better you usemix-blend-mode
. All you need is thediv
under theimage
with thisbackground-color: hsl(277, 64%, 61%);
and applymix-blend-mode: multiply
andopacity: 80%
on theimg
orpicture
selector to activate the overlay blending the image with the color of the div. See the code bellow:img { mix-blend-mode: multiply; opacity: 80%; }
Here's a good article explaining these effects with mix-blend-mode:
https://developer.mozilla.org/en-US/docs/Web/CSS/mix-blend-mode
βοΈ I hope this helps you and happy coding!
Marked as helpful1@Faris-ThibaniPosted about 2 years ago@correlucas Thank you for the feedback the code was added to the project visit the github repo, perhaps you'd be happy to read the README.md file where you received a nice mention :D
0 - @VCaramesPosted about 2 years ago
Hey @Faris-Thibani, some suggestions to improve you code:
-
The stat numbers are not headings. They should be wrapped in a Paragraph Element.
-
The statistics section is a list of statistics, so it should be built using an Unordered List along with a List Items Element.
-
The image serve no other purpose than to be decorative; It adds no value. The Alt Tag should left blank and have an aria-hidden=βtrueβ to hides it from assistive technology.
-
To get the image to look like the FEM example, you are going to want to use the Mix-Blend-Mode along with the Multiply Value and include a Opacity with the value of 0.8.
Code:
img { opacity: 0.8; mix-blend-mode: multiply; }
- Implement a Mobile First approach π± > π₯
With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.
Happy Coding! π»π
Marked as helpful1@Faris-ThibaniPosted about 2 years ago@vcarames Thank you for the feedback the code was added to the project visit the github repo, perhaps you'd be happy to read the README.md file where you received a nice mention :D
0 -
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