Design comparison
Community feedback
- @0xabdulkhaliqPosted over 1 year ago
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.
POINTING CURSOR ↗️:
- Looks like the component's
button
element has not a pointer, this property plays a major-role in terms of both UI & UX
- The
cursor: pointer
CSS property is important for button-like elements because it changes the cursor from the default arrow to a pointer when hovering over the element. This provides a visual cue to the user that the element is clickable and encourages interaction.
- In terms of UI/UX, using
cursor: pointer
helps to improve the usability of the interface by making it easier for users to identify interactive elements. It also helps to provide feedback to the user by indicating which elements are clickable and which are not.
- So we want to add this property to the following add to cart
button
element
.butom { cursor: pointer; }
- Now your component's
button
has got the pointer & you learned about this property as well
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
0 - @Kira-SannPosted over 1 year ago
Não e necessário colocar cada elemento dentro de uma div em vez disso faça algo assim
<section class="content"> <p class="title">titulo</p> <h1 class="sub-title">subtitulo</h1> <p class="description"> Lorem ipsum dolor sit amet consectetur adipisicing elit. Nobis, consequuntur! </p> <div class="price"> <h2 class="discoumted-price">$149.99</h2> <p class="full-price">$169.99</p> </div> <button class="btn">...</button> </section>
Depois use os atributos css para estilizar
Vi que vc usou vários
<br>
não e legal fazer isso use o css para dar espaçamentos e ajustar posições.Além disso utilize apenas um idioma no seu código vi que usou português e inglês escolha apenas um deles de preferencia o inglês mas se não souber inglês não tem problema usar o português desde que seja apenas ele.
✅ recomenda-se usar html semântico como
<main>
,<section>
, <footer> entre outros ao invés de somente<div>
isso melhora o SEO do sitese quiser ver meu codigo ➡️click here
0
Please log in to post a comment
Log in with GitHubJoin 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