Sam Mathew
@dayumsamAll comments
- @DjboundawSubmitted almost 3 years ago@dayumsamPosted almost 3 years ago
It looks great!
Try using semantic tags: If you look at your report there are 4 accessibility issues almost all of them are because you lack a semantic tag
Solution: change your card div from a div to a <main> that should fix it.
Also can also change the font in your document to the one in the style guide
Keep it up!
Marked as helpful0 - @WesleyKaiharaSubmitted almost 3 years ago
How can I resolve the small responsiveness error, or in a real case it wouldn't affect it?
@dayumsamPosted almost 3 years agoGreat effort! you should try using semantic tags like <main> <header> <footer> instead of <div> which do not tell the browser about the content.
In your project for example you can replace the element with class Prices_NFTs to a <main class="Prices_NFTs"> instead of div.
Keep up the efforts!
Marked as helpful0 - @ivandro-netoSubmitted almost 3 years ago
Hi, I would like receive your opinion and tips to improve my code.
@dayumsamPosted almost 3 years agoIt looks great! try changing the font color on the paragraph from 212, 45%, 89% to 220, 15%, 55%
0