kxxt / obsidian-advanced-paste

Advanced pasting functionality for obsidian
MIT License
42 stars 2 forks source link

Breaks paste in Search field #7

Closed luckman212 closed 1 year ago

luckman212 commented 1 year ago

Thanks for this plugin!

Just a quick report: if bound to ⌘ + V, it breaks paste function when trying to paste a string into the Search field. Not sure if there is any way to detect & work around that.

kxxt commented 1 year ago

Thanks for your bug report! I will look for workarounds.In the mean time You can try binding the command to other shortcuts to avoid this bug.

kxxt commented 1 year ago

Fixed in v2.3.1. Please unbind ⌘ + V. The default paste command now just works.

luckman212 commented 1 year ago

Awesome - I will give it a try, thank you!

luckman212 commented 1 year ago

@kxxt I forgot to mention, I'm on a Mac. I don't think this is working. When I enabled the new feature and tried it using ⌘V, it did not trigger Advanced Paste. I had to revert back to the previous method.

screenshot_ioqMa6RL

kxxt commented 1 year ago

@luckman212 Did you restart obsidian after updating the plugin? It should work on Mac despite the keyboard shortcut is ⌘V.

See the release note for more info: https://github.com/kxxt/obsidian-advanced-paste/releases/tag/2.3.1

If it still not works, please attach an screenshot of the console. (View > Toggle Developer Tools)

luckman212 commented 1 year ago

I re-tested it a few times (yes I restarted Obsidian each time)

It's not working for me (?)

Screenshots:

image

image

image

kxxt commented 1 year ago

@luckman212 Can you test it in a sandbox?

image

I guess some other plugins handled the paste event so my plugin can't handle it.

luckman212 commented 1 year ago

I made a dumb mistake. Sorry!

I was thinking (why??) that Ctrt/Cmd+V would trigger my custom transform.js. Oops.

It works fine with the normal behavior. Hope I didn't waste too much of your time 😔

kxxt commented 1 year ago

No need to say sorry. We all make mistakes from time to time. :)