I'd appreciate any feedback (":
Elizabeth Sotomayor
@elizabethrsotomayorAll comments
- @eslamwaleed1Submitted over 1 year ago@elizabethrsotomayorPosted over 1 year ago
Your solution looks great! One suggestion I have would be self-hosting your fonts using google-webfonts-helper rather than downloading fonts, which will allow your site to be more stable in case the link doesn't work for whatever reason. Also, you can use an
aria-label
for the h1 you have rather than making a whole separate class for it. One more thing, I think yourmain
section needs to have arole
. See here. I hope this helps!Marked as helpful1 - @dupakarovskySubmitted almost 2 years ago
Spent more time on this than a care to admit :/
@elizabethrsotomayorPosted almost 2 years agoYour solution looks nice! I like the hover effect for the links. I'm not familiar with SCSS but the only things I can think of would be to make the links at the top darker and the "new" link headings bolder. Also maybe make the "read more" link a little bit larger with less margin so everything lines up. Great work!
0