-
Notifications
You must be signed in to change notification settings - Fork 816
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
gui/macOS: Remove "advanced settings" section for macOS VFS settings #7928
Conversation
85c3477
to
324121b
Compare
|
text: qsTr("Request sync") | ||
visible: !root.syncStatus.syncing | ||
hoverEnabled: true | ||
onClicked: root.controller.signalFileProviderDomain(root.accountUserIdAtHost) |
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.
I think you meant to emit the domainSignalRequested
signal here
Signed-off-by: Claudio Cambra <[email protected]>
… component Signed-off-by: Claudio Cambra <[email protected]>
…le Provider settings Signed-off-by: Claudio Cambra <[email protected]>
… tooltip in request sync button Signed-off-by: Claudio Cambra <[email protected]>
324121b
to
e7fcb91
Compare
/backport to stable-3.16 |
Artifact containing the AppImage: nextcloud-appimage-pr-7928.zip SHA256 checksum: To test this change/fix you can download the above artifact file, unzip it, and run it. Please make sure to quit your existing Nextcloud app and backup your data. |
It's not necessary. Space can be saved by simplifying the text for the button and moving it alongside the sync status item
After changes: