I went through and tested a bunch of themes and took some notes. Some of these are bugs and some of these are more questions/opinions. If you think something is done/addressed/etc, then just check it off. Some of these may already be obvious, but when I wasn't sure I added them to the list.
- [ ] Should ESC clear the search and then exit the overlay? Currently immediately exits the overlay. I feel like I want it to only exit the overlay once it has cleared the box. Maybe we should also rescroll the page back up to the top when the user first hits ESC?
- [ ] There should be an X in search box for clearing it right?
This is being added as part of this PR: https://github.com/Automattic/jetpack/pull/14423
- [ ] Should we have a max height on the titles? Some themes are very large
I think that's something we'll have to accept since the overlay is inheriting styles from the base theme. Undeniably some will look better than others, but my best guess is that the minute we impose some harsher rules for all themes is the moment we'll break compatibility with some of them.
I do think we should keep logging these to evaluate if it's a bigger problem than assumed initially.
A lot of cases where my browser autocomplete still shows after the overlay opens
What browser is this in @gibrown?
I wonder if we can prevent it by disabling search inputs that aren't in the overlay when the overlay is open.
Theme bug in zerif-lite: bad css on filters: https://cloudup.com/cZ6Tl7CN-io for http://gibrown.wpsandbox.me/demo/?blog_id=20115252&theme=zerif-lite&s=related
Should be fixed in #14493.
What browser is this in @gibrown?
I'm using Chrome.
@keoshi adding to this I was playing around with mobile and had a few questions:
Broke all of these out into separate issues so we can separately prioritize them.