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

Adding some UI manifest attributes as parameters to the CreateManifest class. #158

Merged
merged 1 commit into from
May 2, 2019

Conversation

TurekBot
Copy link
Collaborator

@TurekBot TurekBot commented Apr 14, 2019

This should allow easier customization of the launcher UI because you
can tell FXLauncher to put the UI customizations in the manifest for you.

The parameters that I added—which already existed in FXManifest—are

  • updateText (--update-text)
  • updateLabelStyle (--update-label-style)
  • progressBarStyle (--progress-bar-style)
  • wrapperStyle (--wrapper-style)

class.

This should allow easier customization of the launcher UI because you
can tell FXLauncher to put the UI customizations in the manifest for you.

The parameters that I added—which already existed in Manifest—are
 * updateText (--update-text)
 * updateLabelStyle (--update-label-style)
 * progressBarStyle (--progress-bar-style)
 * wrapperStyle (--wrapper-style)
@TurekBot
Copy link
Collaborator Author

I realized after I updated my project that frknikiz in PR #144 had already included the updateText parameter. Sorry to move your code, but now it's in the same order as it is in the fxlauncher.FXManifest class. Hopefully, we're all good.

TurekBot added a commit to TurekBot/fxlauncher-gradle-plugin that referenced this pull request Apr 14, 2019
It allows you to customize the UI. The fields to customize the UI were
already apart of FXLauncher, there was just no easy way to change them.

The fields are
 * updateText (--update-text)
 * updateLabelStyle (--update-label-style)
 * progressBarStyle (--progress-bar-style)
 * wrapperStyle (--wrapper-style)
@TurekBot TurekBot requested a review from edvin April 30, 2019 20:57
@edvin edvin merged commit edfc7a5 into edvin:master May 2, 2019
@TurekBot
Copy link
Collaborator Author

TurekBot commented May 2, 2019

Thanks @edvin, really appreciate it.

@TurekBot TurekBot deleted the ui-parameters branch May 2, 2019 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants