This does not do anything of the kind of fixing bug. It just accepts the event to tell that event receiver is interested in listening to it, and ignore just ignores the event for any specific scenario.
Details
Diff Detail
- Repository
- R223 Okular
- Branch
- event
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 7984 Build 8002: arc lint + arc unit
Context? What bug is this fixing? Why is this necessary?
That would be good information to put into the Summary section. :)
Unless you have a bug that needs this fix, i really prefer if you don't touch event handling
The ping is on your side, you still haven't explained why this is needed and what it fixes
I told you in my previous comment that it was handled the same way in ktexteditor, that is why I added it.
You have little conversation output, and I have little understanding of QEvent.
But lets start to explain why the event should not be propagated to another widget, although there was no annotation or the mouseMode is not TextSelect? Alternatively: Which other widget would need right/middle clicks?
And why does mouseDoubleClickEvent() ignore() events and mouseWheelEvent() not? Is there anything special with double clicks?