Ubuntu
ubuntu-ui-toolkit package

Right clicks go through popovers

Bug #1488540 reported by Olivier Tilloy
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Medium
Cris Dywan

Bug Description

When displaying e.g. an ActionSelectionPopover (but this is valid for generic Popovers too), clicking anywhere outside of the popover will dismiss it by default. A left-click inside the popover will typically trigger the action under the cursor. But a right-click is not intercepted and goes through the popover, to be processed by whatever is below it. I would tend to think that this is unwanted behaviour in the general case.

Related branches

lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/clickThru
Zsombor Egri: Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 260 lines (+209/-0)
4 files modified
src/Ubuntu/Components/Popups/1.2/PopupBase.qml (+1/-0)
src/Ubuntu/Components/Popups/1.3/PopupBase.qml (+1/-0)
tests/unit_x11/tst_components/tst_popover.qml (+33/-0)
tests/unit_x11/tst_components/tst_popover13.qml (+174/-0)
Revision history for this message
Cris Dywan (kalikiana) wrote :

Can you give an example? I can't see anything like that - I tried with a Popover and right-clicking with Button's underneath but the clicks never make it through.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Incomplete
Revision history for this message
Olivier Tilloy (osomon) wrote :

Attaching a simple standalone reproducer.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Incomplete → New
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
assignee: nobody → Christian Dywan (kalikiana)
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1688+15.10.20151018.1-0ubuntu1

---------------
ubuntu-ui-toolkit (1.3.1688+15.10.20151018.1-0ubuntu1) wily; urgency=medium

  [ Tim Peeters ]
  * Add delegate property to ActionBar.
  * Introduce PageHeader component.
  * Add header property to Page.
    Fixes LP: #1248898, LP: #1350893, LP: #1445016, LP: #1492328
  * Use the correct navigation overflow icon in PageHeader.

  [ Loïc Molinari ]
  * Fixed OpenGL ES 2 mipmap fallback. OpenGL ES 2 doesn't support
    GL_TEXTURE_MAX_LEVEL by default, it is only available via an extension. It
    was working on the devices where this extension is available but not on
    the emulator. Fixes LP: #1505221.

  [ Christian Dywan ]
  * Handle incubateObject returning null in PageWrapperUtils.
    Fixes LP: #1505197
  * Add module header to PageWrapperUtils.

  [ Zoltan Balogh ]
  * Improve test plan
    - Ad -q parameter to provision the device for normal use with a PPA
    - Fix up the camera and clock app on the device to acces services
    - Replace phablet tools with manual enabling writable image
    - Fix PPA pin up for staging and other regular PPAs.

  [ Zsombor Egri ]
  * Emit Dropped event for live ListItem drag'n'drop. Fixes LP: #1500118.

  [ Timo Jyrinki ]
  * Build dep on libinput-dev and libxkbcommon-dev (LP: #1437192).

  [ Benjamon Zeller ]
  * Remove unused variables.

  [ CI Train Bot ]
  * No-change rebuild.

 -- Zoltan Balogh <email address hidden> Sun, 18 Oct 2015 03:47:27 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

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