@Aatypic
Posted
Hello and congratulations on your first solution, it's looking very good !
Concerning your question, I'm a newbie but I think on others challenges with more content would be easier to point out what can be improved accessibility wise.
-For the prices you could have a <span>
with an .sr-only
class explaining which one is, kind of like their class name. Other than that the texts are self explanatory.
-Another thing in the button you can achieve the same results without the <p>
element and I don't think it's appropriate for it's content, you already have the <img> and the <span>.
Anyway some minor stuff
Good luck ✌️
Marked as helpful
@KellyCHI22
Posted
Hello @Aatypic, thank you for your feedback!
About your suggestions:
- I didn't know about
.sr-only
, thank you! I will find more info on that. - At first I was thinking that I couldn't set both
display: flex
anddisplay: block
on the button at the same time, but it turned out I can! This way I can deleted the <p> element. Thank you!
Good luck for your learning too :)