Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dragon Ball Devolution no download option in context menu #5696

Closed
MaverickHunter07 opened this issue Nov 21, 2021 · 5 comments · Fixed by #10127
Closed

Dragon Ball Devolution no download option in context menu #5696

MaverickHunter07 opened this issue Nov 21, 2021 · 5 comments · Fixed by #10127
Labels
bug Something isn't working extension Related to the Ruffle WebExtension

Comments

@MaverickHunter07
Copy link

Describe the bug

I manually installed the extension in Microsoft Edge, then opened the game. When I right clicked it, there was no download option in the context menu. I made sure to toggle it on in the settings menu. The game's official website is https://www.txori.com/index.php?static5/dbdevolution.
Context menu:
screenshot (95)
My settings:
screenshot (96)
Someone on Ruffle's official discord (nosamu#3385) stated that they "think it must be a conflict between the selfhosted and extension versions of Ruffle, since the website is self-hosting" and that they "think that even though the extension version of Ruffle is loaded, the preferences are loaded from the webpage instead of from the extension".
You can see the full Discord conversation here: https://discord.com/channels/610531541889581066/722872096002408529/911812753604378634.

Expected behavior

I expected there to be a download option in the context menu.

Affected platform

Browser's extension

Operating system

Windows 10

Browser

Microsoft Edge Version 96.0.1054.29 (Official build) (64-bit)

Additional information

No response

@MaverickHunter07 MaverickHunter07 added the bug Something isn't working label Nov 21, 2021
@Poopooracoocoo Poopooracoocoo added the extension Related to the Ruffle WebExtension label Nov 22, 2021
@ousia
Copy link
Contributor

ousia commented Nov 22, 2021

@MaverickHunter07, sorry but I think I’m not getting your point here.

Preferences such as this one should be read from the selfhosted extension (if the site has any).

There may be legitimate reasons to publish animations and to disallow downloading them.

If that could be bypassed by installing the browser extension, it would make no sense to have the option on the selfhosted extension.

Many thanks for your help.

@danielhjacobs
Copy link
Contributor

danielhjacobs commented Nov 22, 2021

@ousia This was added to the extension options (the same place as where you choose whether to load Ruffle in the current tab) but the self-hosted configuration takes priority. showSwfDownload defaults to false, so you can't download the game if the site self-hosts, and most site-owners are not going to explicitly allow the swf to be downloaded in the configuration, but may not have a problem with users doing so either.

This issue also affects logLevel and warnOnUnsupportedContent.

Edit: See #5825

@ousia
Copy link
Contributor

ousia commented Nov 23, 2021

@danielhjacobs, many thanks for your reply.

I have tried it at both https://free-culture.tk/ and https://free-culture.tk/original/.

The first one uses a loader (https://free-culture.tk/lfc-loader.swf) for the actual presentation (https://free-culture.tk/lfc-presentation.swf) with loadMovie().

In such a case, it seems that only the loader can be downloaded (btw, I don’t know whether this is intended).

I think that a good compromise solution would be that if and only if the site owner explicitly sets showSwfDownload: false, this configuration should be honored by the browser extension.

Both users and site owners would have options in this way.

@Txori
Copy link
Contributor

Txori commented Mar 16, 2023

This isn't a bug. showSwfDownload being set to false works exactly as I intend it to on my website.
Please support my work if you want the game to continue evolving.

@danielhjacobs
Copy link
Contributor

Hi @Txori. I agree with you that the extension options should not take precedence over the options that a website owner explicitly sets, and the PR I created will not do so. However, there is a current bug where the extension options do not get taken into account even for options that are not set by a website owner. If you have showSwfDownload set to false, there will be no option to download the SWF even with that PR that closes this issue, but if you don't set showSwfDownload, someone with the extension should be able to download the SWF if they toggle that option in their extension, and they will be able to do so after that PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working extension Related to the Ruffle WebExtension
Projects
None yet
5 participants