Merge lp://staging/~cruzjbishop/loggerhead/fix-310255 into lp://staging/loggerhead

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 5/2/2012 2:39 PM, Cruz Bishop wrote:
> Cruz Bishop has proposed merging
> lp:~cruzjbishop/loggerhead/fix-310255 into lp:loggerhead.
>
> Requested reviews: Loggerhead Reviewers (loggerhead-reviewers)
> Related bugs: Bug #310255 in loggerhead: "6 digit line numbers get
> cutoff" https://bugs.launchpad.net/loggerhead/+bug/310255
>
> For more details, see:
> https://code.launchpad.net/~cruzjbishop/loggerhead/fix-310255/+merge/104379
>
> Fix for bug #310255
>
> This merge will increase the width of each line number box by 1%,
> preventing formatting anomalies when numbers reach six or more
> digits
>
> Example after this merge: http://i.imgur.com/WnwBl.png

Do you have an example from before this change?

John
=:->

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk+hSw8ACgkQJdeBCYSNAAMTTQCgkEdjelHdpajO60N6mt8R69cG
nV8AoLDOefAZL3hJpPNoBFSzc9AfcDB4
=1nR7
-----END PGP SIGNATURE-----

This seems ok, but it makes me wonder if there is a better way to force it to have a minimum width based on the entries themselves.

That said, this is probably a reasonable stopgap measure.

I've updated the description to match more recent builds and include some pre-merge screenshots

@jameinel Removing the padding to the left of the line numbers worked to a certain degree, but it still came out uglier than before, so I changed it to this

Sorry, accidentally marked this as merged.

Is there any news on this? Richard/lifeless marked it as fix released, even though it wasn't merged, so I'm assuming that either I overlooked something or somebody else got in first.