Design comparison
Solution retrospective
Couldn't center main Div with comand "margin: 0 auto;"
Community feedback
- @MelvinAguilarPosted about 2 years ago
Hi @nenadmne ๐, good job on completing this challenge! ๐
Here are some suggestions you might consider:
Try to use semantic tags in your code. It should have the main tag that groups all of the product elements.
- For specificity reasons you should work better with
classes
since they are reusable, and you can leave theID
when you work with Javascript. - Add an h1 tag to your solution. The
<h1>
element is the main heading on a web page. There should only be one<h1>
tag per page, and always avoid skipping heading levels; Always start from<h1>
, followed by<h2>
, and so on up to <h6> (<h1>,<h2>,...,<h6>). The HTML Section Heading elements (Reference)
Solution:
<h1 class="heading">Gabrielle Essence Eau De Parfum</h1>
- You can use a <picture> tag when you need to change an image in different viewports. Using this tag will prevent the browser from loading both images, saving bandwidth and preventing you from utilizing a media query to modify the image.
Example:
<picture> <source media="(max-width: 576px)" srcset="./images/image-product-mobile.jpg"> <img src="./images/image-product-desktop.jpg" alt="your_alt_text"> </picture>
- The cart icon is for decoration purposes only, so it can be hidden from screen-readers by adding
aria-hidden="true"
and leaving its alt attribute empty:
<img class="cart" src="./images/icon-cart.svg" alt aria-hidden="true">
- You could use the <del> tag to display the old price:
<del class="small-price"> <span class="sr-only">Old price: </span>$169.99 </del>
Note that I added the <span> with the
sr-only
class to thedel
element, this will provide more information about what your old price is about.The
sr-only
class is a class that you can add to hide content visually but is only visible to screen-readers.I hope those tips will help you.
Good job, and happy coding!
Marked as helpful1 - For specificity reasons you should work better with
- @AdrianoEscarabotePosted about 2 years ago
Hi Nenad Cosovic, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
- every Html document must contain the main tag, so we can identify the main content, to fix this, wrap all the content with the main tag. HTML5 landmark elements are used to improve navigation experience on your site for users of assistive technology.
- images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty
alt=""
and addaria-hidden="true"
attributes to make all the assistive technologies of the Web, as screen reader. Learn the differences between decorative/meaningless images vs important content.
The rest is great!
I hope it helps... ๐
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