Read-only text field doesn't look read-only

Bug #1370571 reported by Matthew Paul Thomas
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Cris Dywan
Vivid
New
Undecided
Unassigned
ubuntu-ui-toolkit (Ubuntu RTM)
New
Undecided
Unassigned

Bug Description

Ubuntu 14.10 r244

1. Connect the phone to a Wi-Fi network if it isn't already.
2. Navigate to System Settings > "Wi-Fi" > the chevron next to the current network.
3. Tap in the "IP address" field.

What you see:
2. The field is grey.
3. The field is white.

What you should see: Since the field is read-only, it should look exactly the same when it isn't focused as when it is.

(The use of a read-only text field is a workaround for bug 1286282.)

Related branches

Revision history for this message
Cris Dywan (kalikiana) wrote :

So read only text fields should always look as if they have focus?

That seems very confusing to me.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Incomplete
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

Right now, the read-only field (a) changes appearance when focused, (b) to white, (c) is debossed, and (d) has a blinking insertion point. Combined with the fact that its label isn't greyed out, those four clues strongly indicate that I can type into it. The only thing that prevented me from thinking that the OSK not appearing here was a bug was the fact that that I happened to design this screen myself.

That's why I didn't say a read-only text field should always look focused; that wouldn't solve the problem at all. I said it should look exactly the same when it isn't focused as when it is.

In most toolkits, read-only text fields (a) aren't even focusable, (b) or if they are, they don't change color, (c) they have flatter edges than read-write fields, and (d) they don't have an insertion point (except for accessibility purposes when you can't select text with a pointing device). You can see how your browser renders the difference, for example, by copying and pasting the following line into the address field:
data:text/html,<input><input disabled>

(Unfortunately data: URIs don't work with the Ubuntu Browser; I've just reported that as bug 1377953.)

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

The attached branch does two things:

- The cursor won't blink in read-only text fields.
- The color doesn't change to white on focus.

Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Cris Dywan (kalikiana) wrote :

After some discussion it seems we want read-only text fields to actually look like labels (that can be selected).

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

That would work in most cases. But what about the case where the text scrolls, such as a license agreement or ToS? If a line or paragraph happens to end just above the bottom of the field, there would be no hint that the text can be scrolled. Perhaps the read-only rendering should include a frame if, and only if, the text is longer than the field.

Revision history for this message
Cris Dywan (kalikiana) wrote :

The only hint you have in all cases is text being cut off at half the height of a line. To me the border doesn't make any difference.

Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Committed
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Fix Committed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (4.4 KiB)

This bug was fixed in the package ubuntu-ui-toolkit - 1.1.1311+15.04.20141102-0ubuntu1

---------------
ubuntu-ui-toolkit (1.1.1311+15.04.20141102-0ubuntu1) vivid; urgency=medium

  [ Zoltán Balogh ]
  [ Christian Dywan <email address hidden>
  * Explicitly use Orange in Slider and ProgressBar.
    Fixes: LP: #1372585.
  * Copy components correctly in push_to_phone.sh and make copy.sh
    more robust.
  * Make push_to_phone.sh use a temporary folder and sudo.

  [ Zsombor Egri <email address hidden>
  * ThinDivider margins rolled back to edge-to-edge. Divider colors
    adjusted with design. Fixes: LP: #1380766.

  [ Tim Peeters <email address hidden>
  * Add updated header divider to use rectangles instead of images,
    and to auto-detect its color. Fixes: LP: #1372590.
  * Add internal property to hide the header separator.
    Needed in Unity8.

  [ Leonardo Arias Fonseca <email address hidden>
  * Fixed the infinite loop when running flickable tests.
    Fixes: LP: #1384703.
  * Added an autopilot helper for expandable list items.
    Fixes: LP: #1340227.
  * The autopilot helpers to swipe to show more now get the flickable
    containers by default. Fixes: LP: #1365674.
  * Replaced the tabs for spaces in UbuntuTestCase.
   Fixes: LP: #1324469.

ubuntu-ui-toolkit (1.1.1298+14.10.20141030~rtm-0ubuntu1) 14.09; urgency=low

  [ CI bot ]
  * Resync trunk

  [ Timo Jyrinki ]
  * Modify test .desktop file (LP: #1382414) (LP: #1382414)

ubuntu-ui-toolkit (1.1.1298+14.10.20141016.1-0ubuntu1) 14.09; urgency=medium

  [ Jussi Pakkanen]
  * Remove the thumbnailer image provider here, because thumbnailer
    provides it by itself Fixes LP: #1368492

  [ Lars Uebernickel ]
  * Icon: size correctly even if no or only one dimension is given

  [ Michael Sheldon]
  * Check that orientation changes are acceptable before applying new
    orientation based on active window change. Fixes LP: #1375306

  [ Zoltán Balogh ]
  * Updated test plan executer

  [ Zeller Benjamin ]
  * RenderTimer includes EGL platform files in the rendertime header,
    which can cause problems when included after QDataStream. The
    EGL headers pull the XLib files which define the Symbol Status.
  * Preparing qmlapicheck to support out of source builds for the
    cmake migration

  [ Renato Filho ]
  * Save alarms in invalid time zone. Use invalid time zone to
    floating time alarms.

  [ Tim Peeters ]
  * Make icons from theme in showcase gallery scrollable.
    Fixes: LP: #1336279.
  * Fix width of header contents. Fixes LP: #1378261.

  [ Sebastien Bacher ]
  * Don't use simple quotes around translatable strings, xgetting
    ignore those. Fixes LP: #1378774.

  [ Zsombor Egri ]
  * Alarm update regression caused by EDS changes. Also fixes Tags
    update. Requires qtorganizer5-eds updates as well!.
    Fixes LP: #1364949.
  * Forgotten Interface declaration in objects. Fixes LP: #1378298

  [ Christian Dywan ]
  * Make dialog contents scrollable. Fixes LP: #1376763
  * Make mouseMoveSlowly work with a negative distance.
    Fixes LP: #1343334
  * Read-only text entries don't blink or visibly take focus
    Fixes LP: #1370571
  * Anchor overflow but...

Read more...

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

Other bug subscribers

Remote bug watches

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