Jetpack: Instant Search: Early Design Feedback

Created on 24 Jan 2020  路  8Comments  路  Source: Automattic/jetpack

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.

Instant Search Search [Type] Bug

All 8 comments

  • [ ] 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?

Related: https://github.com/Automattic/jetpack/issues/14306

  • [ ] 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:

  • I thought we had planned to have the filter button slide in the filters from the side and take over the screen rather than doing a drop down from the button? https://wp.me/p9xfpQ-yC#comment-1324 (though now I see in later Figma designs that that changed, may not be that big a deal)
  • I feel like as soon as any of the filter or sort options get selected we should close the filter box and reload the results?
  • Selecting the dropbox for sorting zooms in the page on Chrome mobile (iOS). https://cloudup.com/cVrNq5PxgCI
  • I guess the styling for the sorting options doesn't match the design either and it probably would be faster to change sorting if it is not a drop down on mobile. cc @jsnmoon

Broke all of these out into separate issues so we can separately prioritize them.

Was this page helpful?
0 / 5 - 0 ratings