social-app icon indicating copy to clipboard operation
social-app copied to clipboard

Improve usability of search on web

Open haileyok opened this issue 10 months ago • 2 comments

Don't forget to revert TODO REVERT THESE CHANGES

Why

  • Confusing naming of query, which really is queryTerm
  • Inconsistent behavior on what the "X"/clear input button does compared to other websites/apps
  • No browser history when navigating back through subsequent searches
  • Extreme, ultra, incredible jank on web search input

How

Confusing naming

Pretty obvious from the diff, but I just renamed query to queryTerm here. query sounds more like it should be the result of a useQuery() hook, but it was just a string.

More to come...

haileyok avatar Apr 23 '24 21:04 haileyok

Your Render PR Server URL is https://social-app-pr-3663.onrender.com.

Follow its progress at https://dashboard.render.com/web/srv-cok23q0cmk4c7396qu40.

render[bot] avatar Apr 23 '24 21:04 render[bot]

Old size New size Diff
6.45 MB 6.43 MB -17.12 KB (-0.26%)


github-actions[bot] avatar Apr 23 '24 21:04 github-actions[bot]

If I visit a link for a search (e.g. https://social-app-pr-3663.onrender.com/search?q=cats ), Back shows up instead of Cancel, when there is no history and tapping it doesn't do anything.

This is in Safari on iPadOS 16.6.

IMG_5058

surfdude29 avatar Apr 25 '24 12:04 surfdude29

Sorry to jump in here again as I can see you two are both working hard on this, I wanted to give you a heads up that the user experience in Safari on iPad isn't great if you select the search bar after scrolling down the results a bit, the keyboard appearing causes the entire UI to move up:

https://github.com/bluesky-social/social-app/assets/149612116/24cf449e-f342-4640-a1e5-5454ec0498cf

surfdude29 avatar Apr 27 '24 01:04 surfdude29

@surfdude29 should be fixed, thanks.

gaearon avatar Apr 27 '24 03:04 gaearon

Also included:

  • Fix iOS keyboard jump
  • Reuse previous results while typing
  • Instantly show profile shell on autocomplete navigation
  • Put last clicked recent search at the top

gaearon avatar Apr 27 '24 03:04 gaearon

@surfdude29 should be fixed, thanks.

Confirmed fixed for me on main.bsky.dev, thanks for these great search improvements @haileyok and @gaearon! 🙏

surfdude29 avatar Apr 27 '24 14:04 surfdude29