topMost InverseMouseArea should removeEventFilter when disabled
Bug #1253608 reported by
Daniel d'Andrada
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
Undecided
|
Zsombor Egri | ||
Bug Description
An InverseMouseArea that has the topMost property as true will install itself as an event filter. When it's disabled, there's no use for such event filtering and it will therefore be just useless overhead. Specially when that event filter is installed on the QGuiApplication, which gets quite a lot of events.
I actually don't get why to you install it on the QGuiApplication when your target (sensing area) is the rootItem. It would be great if there was a comment in the code explaining this strange decision. Maybe its because of the bug that you cannot filter events sent to QQuickItems [1] (whose fix is on the way [2]).
[1] - https:/
[2] - https:/
| tags: | added: inversemousearea |
| Changed in ubuntu-ui-toolkit: | |
| assignee: | nobody → Zsombor Egri (zsombi) |
| status: | New → Confirmed |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| assignee: | nobody → Zsombor Egri (zsombi) |
| status: | New → Confirmed |
| no longer affects: | ubuntu-ui-toolkit |
Revision history for this message
| Zsombor Egri (zsombi) wrote | #1 |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | Confirmed → Fix Released |
To post a comment you must log in.
This seems to be fixed some time, as we are no longer filtering the app events, but the window. So I'll mark it as fix released.