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

Product preview card component

beniusisโ€ข 400

@beniusis

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


That is my first submission. Any feedback or suggestions are greatly appreciated!

Community feedback

@MelvinAguilar

Posted

Hello there ๐Ÿ‘‹. Good job on completing the challenge !

I have some suggestions about your code that might interest you.

HTML ๐Ÿท๏ธ:

  • Use semantic elements such as <main> and <footer> to improve accessibility and organization of your page.
  • You should use only one <h1> tag per page. The <h1> tag is the most important heading tag, This can confuse screen reader users and search engines. You can read more about this here ๐Ÿ“˜.
  • Avoid using uppercase text in your HTML because screen readers will read it letter by letter. You can use the text-transform property to transform the text to uppercase in CSS.

I hope you find it useful! ๐Ÿ˜„ Above all, the solution you submitted is great!

Happy coding!

Marked as helpful

1

beniusisโ€ข 400

@beniusis

Posted

@MelvinAguilar Thanks for the feedback! Already implemented changes accordingly.

0
Jean Jackโ€ข 110

@JeanJackcss

Posted

It was very good and very well organized. Congratulations on your work. Keep going and your future will be bright.

0

beniusisโ€ข 400

@beniusis

Posted

@JeanJackcss Thanks!

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