@Joffenhopland
Latest solutions
Latest comments
- @vanzasetia@Smita-14
It's surprising to know that you coded this on a phone so well!✨✨ It is brilliantly done! 👏👏
Marked as helpful - @asifm37@Smita-14
Hey Asif! The reason is that the image is an inline element and properties like font size and line heights can affect it causing it to have little space (padding) beneath it. Set the
display: block;to the image element and the issue will be fixed.Marked as helpful - @timssy@Smita-14
Hey CJ Castillo! Awesome submission! 👏 There's minor issues that you need to look at though:
- border radius is being applied on wrong edges on smaller screen size.
- The size of the cards increases from the bottom the moment the layout hits size smaller than 1440px.
Marked as helpful - @Geoff-Walker@Smita-14
Hey Geoff! You did an excellent work on this challenge. I noticed one issue that on smaller mobile screen size some part of text-block is clipped or it becomes hidden. Hope you take a look on that. Keep it up!
Marked as helpful - @Jimender@Smita-14
Hey Jimender! Your submission is great and almost pixel perfect! Just a few things I would like to point out:
- The attribution part is floating over the card on the mobile screen size.
- The card is trimmed from top on smaller screen size.
- Use semantic html wherever possible. It will help you avoid many issues like the accessibility and html issues.
