[sdk] Switch and CheckBox do not visually indicate focus

Bug #1225139 reported by Mike McCracken
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Medium
Zoltan Balogh
Ubuntu UX
Fix Committed
Medium
Jamie Young
ubuntu-ui-toolkit (Ubuntu RTM)
Fix Released
High
Unassigned

Bug Description

[updated description]

The Switch and CheckBox components can have focus:true set (and can be targets of KeyNavigation.(back)tab, but do not indicate this visually.

If I set a Checkbox to be the target of a TextField's KeyNavigation.tab, I can tab to it and change its state with the return key, but there's no way to tell that it's active.

–––

Update:

This is part of a larger bit of work we're currently doing. These issues will be resolved as part of this project.

Progress: https://canonical.leankit.com/Boards/View/107963243/115888542

Related branches

Mike McCracken (mikemc)
affects: ubuntu-ui-toolkit → ubuntu-ux
John Lea (johnlea)
Changed in ubuntu-ux:
assignee: nobody → Katie Taylor (katie-t)
Katie Taylor (katie-t)
Changed in ubuntu-ux:
status: New → Confirmed
importance: Undecided → Medium
John Lea (johnlea)
summary: - Switch and CheckBox do not visually indicate focus
+ [sdk] Switch and CheckBox do not visually indicate focus
Changed in ubuntu-ux:
assignee: Katie Taylor (katie-t) → Jouni Helminen (jounihelminen)
Changed in ubuntu-ux:
status: Confirmed → Triaged
John Lea (johnlea)
Changed in ubuntu-ux:
status: Triaged → New
John Lea (johnlea)
Changed in ubuntu-ux:
status: New → Triaged
Revision history for this message
Zsombor Egri (zsombi) wrote :

There is no focus indications in general on components.

Changed in ubuntu-ui-toolkit:
status: New → Triaged
importance: Undecided → Wishlist
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Undecided → Wishlist
status: New → Triaged
Revision history for this message
Jouni Helminen (jounihelminen) wrote :

all components will need focus states designed for pointer devices. Will create a work stream for this

Changed in ubuntu-ux:
status: Triaged → In Progress
Changed in ubuntu-ux:
assignee: Jouni Helminen (jounihelminen) → nobody
assignee: nobody → Jamie Young (jamiedawsonyoung)
description: updated
description: updated
description: updated
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Christian Dywan (kalikiana)
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Wishlist → High
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Triaged → In Progress
Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit
affects: ubuntu-ui-toolkit (Ubuntu) → ubuntu-ui-toolkit (Ubuntu RTM)
Changed in canonical-devices-system-image:
assignee: nobody → Zoltan Balogh (bzoltan)
milestone: none → ww08-2016
Changed in canonical-devices-system-image:
importance: Undecided → Medium
status: New → Confirmed
status: Confirmed → In Progress
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu RTM):
status: In Progress → Fix Committed
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Fix Committed
assignee: nobody → Christian Dywan (kalikiana)
importance: Undecided → High
Changed in ubuntu-ui-toolkit (Ubuntu RTM):
assignee: Christian Dywan (kalikiana) → nobody
Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit (Ubuntu)
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Changed in ubuntu-ux:
status: In Progress → Fix Committed
Changed in canonical-devices-system-image:
milestone: ww08-2016 → ww02-2016
Revision history for this message
Łukasz Zemczak (sil2100) wrote :
Download full text (4.0 KiB)

This bug was fixed in the package ubuntu-ui-toolkit 1.3.1778+15.04.20151217.1-0ubuntu1 in https://launchpad.net/~ci-train-ppa-service/+archive/ubuntu/stable-phone-overlay

---------------

ubuntu-ui-toolkit (1.3.1778+15.04.20151217.1-0ubuntu1) vivid; urgency=medium

  [ Andrea Bernabei ]
  * Rework Scrollbars and add ScrollView component.

  [ Zsombor Egri ]
  * Refactor UCSwipeAreaPrivate to use QQuickItemPrivate instead of being
    a QObject.

  [ Tim Peeters ]
  * Reduce toolbar height on phone in landscape orientation.
    Fixes LP: #1336793.
  * Introduce the new Toolbar component for the header edit mode.
  * Reduce the PageHeader height by 1 GU on phone in landscape orientation.
    Fixes LP: #1336793
  * Fix AppHeader 1.2 to use the new MathUtils. This fixes a failing autopilot
    CPO test.

  [ Zoltan Balogh ]
  * Do not wipe when provisioning for normal use and improve
    regression detection.
  * Try to link Gestures more explicitely. (kudos to xnox).

  [ Christian Dywan ]
  * Proper keyboard navigation and focus visuals for buttons.
    Fixes LP: #1225139, LP: #1514850
  * Escape should close popover. Fixes LP: #1523828
  * Surround state overloading message with static boolean. Fixes LP: #1523399
  * Don't exit, return 1 from export_modules_dir.sh.

  [ Michael Sheldon ]
  * Set the activeFocus on the input component when a text input area receive
    focus. Fixes LP: #1518352

  [ Benjamin Zeller ]
  * Move PageTreeNode to C++.
  * Custom property flag needs to be set always even when the value itself does not change.

  [ Albert Astals Cid ]
  * Make some more members slots
    This way we can use them in unity8 to make the drag areas more easily
    testable by making the times to recognize the gestures longer.

  [ CI Train Bot ]
  * No-change rebuild.

ubuntu-ui-toolkit (1.3.1761+16.04.20151216.1-0ubuntu1) xenial; urgency=medium

  [ Zoltán Balogh ]
  * Drop dependency on qtc plugin script and wait for the shell with a local
    function.

  [ Christian Dywan ]
  * runtest.sh needs to add subdirs to the libpath. Fixes: LP: #1524710.
  * Explicit click/ touch threshold beyond handler.
  * Enter/Return/Space should trigger() Button. Fixes: LP: #1523794.
  * Only set ppmm in SwipeArea if screen has valid dpi. Fixes: LP: #1525174.

  [ Tim Peeters ]
  * Implement header slot management for convergence:
    - Adapt the number of action slots to the width of the view.
    - When a new action appears, show a quick fade-in.
    - Reverse the order of the actions in the header.
    - Update icon width to 4GU. Fixes: LP: #1428757.
  * To prevent an invalid sectionIndex, reset the value of sectionIndex to -1
    when the model of Sections is changed. Fixes: LP: #1513933.

  [ Loïc Molinari ]
  * UbuntuShape - Cleaned up includes.

  [ Albert Astals Cid ]
  * Simplify logic for emitting draggingChanged/pressedChanged
    No need for a switch that considers the possible cases, just store the
    status before and check after
    Fixes an issue in which the SwipeArea pressedChanged signal was not emitted
    if switching directly from WaitingForTouch to Recognized.
  * Fix memory leak in SwipeArea
    SwipeAreaPrivate is not deleted d...

Read more...

Changed in ubuntu-ui-toolkit (Ubuntu RTM):
status: Fix Committed → 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.

Other bug subscribers

Related blueprints

Remote bug watches

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