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 button classes for use in mod options #1424

Merged
merged 2 commits into from
Feb 20, 2022
Merged

Add button classes for use in mod options #1424

merged 2 commits into from
Feb 20, 2022

Conversation

originalfoo
Copy link
Member

@originalfoo originalfoo commented Feb 18, 2022

Part of ongoing work for #1356

  • ActionButton triggers handler on click
  • UrlButton opens a url on click
  • OptionButtonBase is base for both buttons
  • OpenUrlsInSteamOverlay - new Main global option (also bumped global config version)
  • ...configurable via General tab option

In preparation for: #813, #733, #659, etc.

Later, once all mod options are using helper components, I plan on updating options screen to be able to 'refresh' when language is changed (rather than forcing C:SL to rebuild entire mod options library).

- `ActionButton` triggers handler on click
- `UrlButton` opens a url on click
- `OptionButtonBase` is base for both buttons
- `OpenUrlsInSteamOverlay` - new global option
- ...configurable via General tab option
@originalfoo originalfoo added docs Documentation UI User interface updates Settings Road config, mod options, config xml Blocking Another issue depends on this issue. labels Feb 18, 2022
@originalfoo originalfoo added this to the 11.6.5.1 milestone Feb 18, 2022
@originalfoo originalfoo self-assigned this Feb 18, 2022
Copy link
Collaborator

@kvakvs kvakvs left a comment

Choose a reason for hiding this comment

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

Nice

originalfoo added a commit that referenced this pull request Feb 19, 2022
@originalfoo
Copy link
Member Author

@krzychu124 any feedback on this?

TLM/TLM/UI/Helpers/ActionButton.cs Outdated Show resolved Hide resolved
originalfoo added a commit that referenced this pull request Feb 20, 2022
Based on feedback from another PR:

#1424 (review)
@krzychu124 krzychu124 self-requested a review February 20, 2022 21:49
@originalfoo originalfoo merged commit 065ec19 into master Feb 20, 2022
@originalfoo originalfoo deleted the option-button branch February 20, 2022 23:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocking Another issue depends on this issue. docs Documentation Settings Road config, mod options, config xml UI User interface updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants