Reskin Dictionaries & Language Packs page using AMO2009/ClearLeft styles

VERIFIED FIXED in 5.0.6

Status

VERIFIED FIXED
10 years ago
3 years ago

People

(Reporter: davemgarrett, Assigned: clouserw)

Tracking

unspecified
5.0.6
Dependency tree / graph

Details

(URL)

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Comment 1

10 years ago
Created attachment 381393 [details] [diff] [review]
reskin
Assignee: nobody → clouserw
Attachment #381393 - Flags: review?(jbalogh)
Target Milestone: --- → 5.0.6
Comment on attachment 381393 [details] [diff] [review]
reskin

Looks good if you apply the patch I'm about to attach.
Attachment #381393 - Flags: review?(jbalogh) → review+
Created attachment 381403 [details] [diff] [review]
making the dictionaries fit in better
(Reporter)

Updated

10 years ago
Attachment #381403 - Attachment is patch: true
(Assignee)

Comment 4

10 years ago
Thanks, r27071 r27072
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Reporter)

Comment 5

10 years ago
Can you also please fix the bottom cell padding? It gives an extra blank line per entry that makes the page way longer than needed.
(In reply to comment #5)
> Can you also please fix the bottom cell padding? It gives an extra blank line
> per entry that makes the page way longer than needed.

I filed bug 496392 for this.
(Reporter)

Comment 7

10 years ago
Ah, ok. Was resisting the urge to file yet another minor layout bug. ;)
Status: RESOLVED → VERIFIED
(Reporter)

Updated

10 years ago
Blocks: 496392
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.