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

avm2: Support AIR application arguments #17609

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Mesteery
Copy link
Contributor

Depends on #17299

@Mesteery Mesteery marked this pull request as draft August 22, 2024 10:17
@evilpie evilpie added the air Adobe AIR label Aug 25, 2024
@Mesteery Mesteery marked this pull request as ready for review September 9, 2024 19:36
@danielhjacobs danielhjacobs added A-avm2 Area: AVM2 (ActionScript 3) T-feature Type: New Feature (that Flash doesn't have) labels Sep 17, 2024
Copy link
Contributor

@Dinnerbone Dinnerbone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay in reviewing this. A few comments but otherwise looks good to me, thank you!

@@ -1,6 +1,7 @@
network-settings = Paramètres réseau
player-settings = Paramètres du lecteur
movie-parameters = Paramètres de l'animation
air-arguments = Arguments de l'application AIR
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI this will be deleted, these files are auto generated - you need to provide the translation on crowdin after the PR lands

}
});

Grid::new("open-file-params")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ID needs to be unique

@@ -337,6 +337,10 @@ impl OpenDialog {
self.movie_parameters(locale, ui);
});

ui.collapsing(text(locale, "air-arguments"), |ui| {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we only show this if the player type is air?

@@ -667,6 +667,13 @@ export interface BaseLoadOptions {
* This allows you to emulate Adobe AIR or Adobe Flash Player.
*/
playerRuntime?: PlayerRuntime;

/**
* Arguments variables to pass to the AIR application.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Arguments variables to pass to the AIR application.
* Arguments to make available to the content through AIR's `NativeApplication.arguments`. This is only applicable if `playerRuntime` is `"air"`.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-avm2 Area: AVM2 (ActionScript 3) air Adobe AIR T-feature Type: New Feature (that Flash doesn't have)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants