Skip to content

Add search history to search bar - #774

Open
Dominik7272 wants to merge 2 commits into
rr-:masterfrom
Dominik7272:feature/search-history
Open

Add search history to search bar#774
Dominik7272 wants to merge 2 commits into
rr-:masterfrom
Dominik7272:feature/search-history

Conversation

@Dominik7272

Copy link
Copy Markdown

Closes #768
This PR implements a client-side search history feature to the search bar.

Features

  • Searches will be stored per-account and synced across devices
  • Leaving the search bar empty will show 10 recent searches, pressing the "Show more" button shows 10 more
  • stores a max of 50 searches
  • disable search history and clear it in the settings

@po5

po5 commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Interesting, thank you!
Do you have screenshots of this feature?
I will test it out when I can, though it might take me a while.
I think this has a place in szurubooru and I'd like to merge it.

Nitpicks from a quick glance:
API requests should use uri.formatApiLink(), and calling require() within functions doesn't follow the rest of the project.
I can fix stuff up, it's fine.

@Dominik7272

Copy link
Copy Markdown
Author

Thanks for the feedback, I'm really glad to hear you'd like to merge it.
Regarding the nitpicks: I've went ahead and pushed a commit to resolve both.

Here are some screenshots below showing the search history dropdown:

Less than 10 queries:
image

More than 10 queries:
image

Options in the Settings menu:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add search history

2 participants