@Alex-Archer-I
Posted
Hi!
Cool work! It looks neat and your code is clean =)
By the way, you can omit the source
with desktop image in the picture
tag - it will be loaded through img
. Also there is no need for aria-lable
for this button - it has a text which will be recognized by screen readers.
As for BEM - it's really hard to see and fell the advantages of this methodology on such small projects. It seems to me that you did it well (the product-category
class looks like it miss a pattern, but that is not a big deal =)).
Marked as helpful