touch-packages team mailing list archive

[Zoltan Balogh <zoltan.balogh@xxxxxxxxxxxxx>]

** Changed in: ubuntu-ui-toolkit (Ubuntu)
       Status: New => Confirmed

** Changed in: ubuntu-ui-toolkit (Ubuntu)
     Assignee: (unassigned) => Zsombor Egri (zsombi)

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to ubuntu-ui-toolkit in
Ubuntu.
https://bugs.launchpad.net/bugs/1253608

Title:
  topMost InverseMouseArea should removeEventFilter when disabled

Status in Ubuntu UI Toolkit:
  Confirmed
Status in “ubuntu-ui-toolkit” package in Ubuntu:
  Confirmed

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://bugreports.qt-project.org/browse/QTBUG-32004
  [2] - https://codereview.qt-project.org/#change,71241

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1253608/+subscriptions