Ubuntu
qtmir package

Mouse wheel does not work consistently

Bug #1497091 reported by Daniel van Vugt
44
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Canonical Pocket Desktop
Fix Released
High
kevin gunn
qtmir (Ubuntu)
Fix Released
Undecided
Daniel d'Andrada
qtubuntu (Ubuntu)
Fix Released
Undecided
Lukáš Tinkl

Bug Description

Mouse wheel does not work (using Unity8 desktop and the browser in wily).

Although I suspect this might also be QtMir's fault like bug 1487366.

Tags: input

Related branches

lp://qastaging/~lukas-kde/qtubuntu/wheelEvent
Daniel d'Andrada (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 66 lines (+37/-10)
1 file modified
src/ubuntumirclient/input.cpp (+37/-10)
lp://qastaging/~lukas-kde/qtmir/wheelEvent
Daniel d'Andrada (community): Abstain
Gerry Boland (community): Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Diff: 268 lines (+111/-17)
8 files modified
src/modules/Unity/Application/mirsurface.cpp (+38/-8)
src/modules/Unity/Application/mirsurface.h (+1/-0)
src/modules/Unity/Application/mirsurfaceinterface.h (+1/-0)
src/modules/Unity/Application/mirsurfaceitem.cpp (+5/-1)
src/platforms/mirserver/qteventfeeder.cpp (+54/-7)
src/platforms/mirserver/qteventfeeder.h (+6/-0)
tests/mirserver/QtEventFeeder/mock_qtwindowsystem.h (+5/-1)
tests/modules/common/fake_mirsurface.h (+1/-0)
lp://qastaging/~lukas-kde/qtubuntu/fixWheel
Daniel d'Andrada (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Gerry Boland: Pending requested
Diff: 26 lines (+5/-6)
1 file modified
src/ubuntumirclient/input.cpp (+5/-6)
lp://qastaging/~dandrader/unity8/mouseWheel
Lukáš Tinkl (community): Approve
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Diff: 77 lines (+16/-4)
4 files modified
CMakeLists.txt (+1/-1)
debian/control (+3/-3)
plugins/Cursor/MousePointer.cpp (+11/-0)
plugins/Cursor/MousePointer.h (+1/-0)
lp://qastaging/~dandrader/qtmir/fixWheel
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Lukáš Tinkl (community): Approve
Daniel d'Andrada: Pending requested
Gerry Boland: Pending requested
Diff: 189 lines (+46/-43)
7 files modified
CMakeLists.txt (+1/-1)
debian/control (+2/-2)
src/platforms/mirserver/cursor.cpp (+21/-0)
src/platforms/mirserver/cursor.h (+1/-0)
src/platforms/mirserver/qteventfeeder.cpp (+19/-30)
src/platforms/mirserver/qteventfeeder.h (+1/-6)
tests/mirserver/QtEventFeeder/mock_qtwindowsystem.h (+1/-4)
Daniel van Vugt (vanvugt)
summary: - Mouse wheel does not work
+ Mouse wheel does not work in Unity8 apps
Revision history for this message
Olivier Tilloy (osomon) wrote : Re: Mouse wheel does not work in Unity8 apps

The wheel works in webbrowser-app in unity7.

Changed in webbrowser-app (Ubuntu):
status: New → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in qtmir (Ubuntu):
status: New → Confirmed
Changed in unity8 (Ubuntu):
status: New → Confirmed
kevin gunn (kgunn72)
no longer affects: webbrowser-app (Ubuntu)
Changed in canonical-pocket-desktop:
assignee: nobody → kevin gunn (kgunn72)
importance: Undecided → High
Gerry Boland (gerboland)
Changed in qtubuntu:
status: New → In Progress
assignee: nobody → Gerry Boland (gerboland)
Daniel van Vugt (vanvugt)
Changed in qtmir:
status: New → Confirmed
Lukáš Tinkl (lukas-kde)
Changed in qtubuntu:
assignee: Gerry Boland (gerboland) → Lukáš Tinkl (lukas-kde)
Lukáš Tinkl (lukas-kde)
Changed in qtmir (Ubuntu):
assignee: nobody → Lukáš Tinkl (lukas-kde)
status: Confirmed → In Progress
Lukáš Tinkl (lukas-kde)
no longer affects: unity8 (Ubuntu)
Changed in qtubuntu:
status: In Progress → Fix Committed
Changed in qtmir (Ubuntu):
status: In Progress → Fix Committed
Lukáš Tinkl (lukas-kde)
Changed in qtubuntu:
status: Fix Committed → In Progress
Changed in qtmir (Ubuntu):
status: Fix Committed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtmir - 0.4.6+15.10.20151021-0ubuntu1

