feat: Improve spotlight UX

Open
#6,230 2 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
42/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
react-native, typescript

Research direction

Start with RoomsListView and the other Spotlight search implementations, then reproduce the delayed results by entering “ninguем” quickly on both Android and iOS while logged in as diego.mello2. Review the debounce and local-database flow; done means autocorrect and capitalization are disabled per platform and local results appear promptly before any server request fills remaining space.

Written by the indexing model from the issue text.

Description

Spotlight (RoomsListView and others search) takes too long to respond and has some awkward interactions with keyboard.

To be done
  • Disable auto correct and init cap

    • pay attention to test both on Android and iOS. Props are different.
  • Refactor the debounce logic to return results faster

    • Suggestion: create a mechanism to return results immediately (or almost) in case the search hits local db data only

    • Test: log in as diego.mello2 and enter ninguem quickly. It’s going to take a few seconds to show the results.

    • I think it might be requesting data right after realizing there’s not enough data locally.

      • We could instead return the local data first and trigger the server request to fill the empty space later.
Dominant language
TypeScript
Stars
2.4k
Forks
1.5k
Avg merge
1d 15h
Merged PRs (30d)
29

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from RocketChat/Rocket.Chat.ReactNative

All issues in RocketChat/Rocket.Chat.ReactNative

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.