Popover steals focus on open

Bug #1528178 reported by Albert Astals Cid
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Michał Sawicz
ubuntu-ui-toolkit (Ubuntu)
Won't Fix
Undecided
Unassigned
unity8 (Ubuntu)
Fix Released
High
Albert Astals Cid

Bug Description

Steps:
* go into dash, search for something
* cancel the search
* press the search button again

Expected:
* history pops up
* search field focused and OSK comes up

Current:
* history pops up
* search field unfocused and no OSK

Until the version 1.3.1761+16.04.20151216.1-0ubuntu1 (and included) of ubuntu-ui-toolkit the following code

http://paste.ubuntu.com/14126916/

Kept the focus on the TextField after pressing on it.

Using 1.3.1778+15.04.20151217.1-0ubuntu1 that just landed in stable-phone-overlay PPA the TextFeild doesn't have focus anymore.

It seems like a regression to me. Or was this done on purpose?

Related branches

Revision history for this message
Launchpad Janitor (janitor) wrote :

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

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
Revision history for this message
Michał Sawicz (saviq) wrote :

This is a UX decision to support keyboard navigation, we'll need to work around it. I've filed bug #1528198 to track the need for a Combo box component.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → Won't Fix
Changed in unity8 (Ubuntu):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Albert Astals Cid (aacid)
summary: - Regression in handling on focus on TextField+Popover
+ Popover steals focus on open
Changed in canonical-devices-system-image:
assignee: nobody → Michał Sawicz (saviq)
importance: Undecided → High
milestone: none → ww02-2016
status: New → Confirmed
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: Confirmed → In Progress
Revision history for this message
Olivier Tilloy (osomon) wrote :

Note that this also affects webbrowser-app, where context menus are implemented as popovers, but stealing active focus from the webview is not desirable. I’ve found a (fragile) workaround, see bug #1526884.

Michał Sawicz (saviq)
description: updated
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.11+16.04.20160111.1-0ubuntu1

---------------
unity8 (8.11+16.04.20160111.1-0ubuntu1) xenial; urgency=medium

  [ Albert Astals Cid ]
  * Clear fake item icon name as soon as it goes invisible (LP:
    #1531172)
  * Fix LVWPH test failing on Xenial (Qt 5.5)
  * Fix dragging so that test passes on Xenial
  * Fix focus on the dash page header text field with new SDK (LP:
    #1528178)
  * Make the card creator test pass again
  * Quit the dash communicator thread before destroying it (LP:
    #1508485)
  * Remove duplicated override_dh_auto_clean
  * SDK changed the name they give to the buttons, follow

  [ Andrea Cimitan ]
  * Fix a broken binding

  [ CI Train Bot ]
  * Resync trunk.
  * Update translation template

  [ Daniel d'Andrada ]
  * Have "make tryFoo" work with Qt 5.5
  * Make DragHandle tests run again.

  [ Lukáš Tinkl ]
  * Fix dismissing the launcher when clicking/tapping outside (LP:
    #1531339, #1530940)
  * Fix panel drop shadow and click-to-focus of maximized apps (LP:
    #1531854)
  * Fix wifi access point indicator items signal strength icon on
    desktop

  [ Michael Terry ]
  * Guard against a couple odd timing scenarios for the too-many-failed-
    login-attempts lockout screen.
  * Skip the wizard's password screens if there is already a password
    set by other means. (LP: #1531268)
  * Stop the emergency dialer from accidentally exposing the mtp
    service. (LP: #1525981)

  [ Michael Zanetti ]
  * Make sure the triangle of the quicklist is in bounds (LP: #1531340)

  [ Pawel Stolowski ]
  * Changes 'shareData' and 'contentType' attributes to 'share-data' and
    'content-type', to match existing naming schema.

 -- Michał Sawicz <email address hidden> Mon, 11 Jan 2016 17:38:49 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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