-
Notifications
You must be signed in to change notification settings - Fork 668
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
Bugfix: add open local/remote folder options #9420
Conversation
416252c
to
b9fa51d
Compare
src/gui/accountsettings.cpp
Outdated
auto info = _model->infoForIndex(index); | ||
OC_ASSERT(info); | ||
if (auto info = _model->infoForIndex(index)) { | ||
QAction *ac = menu->addAction(tr("Open Remote Folder in Browser")); |
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.
see addAction(openLocallyLabel, this, []{
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.
Open in web browser maybe?
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.
Hmm, I wanted to make it clear that it's the remote folder that's being opened. But maybe that's too obvious. What do you think?
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.
src/gui/accountsettings.cpp
Outdated
auto info = _model->infoForIndex(index); | ||
OC_ASSERT(info); | ||
if (auto info = _model->infoForIndex(index)) { | ||
QAction *ac = menu->addAction(tr("Open Remote Folder in Browser")); | ||
QString path = info->_folder->remotePathTrailingSlash(); | ||
path += info->_path; |
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.
for the root folder this results in //
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.
No, remotePathTrailingSlash()
only appends the slash if there isn't already a trailing slash.
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.
yes but info->_path will be /
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've tried :D
7268a82
to
1ae5944
Compare
Add the "open local/remote folder" context menu items for non-sync-root items back into the accounts tab in the settings dialog. #9405
1ae5944
to
90af240
Compare
Kudos, SonarCloud Quality Gate passed! |
Add the "open local/remote folder" context menu items for non-sync-root
items back into the accounts tab in the settings dialog.
#9405