@0xabdulkhaliq
Posted
Hello there π. Congratulations on successfully completing the challenge! π
- I have a suggestion regarding your code that I believe will be of great interest to you.
HTML π·οΈ:
- The
<span>PERFUME</span>
, must want to be changed to normal case likePerfume
- Itβs not great for screen readers, because uppercase text will be read out as individual letters (like abbreviations). So CSS is read as C.S.S. Just as PERFUME would be read as P.E.R.F.U.M.E, not as the word "Perfume"
- You can read more about that in this article on Medium
- Don't worry we can use
text-transform: uppercase
to transform the letter andletter-spacing: 1rem
for the textPerfume
- If you have any questions or need further clarification, you can always check out
my submission
and/or feel free to reach out to me.
- Additionally, you need to update the Preview Site as "https://pabloodev.github.io/product-preview-card/". Seems like you updated your repository name but you forget to update the preview link so that this is happening. Please do update so that it could help other reviewers!
.
I hope you find this helpful π Above all, the solution you submitted is great !
Happy coding!
Marked as helpful