Skip to content
OverviewSolutions
13
Comments
5
José Alfredo Mário Dos Santos
@Josemario17

All comments

  • José Alfredo Mário Dos Santos•160
    @Josemario17Posted over 1 year ago

    awesome, great work. keep on

    1
  • Cloud Custodian•80
    @joeterleckiSubmitted over 1 year ago

    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?

    Product Preview Card | HTML and CSS Only

    2
    José Alfredo Mário Dos Santos•160
    @Josemario17Posted over 1 year ago

    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
  • José Alfredo Mário Dos Santos•160
    @Josemario17Posted over 1 year ago

    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
  • José Alfredo Mário Dos Santos•160
    @Josemario17Posted over 1 year ago

    I did the same thing kkkk, continue in this one that is too beautiful.

    0
  • José Alfredo Mário Dos Santos•160
    @Josemario17Posted over 1 year ago

    great work, but it isn´t in center viweport

    1