Burak Apaydın• 50
@burakapaydin
Submitted
José Alfredo Mário Dos Santos
@Josemario17
All comments
@Josemario17
Posted
awesome, great work. keep on
1- PCloud Custodian• 80
@joeterlecki
Submitted
Here is my second solution and I incorporated the feedback from my first challenge.
My question would be, how would you change the dynamic photo for responsive design? In this case, a second image was provided and the only way to do it with an img src tag was to use srcset and put the media queries in HTML. To me, this seems like an anti-pattern as those breakpoints and image changes should be in the CSS styling. But that brings another issue to me. If you define an image in background styles in a div, then what's the appropriate way to apply alt values for accessibility?
@Josemario17
Posted
you can change it by screen size using the picture tag incorporated with the source tag and then the img tag, see this at: https://www.w3schools.com/tags/tag_picture.asp
keep on
0 - Kraken Garrit• 10
@IsraeliPS
Submitted
I think it was a fairly simple project, made with SCSS and React that facilitates modularization
I appreciate any comments or suggestions for improvement
@Josemario17
Posted
Good job, for this solution my suggestion is the following: increase the height of the container or decrease its width, to make it as square as possible, I hope I helped.
enjoy the code
0 - metehankasapp• 160
@metehankasapp
Submitted
thanks :)
@Josemario17
Posted
I did the same thing kkkk, continue in this one that is too beautiful.
0 - Shashikant• 430
@shashikantdev3
Submitted
Responsive Product Preview Card Component | Insp by Kevin Powell's Sol
#accessibility#bem#progressive-enhancement@Josemario17
Posted
great work, but it isn´t in center viweport
1
