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
Fix 2981 #2985
Fix 2981 #2985
Conversation
Moved bindings for VaultOptionsComponent and HealthCheckComponent from MainWindowModule to FxApplicationModule
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the naming in UnlockInvalidMountpointController
:
I thought longer about keeping just the boolean showPreferences()
function, because what we actually doin is deciding between two options and not just swichting on option off. Hence, please refactor to a proper decision model (create enum, add another function showVaultOptions()
etc)
src/main/java/org/cryptomator/ui/fxapp/FxApplicationWindows.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cryptomator/ui/unlock/UnlockInvalidMountPointController.java
Outdated
Show resolved
Hide resolved
src/main/java/org/cryptomator/ui/unlock/UnlockInvalidMountPointController.java
Outdated
Show resolved
Hide resolved
Resolved by a3c953d. |
Closes #2981
This PR adds support for showing either the "Open Vault Options" or the "Open Preferences" button in the Invalid mountpoint dialog, depending on the actual type of invalid custom mountpoint.
This required refactoring the internal logic of how the dialog is populated and moving the dagger bindings for
VaultOptionsComponent
andHealthCheckComponent
fromMainWindowModule
toFxApplicationModule.
The latter change can be seen below:
Changes in dagger graph introduced by e5047c2
Before:
After:
Furthermore the utility method
#showVaultOptionsWindow()
was added toFxApplicationWindows
to facilitate easy access to the vault options. This method is also subject of #2988.