6 digit line numbers get cutoff
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | loggerhead |
Low
|
Cruz Bishop | ||
Bug Description
Loggerhead cuts off the last digit from a 6 digit line number when doing diffs. This is demonstrated here: http://
Related branches
- John A Meinel: Approve on 2012-05-02
-
Diff: 21 lines (+2/-2)1 file modifiedloggerhead/static/css/diff.css (+2/-2)
| Vadim Peretokin (vperetokin) wrote : | #1 |
| Changed in loggerhead: | |
| importance: | Undecided → Medium |
| status: | New → Triaged |
| Michael Hudson-Doyle (mwhudson) wrote : | #2 |
Should it not offer a scrollbar? The code goes to overflow oddly too.
| Max Kanat-Alexander (mkanat) wrote : | #4 |
I would say that this qualifies as fairly Low priority based on the fact that very few files have line numbers that high.
| Changed in loggerhead: | |
| importance: | Medium → Low |
| John A Meinel (jameinel) wrote : | #5 |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 01/20/2011 02:50 AM, Max Kanat-Alexander wrote:
> I would say that this qualifies as fairly Low priority based on the fact
> that very few files have line numbers that high.
>
> ** Changed in: loggerhead
> Importance: Medium => Low
>
I think the main issue was because of version numbers of stuff like
Emacs, where you have >100k revisions.
So more an issue on the "changes" page, not individual file annotation.
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAk0
WNQAn2i7hBrzlU6
=V68x
-----END PGP SIGNATURE-----
| Max Kanat-Alexander (mkanat) wrote : | #6 |
> So more an issue on the "changes" page, not individual file annotation.
Actually, the bug report is against the "diff" page. There is an example URL provided.
| Changed in loggerhead: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Cruz Bishop (cruzjbishop) |
| Cruz Bishop (cruzjbishop) wrote : | #7 |
Opened a merge request to fix this. :)
| Changed in loggerhead: | |
| status: | In Progress → Fix Committed |
| Changed in loggerhead: | |
| status: | Fix Committed → Fix Released |

Well, actually the linenumber column is using a relative width, so if you make your window wider you'll see more digits :)
It's fairly lame I guess, but I can't think of how to do better in CSS's model :/