feat: add settings page search (@OmTiwariPandey)#8135
Conversation
|
Hi @OmTiwariPandey , thank you for your contribution. I am not sure a search feature is needed on the settings page, that is what the command line is for, where you can search and change settings in one go. E.g. searching for custom in the commandline will give you search results for the setting values as well: From the implementation I think it is quite odd to parse the page instead of using the raw data. |
|
Hi! I know. It is obviously up to you guys. But I asked many of my friends who use monkeytype and they said they didnt even know if the command thing existed using escape key because people usually expect it to be near the top. and they didnt even read the note that literally says to press esc to open the command thing. I was experiencing the same that is why I asked. once again, up to you guys, but I thought it might be a good idea. |
|
maybe we can add the commandline directly on top? |
|
Right. But dont you think that place for the command line would be kinda off cuz it plays more functions than just a search bar, and the command line still doesnt scroll to that area, just provides a minimal interface to change the setting quickly. While I know two search options would look kinda excessive but the command line can be used to serve a different purpose, or maybe both could coexist? Just sharing an opinion. Thanks for reading :D |
|
I talked to @Miodec about this feature:
The search would act like a filter, instead of showing the result list as dropdown only show the settings that match the filter. Match the name, description or any option |
|
On it! Will make another commit. |
|
Oh, and please implement this in solid-js directly, not parsing and modifying the page using javascript |
|
Okay! |
|
Just made the commit. Do check it out. Works as expected. Thanks :) |
|
Oh yeah! mb. on it. |
|
@fehmer Committed the changes. Please review. |
|
Looks good. Should we add a quick way to delete the current search? maybe an x on the right side. |
|
Good idea. Can do. Will do it in the next commit. |
|
did a couple commits. added a clear x button on the right side of the search so you can wipe the query in one click. |
|
@fehmer please review them and do ping me if you see any bugs. Thanks! Also I am sorry for all these pings lol. |



Description
The settings page has a lot of options cluttered, and finding a setting would mean scrolling through everything. you can press escape to search, yes but still it just makes you quickly change a setting, not searching it and its neighboring settings.
This PR adds a search box at the top of the settings page. As you type, matching settings show up in a dropdown, and picking one scrolls to that setting and briefly flashes it so it's easy to spot. You can also use arrow keys and enter to do the same.
A few notes on the approach:
Checks
Closes #