idukpaye alex• 725
@Esesosa-max
Posted
- ✅ Good Looking Design(both mobile , desktop) 5/5
- ✅ Perfectly Responsive 5/5
- ✅ Good Application of Tools(BEM,SASS/SCSS) 5/5
- ✅ Readablity 3.5/5
- 🥈 Pixel Perfect 4.6/5
- 💡 Personal Tip: You do not have to two images just change the image with CSS when it in mobile view e.g
content:url(path/to/image). In Mobile give the <br > adisplay:none
Marked as helpful
1
Karol Binkowski• 1,640
@GrzywN
Posted
@Esesosa-max Thanks for your feedback :)
Next time I'll definitely use content. I didn't even know that it is possible to use it like that. Some things regarding readability could have been done better, but I'm short on time, so yeah. I will improve this solution in a few days.
0
John Mirage• 1,610
@john-mirage
Posted
@GrzywN You can also use this:
<picture>
<source srcset="path/to/desktop.png" media="(min-width: 992px)">
<img src="path/to/mobile.png" alt="description">
</picture>
1
Karol Binkowski• 1,640
@GrzywN
Posted
@ChaosDynamix Yeah, I read article about picture tag after I submitted, but I prefer to use classes and media queries inside css than using media queries in HTML. Thanks tho
0
