scambier / obsidian-omnisearch

A search engine that "just works" for Obsidian. Supports OCR and PDF indexing.
GNU General Public License v3.0
1.24k stars 65 forks source link

[Feature request] UX improvements #343

Open ZeroEkkusu opened 9 months ago

ZeroEkkusu commented 9 months ago

Hi, @scambier,

I've discovered the plug-in in Obsidian's 2023 Gems and prefer it over the core plug-in.

I have a few small UX suggestions.


Is your feature request related to a problem? Please describe.

Describe the solution you'd like





xfadsf



Describe alternatives you've considered

Additional context

Thank you!

scambier commented 9 months ago

Hi ZeroEkkusu

I usually search inside the file. Add a configuration option to set the default to file or vault.

There is a command to directly open Omnisearch with the In-File context image

Drop the to. Just navigate, cycle history, open, etc. It's cleaner.

Not sure about that one. I don't disagree, but I'd rather keep following the naming convention from the Quick Switcher core plugin 🤷‍♂️

This is confusing. Instead, change the labels to Search file instead and Search vault instead

👍

ZeroEkkusu commented 9 months ago

Hey!

There is a command to directly open Omnisearch with the In-File context

This is inconvenient. I use the ribbon icon on both mobile and desktop, which sets the search to vault by default. Besides, the average user shouldn't have to deal with commands.

I'd rather keep following the naming convention from the Quick Switcher

OK.

scambier commented 9 months ago

I use the ribbon icon on both mobile and desktop, which sets the search to vault by default.

Noted, I haven't thought about this button 👍

Besides, the average user shouldn't have to deal with commands.

I mentioned the command because it can be bound to a keyboard shortcut, just like ctrl+o will open Quick Switcher by default

ZeroEkkusu commented 9 months ago

Heads up that the command to search file doesn't work on mobile (nothing happens when executed).

Also, the hint text is neither left-aligned, nor centered.

I'll add those two to the original post above.

ZeroEkkusu commented 9 months ago

Added.