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

UserInteractions Refactoring #4379

Merged

Conversation

baconpaul
Copy link
Collaborator

Addresses #4337
Closes #2671 #3435

Refactor/Remove the UserInteractions in favor of JUCE equivalents
and appropirate wrappers

  • openURL -> juce::URL.lauch
  • showHTML -> surgeguieditor builtin
  • Remove openFileInFinder or whatnot for juce::URL(juce::File()).laucnh
  • promptInfo removed; single use replaced with an Alert box
  • Remove UserInteractions::promptError to be data driven
  • promptOKCancel moved to a gui free function which can be replaced and
    is replaced in SGE
  • promptFileOpen replaced with juce::FileChooser
  • Finally, remove the old files which were no longer used

Addresses surge-synthesizer#4337
Closes surge-synthesizer#2671 surge-synthesizer#3435

Refactor/Remove the UserInteractions in favor of JUCE equivalents
and appropirate wrappers

- openURL -> juce::URL.lauch
- showHTML -> surgeguieditor builtin
- Remove openFileInFinder or whatnot for juce::URL(juce::File()).laucnh
- promptInfo removed; single use replaced with an Alert box
- Remove UserInteractions::promptError to be data driven
- promptOKCancel moved to a gui free function which can be replaced and
  is replaced in SGE
- promptFileOpen replaced with juce::FileChooser
- Finally, remove the old files which were no longer used
@baconpaul baconpaul merged commit 4ca0fa8 into surge-synthesizer:xt-alpha Apr 23, 2021
@baconpaul baconpaul deleted the user-interactions-4337 branch April 23, 2021 22:08
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.

1 participant