@jamesbarnett
Posted
-
Semantic HTML Yes. Always good to include an h1 and nice use of article as well.
-
Accessibility Using pixels for font sizes breaks the users ability to zoom. I think this article explains it nicely: https://www.seeratawan.me/blog/why-you-should-move-from-px-to-rem-in-css/ Another nice addition for accessibility would be to utilize the HTML time tag: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/time
-
Layout/Screen Sizes Looks great! Everything is proportional and reasonable sized.
-
Code Clean markup and CSS. Great job!
-
Differ If memory serves on this challenge, isn't there supposed to be a hover effect that changes the dropshadow?
Marked as helpful