#1412 closed defect (fixed)
Size column in WUI directory listings should be right-justified
Reported by: | davidsarah | Owned by: | zooko |
---|---|---|---|
Priority: | minor | Milestone: | 1.9.0 |
Component: | code-frontend-web | Version: | 1.8.2 |
Keywords: | easy reviewed | Cc: | |
Launchpad Bug: |
Description
<stercor> A nit: in the Tahoe-LAFS Directory screen, it would be nice to have the Size column right-justified so that the size of the data is easier to see. <stercor> Well, it can be seen OK, but right-alignment would be better.
Attachments (3)
Change History (12)
comment:1 Changed at 2011-05-26T22:50:49Z by zooko
- Keywords easy added
comment:2 Changed at 2011-06-05T22:31:39Z by stercor
- Owner set to stercor
- Status changed from new to assigned
Changed at 2011-06-05T22:52:25Z by stercor
Changed at 2011-06-06T15:53:18Z by stercor
Code to right-justify in Size field of "TAHOE-LAFS SI=" WUI screen.
comment:3 Changed at 2011-06-06T15:58:13Z by stercor
- Resolution set to fixed
- Status changed from assigned to closed
comment:4 Changed at 2011-06-06T16:10:21Z by stercor
- Keywords needs-review added; wui usability easy removed
- Resolution fixed deleted
- Status changed from closed to reopened
comment:5 follow-up: ↓ 6 Changed at 2011-06-06T17:18:17Z by warner
- Keywords needs-patch added; needs-review removed
Oh, one tiny tiny nitpick: could you change the patch to retain the amount of leading whitespace? It looks like it moved from 6 spaces to 10 or so (maybe a tab showed up in your editor?). That'd just make the final code line up nicer. Thanks!
comment:6 in reply to: ↑ 5 Changed at 2011-06-07T00:33:41Z by stercor
Replying to warner:
Oh, one tiny tiny nitpick: could you change the patch to retain the amount of leading whitespace? It looks like it moved from 6 spaces to 10 or so (maybe a tab showed up in your editor?). That'd just make the final code line up nicer. Thanks!
The table width is fixed at 100% of the page. HTML attempts to lay out the table as it sees fit. I tried removing the "width: 100%" and everything lined up pleasingly. However, the variable-width table did not look good on the page.
I'd like to hear an easy solution to this. Setting column widths would not be a good solution, IMHO. If that's the only way to do it, then...
Ted
Changed at 2011-06-07T01:06:05Z by terrell
same as stercor's, but without the spurious tab character
comment:7 Changed at 2011-06-07T01:12:07Z by terrell
- Keywords easy reviewed added; needs-patch removed
- Milestone changed from undecided to 1.9.0
- Owner changed from stercor to zooko
- Status changed from reopened to new
also, i've reviewed the change, and it applies cleanly, and does what the label says.
comment:8 Changed at 2011-06-11T15:50:28Z by zooko@…
- Resolution set to fixed
- Status changed from new to closed
In 06bf0360d63141e6:
(The changeset message doesn't reference this ticket)
comment:9 Changed at 2011-06-11T16:49:22Z by zooko@…
In be1ebde6e5db1fae:
Patch file.