Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

3-column-card-responsive-challenge

Phyu Sin Linβ€’ 60

@oliviaphyu

Desktop design screenshot for the 3-column preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


This is my third frontend mentor challenge and I would like to receive any feedback from this community.

Community feedback

Ferβ€’ 3,970

@fernandolapaz

Posted

Hi πŸ‘‹, perhaps some of this may interest you:

HTML / ACCESSIBILITY:

  • The main content of every page (the card in this case) should be wrapped with the <main> tag.
  • Every page should have an <h1> to improve user experience and because it is an important element when it comes to SEO. The headings you used are fine, maybe add it at the beginning of the page and hide it with CSS for example...
  • The icons are decorative images and therefore need an empty alt attribute to be ignored by a screen reader.
  • Remember that <button> should be used for any interaction that performs an action on the current page and <a> should be used for any interaction that navigates to another view. It seems that links are more appropriate in this case.

CSS:

  • As part of the CSS reset you did, it's important to include the box-sizing property to allow us to include padding and border in the overall width and height of an element.
* {
box-sizing: border-box;
margin: 0;
padding: 0;
}
  • It is better to use min-height: 100vh; for the body, as using height causes the page to be cut off in viewports with small height (such as mobile landscape orientation).
  • It might be good to get used to designing with the Mobile first approach, which means designing for mobile first and then for desktop or any other device, as it is widely considered best practice.

Please let me know if you want more info on any of these topics or disagree with something.

I hope it’s useful πŸ™‚

Regards,

Marked as helpful

0

Phyu Sin Linβ€’ 60

@oliviaphyu

Posted

@fernandolapaz Thank you so much for this enlighting feedback. I don't know how to solve the page cut-off problem until you instruct me to use "min-height: 100vh" instead of "height: 100vh".

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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