Software Updater

Merge lp://qastaging/~evfool/update-manager/lp1081099 into lp://qastaging/update-manager

lp1081099
Merge into main

Proposed by Robert Roth on 2012-11-20

Status:

Superseded

Proposed branch:

lp://qastaging/~evfool/update-manager/lp1081099

Merge into:

lp://qastaging/update-manager

Diff against target:

11 lines (+1/-0)

1 file modified

data/gtkbuilder/UpdateManager.ui (+1/-0)

To merge this branch:

bzr merge lp://qastaging/~evfool/update-manager/lp1081099

Low

Fix Released

Link a bug report

Reviewer	Date Requested	Status
Matthew Paul Thomas (community)		Needs Fixing on 2012-11-20
Ubuntu Core Development Team	2012-11-20	Pending
Review via email: mp+135167@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2012-11-22.

Description of the change

Fixed buttons alignment by adding 6px border width.

Revision history for this message

Matthew Paul Thomas (mpt) wrote on 2012-11-20:

Thanks Robert! Unfortunately, it's still one pixel off on both left and right. <http://imgur.com/IlyvK> I don't know whether that's the fault of the box containing the buttons, or the box containing the rest.

review: Needs Fixing

lp://qastaging/~evfool/update-manager/lp1081099 updated on 2012-11-20

2559. By Robert Roth on 2012-11-20: Fixed 1px difference for pixel-perfect alignment

Revision history for this message

Robert Roth (evfool) wrote on 2012-11-20:

Thanks mpt for the heads-up... that was the lazy me, just checking the result at first sight. I have fixed it now, the problem was with the button box, now there's 12 px border for all components (as far as I have found, it should be 12px, 13px would be kindof strange), and now the button is pixel-perfectly aligned with the treeview and the icon. I guess that should be fine, but let me know if you spot anything else to fix.