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: don't ignore mousePressEvent
in SplitInput
#4177
Conversation
clang-tidy review says "All clean, LGTM! 👍" |
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.
Functionality seems correct, I alt-tabbed a bunch to see if I could re-replicate the old issue but I could not.
Co-authored-by: Felanbird <41973452+Felanbird@users.noreply.github.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.
Thanks for fixing my bug!
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
By default, a
QWidget
will ignore themousePressEvent
. Before #3838,SplitInput
had a handler for this event and thus didn't ignore the event. But the PR removed the event handler.This PR adds the handler back, and only calls the parent handler if the input is hidden. This has the effect that the context menu of the
ResizingTextEdit
is shown again, so that's what I mentioned in the changelog.Fixes #3844.