@vanzasetia
Posted
Hello, Samson! 👋
Congratulations on completing this challenge! 🎉 It's great that you have used the landmark elements correctly. Also, it looks good on both mobile portrait and landscape views. Good effort on this challenge! 👍
Regarding your questions,
- When I was doing this challenge, I used
mix-blend-mode
to create the purple overlay and it looked darker than the design. But, when I gave feedback to @DarrickFauvel that had done this challenge, he took a look at my solution and gave me a suggestion that I should control theopacity
as well. So, because you are using the absolute position method then I would recommend trying to usemix-blend-mode
and you could see his solution as your reference if you want. - The sizes on the
style-guide.md
have nothing to do with the media queries. They are telling you that "this is how your website should look like at these screen sizes". As frontend developers, we should keep making your website looks good in between those screen sizes. Also, like @GrzywN has said earlier you could make the layout into two columns until it has enough space.
I would recommend using a ul
for the stats and wrapping each stats item with li
. By doing that you don't need to wrap each text with p
, you can just wrap the text with span
because the text is already wrapped by li
, which is a meaningful element.
That's it! I hope this information is useful! 😁
Marked as helpful
@samsonsham
Posted
@vanzasetia Thank you so much for your supportive words and helpful feedback! I followed your instruction to not only apply mix-blend-mode
but also update a bit of opacity but no luck. @DarrickFauvel solution is so nice and I would definitely want to check out what's the trick to do that. I also updated the stat part to use ul
li
instead.
For responsive design, should I well define all resolutions then write the style to different media query respectively, or do the mobile-first and only write the necessary style to a desktop media query?
@DarrickFauvel
Posted
@vanzasetia Thanks for the mention. 😊
@vanzasetia
Posted
@samsonsham No, I would highly recommend using mix-blend-mode
and changing the opacity
instead of using absolute positioning to create the overlay.
I would recommend writing the styling with the mobile-first approach, which means that write all the styling for the mobile layout first and then uses the min-width
media query to style bigger screen sizes. By doing that, you will most likely find yourself writing less code, and also mobile users won't be required to process all of the desktop styles. 😉
@samsonsham
Posted
@vanzasetia Thank you so much! I think I could manage to make a very close mix effect finally. And the css is also updated to mobile first approach!
@vanzasetia
Posted
@samsonsham Good job with the update! It looks great on my mobile view both landscape and portrait mode! 👏
Keep it up! 👍