toolkit dialogs have wonky padding

Bug #1387806 reported by Matthew Paul Thomas
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Canonical Devices Products
ubuntu-system-settings (Ubuntu)
Fix Released
Low
Unassigned
ubuntu-system-settings (Ubuntu RTM)
Won't Fix
Undecided
Unassigned
ubuntu-ui-toolkit (Ubuntu)
Fix Released
High
Cris Dywan
ubuntu-ui-toolkit (Ubuntu RTM)
Fix Released
High
Cris Dywan

Bug Description

Ubuntu 14.10 r135

1. Go to System Settings > "Wi-Fi".
2. If Wi-Fi is off, turn it on.
3. Scroll to the bottom of the list and choose "Connect to hidden network".

What you see: A dialog where the left padding (x1) is less than the right padding (x2), and the top padding (y1) is less than the bottom padding (y2).

What you should see: A dialog where the left and right padding are equal (x1 = x2), and the top and bottom padding are equal (y1 = y2).

I don't know whether this is a System Settings bug, a toolkit bug, or a combination.

Tags: ota-1

Related branches

Revision history for this message
Matthew Paul Thomas (mpt) wrote :
description: updated
Revision history for this message
Sebastien Bacher (seb128) wrote :

The x1/x2 difference is a toolkit issue for sure (the toolkit demo gallery has the same margins difference), it looks like the y1/y2 is as well but there might be some settings tweak as well then so keeping a component in low priority, to check again how are things looking once the uitk is fixed

Changed in ubuntu-system-settings (Ubuntu):
importance: Undecided → Low
summary: - "Connect to Hidden Network" dialog has wonky padding
+ toolkit dialogs have wonky padding
Changed in ubuntu-system-settings (Ubuntu):
status: New → Confirmed
Revision history for this message
Sebastien Bacher (seb128) wrote :

note that the title has the same issue, it's not correctly horizontally centered

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
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit:
status: New → Confirmed
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit:
assignee: nobody → Christian Dywan (kalikiana)
status: Confirmed → In Progress
Revision history for this message
Tim Peeters (tpeeters) wrote :

The contents of the dialog is horizontally centered (or should be), so that is a bug.

Vertically it goes directly under the title, so not vertically centered. To change that we'd need input from design.

Revision history for this message
Sebastien Bacher (seb128) wrote :

@Tim, right, the design document [1] has details about the position of the dialog but not about the spacing between its elements, I guess that's something for design to detail

[1] https://docs.google.com/a/canonical.com/document/edit?hgd=1&id=1nFm8xiYhKXXuEO_IvMXoD0lASbYzYXva1BWMVanU3iw#heading=h.7zj70oabnqs6

Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit (Ubuntu)
tags: added: ota-1
Changed in ubuntu-ui-toolkit:
importance: Undecided → High
Zoltan Balogh (bzoltan)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Christian Dywan (kalikiana)
importance: Undecided → High
status: New → In Progress
Zoltan Balogh (bzoltan)
no longer affects: ubuntu-ui-toolkit
Revision history for this message
Oliver Grawert (ogra) wrote :

can we please have RTM tasks for both ? else we will miss when it lands in RTM

Olli Ries (ories)
Changed in canonical-devices-system-image:
assignee: nobody → Canonical Devices Products (canonical-devices-products-team)
importance: Undecided → High
milestone: none → r1
status: New → Confirmed
Revision history for this message
Cris Dywan (kalikiana) wrote :

I don't know what value these RTM tasks have… but there's a branch against RTM if anyone wants it.

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

Quick reminder, the code is there.

Changed in ubuntu-ui-toolkit (Ubuntu RTM):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Christian Dywan (kalikiana)
Revision history for this message
Selene ToyKeeper (toykeeper) wrote :

FWIW, still fails on krillin rtm 176.

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

This bug was fixed in the package ubuntu-ui-toolkit - 1.1.1298+15.04.20141203.1~rtm-0ubuntu1

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

  [ Zoltán Balogh ]
  * Many TextField/TextArea fixes improving user experience greatly and
    making it more in line with the design: handle much easier to drag
    by making its touch area 4 grid units wide and centered around the
    cursor; double tapping anywhere (even around the handle) selects the
    word and opens the contextual menu; long pressing anywhere (even
    around the handle) selects the word and opens the contextual menu;
    make sure that when there is a selection the contextual menu is
    shown and shown above the cursor at the beginning of the selection;
    do not prevent other UI elements from receiving mouse/touch events
    when the contextual menu is shown
  * Revert to 1124
  * Fixed unit test
    tst_components::tst_textinput_touch::test_longtap_when_empty: do not
    cause a double tap before the long press.

  [ Christian Dywan ]
  * Don't subtract itemSpacing from column width (LP: #1387806)
 -- Ubuntu daily release <email address hidden> Wed, 03 Dec 2014 13:12:49 +0000

Changed in ubuntu-ui-toolkit (Ubuntu RTM):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: Confirmed → Fix Released
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

the issue seems to be fixed in the current version

Changed in ubuntu-system-settings (Ubuntu RTM):
status: New → Won't Fix
Changed in ubuntu-system-settings (Ubuntu):
status: Confirmed → 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.