[ListItems] Recent update changed the ListItem.Base margins

Bug #1198116 reported by Sebastien Bacher
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UI Toolkit
Fix Released
Undecided
Florian Boucault
ubuntu-ui-toolkit (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Since the recent updates the spacing in base elements seems to have changed.

I'm packing buttons in lists using:

" ListItem.Base {
            Button {
                anchors {
                    right: parent.right
                    left: parent.left
                    verticalCenter: parent.verticalCenter
                }
                text: i18n.tr("Button in a ListItem.Base")
            }
        }"

Since the update the buttons look a bit bigger, I'm adding screenshots before/after (the color changed as well but I guess that's another issue)

Related branches

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

Look at the line "Button in a ListItem.Base" to see the spacing difference described there

Revision history for this message
Florian Boucault (fboucault) wrote :

The default height of Button has changed as per design.

Changed in ubuntu-ui-toolkit:
assignee: nobody → Florian Boucault (fboucault)
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:ubuntu-ui-toolkit at revision 589, scheduled for release in ubuntu-ui-toolkit, milestone Unknown

Changed in ubuntu-ui-toolkit:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 0.1.46+13.10.20130705.3-0ubuntu1

---------------
ubuntu-ui-toolkit (0.1.46+13.10.20130705.3-0ubuntu1) saucy; urgency=low

  [ Kaleo ]
  * Latest fix in ButtonStyle had 3 typos. Fixing them. .
  * ListItem.SingleControl: Ensure that there is always enough vertical
    padding around the control. (LP: #1198116)

  [ Christian Dywan ]
  * Include alias types in qmlapicheck. (LP: #1192540)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 589
 -- Ubuntu daily release <email address hidden> Fri, 05 Jul 2013 18:09:33 +0000

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

Remote bug watches

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