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 preview card component

Ola135β€’ 60

@Ola135

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


All suggestions and hints are welcome. Thank you. :)

Community feedback

Abdul Khaliq πŸš€β€’ 72,660

@0xabdulkhaliq

Posted

Hello there πŸ‘‹. Congratulations on successfully completing the challenge! πŸŽ‰

  • I have other recommendations regarding your code that I believe will be of great interest to you.

DECORATIVE SVG'S ♨️:

  • The alt attribute is used to provide alternative text for images in HTML documents. The alt attribute is used by screen readers to describe the image to visually impaired users, which is essential for web accessibility.
  • Now, when it comes to decorative SVGs, they are used purely for aesthetic purposes and do not convey any important information or functionality to the user.
  • Since these images do not convey any important information or functionality, there is no need for an alt attribute.
  • So feel free to set the alt attribute as "" for decorative svg's, because alt="" will be skipped by screen readers they will consider the image as decoration

Example:

<img src="images/decorative.svg" alt="">

<img src="images/icon-luxury.svg" alt="Luxury-icon">
πŸ‘‡
<img src="images/icon-luxury.svg" alt="">

.

I hope you find this helpful πŸ˜„ Above all, the solution you submitted is great !

Happy coding!

Marked as helpful

1

Ola135β€’ 60

@Ola135

Posted

Hello @0xAbdulKhalid!

Thank you for the feedback and your time. I'll go through the code and make required changes.

Happy coding to you too! :)

0

@VCarames

Posted

Hey there! πŸ‘‹ Here are some recommendations for enhancing your code:

  • The β€œcar icons” in this component are purely decorative⚠️. Their alt tag should be left blank to remove them from assistive technology.

More Info:πŸ“š

https://www.w3.org/WAI/tutorials/images/

  • The headings in your component are being used incorrectly❌. Since the h1 heading can only be used once ⚠️, it is always given to the heading with the highest level of importance. This component has three headings of equal importance, so the best option would be to use an h2 heading βœ… since it is reusable and it will give each heading the same level of importance.
  • For improved accessibility πŸ“ˆ for your content, it is best practice βœ… to use rem for your font-size and other property values. While em is best for media-queries. Using these units gives users the ability to scale elements up and down, relative to a set value.
  • ALWAYS Implement a "Mobile First" approach πŸ“± > πŸ–₯

Mobile devices are now the dominant πŸ‘‘ way in which people browse the web. So when building your content, you will start building with small screen sizes (starting at 320px) and work your way to larger screens using min-width.

More Info: πŸ“š

Mobile First

If you have any questions or need further clarification, feel free to reach out to me.

Happy Coding! πŸ‘Ύ

Marked as helpful

1

Ola135β€’ 60

@Ola135

Posted

Hello @vcarames!

Thank you kindly for your feedback. It helps a lot.

Could you please explain to me whether it is necessary to use h1 heading at least once in the project or can we skip it and just use h2 heading as you suggested?

Thank you in advance. Happy coding to you too! ^^

1

@VCarames

Posted

@Ola135

  1. headings provide structure the content. An h1 is always giving to the introductory text in your site to let users know what the page is about. Think about a book; the title is an h1, the chapters are h2 and everything else is either a paragraph.
  2. This is a component, something that is meant to be part a larger site. So in the real world it would never have an h1. On top of that, since it has multiple headings of equal importance, and h2 will do fine.

Marked as helpful

1
NixyMcβ€’ 190

@Nix7amcm

Posted

Hey @Ola135! 😊

Excellent job with this challenge! πŸ˜ƒ

I have just a couple tips that might be helpful!

First, you might consider setting your breakpoint to something around 920px, switching to column a little earlier to prevent the box width becoming too small as the screen size narrows.

Then, within your query, to prevent the boxes then being too wide, you could set a max-width to your .card element, maybe something around 500px. You could play around with this to see what you like best!

I hope this helps! Great work! πŸš€πŸ˜ƒπŸ‘πŸ»

Marked as helpful

1

Ola135β€’ 60

@Ola135

Posted

Hello @Nix7amcm !

I am grateful for your comment. Thank you for your tips, these will definitely help.

Wish you the best and happy coding! ^^

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