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
Port LoginRequiredDialog
to an AccountModalWidget
#11614
base: master
Are you sure you want to change the base?
Conversation
And thereby not embed a complete `QDialog` in the settings window. Fixes: #11592
src/gui/accountmodalwidget.h
Outdated
@@ -27,10 +28,19 @@ class AccountModalWidget : public QWidget | |||
public: | |||
AccountModalWidget(const QString &title, QWidget *widget, QWidget *parent); | |||
|
|||
enum DialogCode { Rejected, Accepted }; |
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.
enum class. and q_emum please.
Would Result be a better name?
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.
done
59dcb89
to
73f38e9
Compare
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.
Is addModalLegacyDialog still used anywhere, if not could you open a follow-up and remove it?
The share dialog and the folder wizard still use it. |
And thereby not embed a complete
QDialog
in the settings window.Fixes: #11592