tth05 / obsidian-completr

Auto-completion plugin for the obsidian editor.
MIT License
291 stars 18 forks source link

Add completions for Obsidian Callouts #44

Closed eth-p closed 1 year ago

eth-p commented 1 year ago

Adds completion/suggestion support for Obsidian callouts.

image

Reviewer Note

Each commit is a distinct change, and is easier to review one-by-one instead of as a whole.

tth05 commented 1 year ago

On a side note: I was also planning on adding icons, but representing the provider that contributed the given suggestion instead. This would of course conflict with your icons. Do you have an idea how this could be handled? Two icons would look weird, even if one was in the front and one was in the back. What I'm thinking of right now is maybe the provider icon taking the whole space if the suggestion doesn't have an icon, otherwise it'd be displayed in the corner (C for Callout): image

eth-p commented 1 year ago

Regarding the side note:

I prefer only seeing the provider when there isn't an icon, as a way to help keep the suggestions all nicely aligned in a grid. But, I also see the value in knowing where the suggestions came from. In terms of usability and aesthetics, I think your idea of overlying the provider on top of the icon in the corner is the best approach, though. At the same time, I'm a fan of configurability and "having it your way", so maybe it could be an option in the settings?

For example:

tth05 commented 1 year ago

Regarding the side note:

I prefer only seeing the provider when there isn't an icon, as a way to help keep the suggestions all nicely aligned in a grid. But, I also see the value in knowing where the suggestions came from. In terms of usability and aesthetics, I think your idea of overlying the provider on top of the icon in the corner is the best approach, though. At the same time, I'm a fan of configurability and "having it your way", so maybe it could be an option in the settings?

Multiple options seem like the best way. I extracted your images into a new issue.