Dorion icon indicating copy to clipboard operation
Dorion copied to clipboard

[Feature Request] Right click able to copy video media link (shelter mod)

Open khewweifeng opened this issue 1 year ago • 6 comments

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Currently in Dorion, you can view image with left click and right click to copy image link but this does not apply for video media.

Describe the solution you'd like A clear and concise description of what you want to happen.

Able to copy video media link and paste to anywhere like official discord client

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Anything that can copy link into clipboard

Additional context Add any other context or screenshots about the feature request here.

Video didn't have the option to copy link, please refer to below attached photo.

image

khewweifeng avatar Jun 15 '24 13:06 khewweifeng

Edit: misunderstood what the screenshot was for lol sorry

SpikeHD avatar Jun 15 '24 17:06 SpikeHD

I'm not entirely sure if it's how Dorion is handling the view, if something changed with Edge(?), or Discord itself changed how it works. What I do know is that it was working prior, and then I updated Dorion to 5.0.1 and it borked 🙃 It seems to be some sort of mismatch on how to handle right clicks, whether they're a Discord Context right click or a Web Browser Right Click. Discord Desktop: image Dorion: image Web Client (Firefox): image Upon checking all 3 I'm inclined to believe it's how the web client is set up now, which is probably worse and more unfortunate.

The loss of contextual menus can also be seen with URLs: Desktop: image Dorion/Web Client: image

And the text input box: Desktop: image Dorion/Web Client: image

Wondering if this functionality returning will have to fall to a plugin. Thanks. (realizing now the reason my previous version of Dorion was still working is because the client had been open for 2 weeks 🤣 )

BrennanAyers avatar Jun 29 '24 01:06 BrennanAyers

Great investigation! Yea, it looks like they differentiate now. It's especially evident since the menu items' actions pass a string that specifies web_:

image

Weird nerf tbh. This could probably be brought back in a plugin but I haven't taken a deep dive yet.

SpikeHD avatar Jun 29 '24 03:06 SpikeHD

Oh! Vencord has a plugin that does this already (and it's enabled by default iirc). Can probably just see what it does and reimplement it shelter style.

SpikeHD avatar Jun 29 '24 03:06 SpikeHD

Oh! Vencord has a plugin that does this already. Can probably just see what it does and reimplement it shelter style.

Thanks, I will use this first

khewweifeng avatar Jun 29 '24 03:06 khewweifeng

oh, huh, I didn't see that plugin earlier. Thanks!

BrennanAyers avatar Jun 29 '24 04:06 BrennanAyers

I have gone over this again and the work involved to recreate something that already exists as a Vencord plugin doesn't feel worth my time. If someone else wants to make a Shelter-based context menu plugin, feel free to open a PR to add it as a default plugin in Dorion!

SpikeHD avatar Sep 30 '24 23:09 SpikeHD