Design comparison
Solution retrospective
Any feedback would be appreciated. π
Community feedback
- @statanasovaPosted almost 4 years ago
Hey, I have another tip :)
Try using the
<picture>
element to serve different images on mobile and desktop.This code will show the hero-mobile.jpg by default and will switch to hero-desktop.jpg once the resolution is 500px or larger. I use this in all of my projects nowadays :)
<picture> <source srcset="/images/hero-desktop.jpg" media="(min-width: 500px)" /> <img class="visual__image" src="/images/hero-mobile.jpg" alt="Girl with orange shirt" /> </picture>
Keep it up! Upvote if you found this helpful :)
1@Augusto248Posted over 3 years ago@statanasova
Hi statanasova. Thanks for your reply. π
I didn't know about the "picture" element. From now on I will use it. I think it's great for a more clean css.
0 - @irfan-fauziPosted almost 4 years ago
great work.. It`s responsive but..Try to open in mobile phone live.. try to slide it to right or left,,
1@Augusto248Posted almost 4 years ago@irfan-fauzi Hi irfan. Thanks for your reply.
You are right, I open it in my mobile phone and the scroll bar appeared at the bottom.
I see my code again and the problem was some padding in two elements that surpass the 375px.
0 - @abhik-bPosted almost 4 years ago
Hi Augusto π Merry Christmas ππ
Great job on this challenge , it is responsive and it works as desiredπ―π―
*Just a opinion ** ~ * adding
cursor : pointer
can make button look niceKeep contributing these awesome solutions π
1@Augusto248Posted almost 4 years ago@abhik-b Hi abhik-b. Thanks for your reply. π
I will take into account what you commented.
Merry Christmas ππ
1
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