Implement PDF open menu item for Desktop and Android

NomarCub / obsidian-copy-url-in-preview

Image Context Menus - Obsidian.md image context menus: copy, open in default app, show in system explorer, reveal in navigation context menu. Also Open PDF externally context menu.

MIT License

63 stars 8 forks source link

Implement PDF open menu item for Desktop and Android #10

Closed mnaoumov closed 2 years ago

mnaoumov commented 2 years ago

Fixes #9

NomarCub commented 2 years ago

Is it normal that the menu only appears on about every third hover?
Also, did the normal right-click menu not work? This hover behaviour doesn't mesh well with the others for me.

mnaoumov commented 2 years ago

Every third hover is definitely not right. I worked for me fine, but I'll take another look.

Right click menu doesn't work as this JavaScript event doesn't bubble up in the existing markup

mnaoumov commented 2 years ago

@NomarCub slightly improved the UX

NomarCub commented 2 years ago

Hovers are consistent now, thanks! I will merge and publish shortly. Depending on future feedback I might make this optional (and disabled by default, as it's kind of different from the other features).