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

Add actionlistener for combobox in DialogOptionComponent #5369

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

AaronGullickson
Copy link
Member

@AaronGullickson AaronGullickson commented Apr 17, 2024

This is needed for story arcs. The CreateCharacterDialog tracks how much XP has been used on skills and abilities as players make choices, but currently doesn't have any ability to track changes when they are made to JComboBoxes because there is no listener for those kinds of options. So, this PR adds a listener for those cases. I temporarily disable the listener when the addValue method is called so that the listener is not triggered during initialization.

This PR needs to be done at the same time as PRs in megameklab and MekHQ because they use the DialogOptionListener. Specifically:

For now, it should come after .19 is out.

@AaronGullickson AaronGullickson added the For New Dev Cycle This PR should be merged at the beginning of a dev cycle label Apr 17, 2024
@SJuliez SJuliez merged commit 94111f0 into MegaMek:master Apr 20, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For New Dev Cycle This PR should be merged at the beginning of a dev cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants