[SDK] cursor handle should not be visible if field is empty

Bug #1450213 reported by Bill Filler on 2015-04-29
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical System Image
High
Zoltan Balogh
Ubuntu UX
Medium
Jouni Helminen
ubuntu-ui-toolkit (Ubuntu)
High
Christian Dywan
Vivid
Undecided
Unassigned
ubuntu-ui-toolkit (Ubuntu RTM)
Undecided
Unassigned

Bug Description

On arale
ui-tookit version 1.2.1485+15.04.20150429-0ubuntu1

The big blue cursor handle should not be visible if the text field is empty, as dragging does nothing. It should only be displayed if the field has text (or spaces) in it where dragging would make sense.

Steps to reproduce:

1) reboot phone
2) launch messaging-app
3) swipe up from bottom to create new messaage

Expected results:
- cursor should be blinking in field
- no text handle should be visible

Actual results:
- cursor is blinking in field
- big blue cursor handle is visible

NOTE, you can repeat this same thing by pressing the Search icon in the dash. The blue cursor appears even when nothing in paste buffer.

SOLUTION:

Don't show handler when tapping on an empty textfield. Long press textfield in order to access copy/paste

Related branches

Bill Filler (bfiller) on 2015-04-29
Changed in canonical-devices-system-image:
importance: Undecided → High
assignee: nobody → Zoltan Balogh (bzoltan)
Zoltan Balogh (bzoltan) on 2015-04-30
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Christian Dywan (kalikiana)
importance: Undecided → Critical
importance: Critical → High
Christian Dywan (kalikiana) wrote :

It is no bug that the handler is visible in an empty text field. Tapping the handler opens the context menu and clipboard contents can be pasted.

I'm not sure if this should be special-cased for when the clipboard is in fact empty - this will need design input.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
summary: - cursor handle should not be visible if field is empty
+ {SDK] cursor handle should not be visible if field is empty
Changed in ubuntu-ux:
status: New → Triaged
assignee: nobody → Jouni Helminen (jounihelminen)
importance: Undecided → Medium
Zsombor Egri (zsombi) on 2015-05-14
summary: - {SDK] cursor handle should not be visible if field is empty
+ [SDK] cursor handle should not be visible if field is empty
Jouni Helminen (jounihelminen) wrote :

Discussed this with Christian yesterday.

Solution - don't show handler when tapping on an empty textfield. Long press textfield in order to access copy/paste

description: updated
Changed in ubuntu-ux:
status: Triaged → Fix Committed
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1510+15.10.20150519-0ubuntu1

---------------
ubuntu-ui-toolkit (1.3.1510+15.10.20150519-0ubuntu1) wily; urgency=medium

  [ Loïc Molinari ]
  * Set QSGNode debug description on the shape-based nodes to ease renderer
    debugging. Fixes LP: #1437270
  * Ensured QSGLayer-based source textures (like ShaderEffectSource) are
    updated before rendering. Fixes LP: #1447495

  [ Tim Peeters ]
  * Update sections test program.
  * Document the correct version when Page.head.visible and Page.head.locked
    were added.
  * Revert to use the old header in trunk.
  * Introduce the new ActionBar component.
  * Restore moving property on AppHeader 1.2.

  [ Zsombor Egri ]
  * Provide versioning for theme loading. Each toolkit must be released with a
    well defined system theme version, and theme modules must have the same
    version as the toolkit. Fixes LP: #1447113
  * Fix import versions in entire UITK and examples. Move example code under
    1.3 folder. Fixes LP: #1447113
  * Separate 1.2 from 1.3 versions in all Ubuntu.Component modules.
    Fixes LP: #1447113

  [ Zoltan Balogh ]
  * Fix header CPO for 1.2 without moving attribute. Let the header CPO try
    both old and new buttons.

  [ Christian Dywan ]
  * Add version subfolder to benchmark test paths.
  * Only show handlers if text isn't empty. Fixes LP: #1450213.
  * Only print unique deprecation notes. Fixes LP: #1454281.

  [ CI Train Bot ]
  * New rebuild forced.

 -- CI Train Bot <email address hidden> Tue, 19 May 2015 10:47:09 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Released
Zoltan Balogh (bzoltan) on 2015-05-20
Changed in ubuntu-ui-toolkit (Ubuntu):
milestone: none → ubuntu-15.04
Bill Filler (bfiller) on 2015-05-20
Changed in canonical-devices-system-image:
milestone: none → ww22-2015
Changed in canonical-devices-system-image:
status: New → Fix Released
Changed in ubuntu-ui-toolkit (Ubuntu RTM):
status: New → Fix Released
Changed in ubuntu-ux:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers