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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[qt] Make Close the default button on PPD and actived ond ESC #7956
Conversation
Signed-off-by: Ferenc G茅czi <ferenc.gm@gmail.com>
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.
It makes "Close" a default button for Enter press (before it was "Route from").
But it doesn't fix the original bug with Esc press or wndow close button click.
upd: qt6.2.4
77911b9
to
141c235
Compare
True. How about now? |
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.
works great, thanks!
@@ -26,6 +28,7 @@ void addCommonButtons(QDialog * this_, QDialogButtonBox * dbb, bool shouldShowEd | |||
|
|||
QPushButton * routeToButton = new QPushButton("Route To"); | |||
routeToButton->setIcon(QIcon(":/navig64/point-finish.png")); | |||
routeToButton->setAutoDefault(false); |
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.
Why these calls are needed? To avoid capturing "Enter"?
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, if the user doesn't move the cursor, then the enter would activate the first where the autodefault is not disabled.
No description provided.