Last active
April 3, 2019 16:40
-
-
Save mattzeunert/7e1d8f3bfd2ce7aa29ab1b4e6790966c to your computer and use it in GitHub Desktop.
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Lighthouse homepage
Before:

After:

Adobe homepage
Their nested footer nav uses
visibility: hiddenon the child nav item container andmax-height: 0on the parent. Before we didn't pick up on the child items not being top-level and failed the footer as a result:Before:
After:

GoDaddy UK
The social links contain a large child image, but have a smaller container with
overflow: hidden. This causes problems for us because we don't consideroverflow: hiddenwhen finding the tappable rects.With
elementFromPointvisibility detection we wrongly decide that the tap target is invisible, because the center of the largest rect is invisible (because of theoverflow: hiddencontainer)Before:
After:
Etsy
We now correctly detect a small position absolute tap target (which has
opacity: 0when unfocused)Before:

After:

Eventbrite
These buttons are fairly big (40px) but they are also partially on top of the main tap target for this item, so the overlap ratio is large. I guess this is fine, and they should make their tap targets at least 48px large?
Before:

After:
