P
Matt Studdert• 13,611
@mattstuddert
Posted
Great work completing another challenge Malik, it looks great! 👍
Here are a couple of small pointers after taking a look at your code:
- A lot of the text in the design is uppercased, but I'd recommend against uppercasing that text directly in your HTML. This is because screen readers will typically readout that text letter-by-letter, instead of reading the whole word. To improve accessibility, I'd write the text as normal in your HTML and then use
text-transform: uppercase;
in your CSS to visually change it on the page. - This is a tiny point, but I'd take one more look at the spacing for the "to see a preview" text. You might want to reduce the
word-spacing
a bit but increase theletter-spacing
slightly. This is a really small issue though 🙂
Keep up the great work!
0