---------------
qtmir (0.4.6+15.10.20151021-0ubuntu1) wily; urgency=medium

  [ Alan Griffiths ]
  * Opaquify MirWindowManager to control visibility of upcoming Window
    Management work

  [ Albert Astals Cid ]
  * Don't search for the element again

  [ CI Train Bot ]
  * New rebuild forced.

  [ Daniel d'Andrada ]
  * Improve multimonitor support
  * MirSurfaceItem: Survive holding a surface with an empty texture
  * Shell draws its own cursor using the new Cursor QML element

  [ Gerry Boland ]
  * Initial multimonitor support - react correctly to Mir
    DisplayConfiguration changes. (LP: #1488831, #1488863, #1436735)
  * Workaround for AutoPilot input coordinate positioning being outside
    screen geometry

  [ Lukáš Tinkl ]
  * Implement support for mouse wheel events; correctly pass around
    buttons (LP: #1497091)
  * React to surface modifications (window caption)

  [ Michał Sawicz ]
  * Improve multimonitor support
  * MirSurfaceItem: Survive holding a surface with an empty texture
  * React to surface modifications (window caption)

  [ Nick Dedekind ]
  * Added touch performance tracing and test.
  * Removed the manipulation of the CMAKE_INSTALL_PREFIX from
    debian/rules

 -- Michał Sawicz <email address hidden> Wed, 21 Oct 2015 11:47:16 +0000

Changed in qtmir (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
kevin gunn (kgunn72) wrote :

@ltinkl did your qtmir wheel event branch not make into silo22?
i think i still see this in the very latest image with silo 22 landed

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

@kgunn seems to work fine here, can't reproduce

Revision history for this message
kevin gunn (kgunn72) wrote :

also, per IRC discussion. mouse wheel seems to work fine for browser scrolling. however, with dash or ubuntu-system-settings the mouse wheel does not scroll. (at least ltinkl thinks it may intermittently, kgunn can't get any scrolling on dash/u-s-s)

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

That's probably expected. I'm sure the dash and System Settings would still need some additional logic to respond to the scroll wheel.

When I reported this bug, I was confident of where the problem was because I knew Xmir had working wheel support in Mir's demo shells. But Xmir like any Mir client needed support for mouse wheel added prior to that.

kevin gunn (kgunn72)
summary: - Mouse wheel does not work in Unity8 apps
+ Mouse wheel does not work consistently
Changed in qtmir (Ubuntu):
status: Fix Released → In Progress
kevin gunn (kgunn72)
Changed in canonical-pocket-desktop:
status: New → In Progress
Lukáš Tinkl (lukas-kde)
Changed in qtmir:
status: Confirmed → In Progress
assignee: nobody → Lukáš Tinkl (lukas-kde)
Daniel d'Andrada (dandrader)
Changed in qtmir:
assignee: Lukáš Tinkl (lukas-kde) → Daniel d'Andrada (dandrader)
Changed in qtmir (Ubuntu):
assignee: Lukáš Tinkl (lukas-kde) → Daniel d'Andrada (dandrader)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package qtmir - 0.4.6+16.04.20151125-0ubuntu1

---------------
qtmir (0.4.6+16.04.20151125-0ubuntu1) xenial; urgency=medium

  [ Daniel d'Andrada ]
  * Forward Mir mouse wheel events to the shell cursor (LP: #1497091)
  * Implemented support for cursors set by client surfaces
  * Revert revision 415

  [ Gerry Boland ]
  * Manage frameSwapped signal/slot connection with MirSurface more
    strictly to avoid crash. (LP: #1517571)

 -- Michał Sawicz <email address hidden> Wed, 25 Nov 2015 15:38:57 +0000

Changed in qtmir (Ubuntu):
status: In Progress → Fix Released
Michał Sawicz (saviq)
Changed in qtubuntu:
status: In Progress → Fix Released
Changed in qtmir:
status: In Progress → Fix Released
Changed in canonical-pocket-desktop:
status: In Progress → Fix Committed
kevin gunn (kgunn72)
Changed in canonical-pocket-desktop:
status: Fix Committed → Fix Released
Michał Sawicz (saviq)
affects: qtubuntu → qtubuntu (Ubuntu)
Michał Sawicz (saviq)
no longer affects: qtmir
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.