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

core: Default value logic for MovieClip.tabEnabled #15618

Merged
merged 3 commits into from Apr 1, 2024

Conversation

kjarosh
Copy link
Contributor

@kjarosh kjarosh commented Mar 17, 2024

Related to #5443.

The default value of MovieClip.tabEnabled is neither trivial nor well-documented. This patch implements the logic behind the default value of MovieClip.tabEnabled and covers it with tests.

@danielhjacobs danielhjacobs added avm1 AVM1 (ActionScript 1.0 and 2.0) issues input Issues relating to user input in Flash content and removed avm1 AVM1 (ActionScript 1.0 and 2.0) issues labels Mar 22, 2024
@kjarosh kjarosh force-pushed the tab-ordering-movieclip branch 2 times, most recently from 87141be to 5880d85 Compare March 27, 2024 00:32
@kjarosh kjarosh marked this pull request as ready for review March 27, 2024 00:34
@danielhjacobs danielhjacobs added the waiting-on-review Waiting on review from a Ruffle team member label Mar 27, 2024
@kjarosh kjarosh force-pushed the tab-ordering-movieclip branch 3 times, most recently from ace333a to 3439ba2 Compare March 29, 2024 23:30
@kjarosh kjarosh force-pushed the tab-ordering-movieclip branch 2 times, most recently from c1e1b11 to d4671d5 Compare April 1, 2024 21:42
The default value of `MovieClip.tabEnabled` is neither trivial
nor well-documented. This patch implements the logic behind
the default value of `MovieClip.tabEnabled` and covers it with tests.
This test verifies the logic behind the default value of
`MovieClip.tabEnabled` when it's undefined.
@torokati44 torokati44 enabled auto-merge (rebase) April 1, 2024 22:14
@torokati44 torokati44 merged commit 501b42c into ruffle-rs:master Apr 1, 2024
17 checks passed
@kjarosh kjarosh deleted the tab-ordering-movieclip branch April 1, 2024 23:20
@Lord-McSweeney Lord-McSweeney removed the waiting-on-review Waiting on review from a Ruffle team member label Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
input Issues relating to user input in Flash content
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

4 participants