Closed
Bug 689543
Opened 13 years ago
Closed 13 years ago
Alignment in message thread pane is off
Categories
(Thunderbird :: Theme, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 10.0
People
(Reporter: andreasn, Assigned: Paenglab)
References
(Blocks 1 open bug)
Details
Attachments
(4 files, 4 obsolete files)
136.17 KB,
image/png
|
Details | |
94.01 KB,
image/png
|
Details | |
27.51 KB,
image/png
|
Details | |
11.02 KB,
patch
|
Paenglab
:
review+
Paenglab
:
ui-review+
|
Details | Diff | Splinter Review |
From the Department of Alignment and Agriculture. The alignment between the text in the tree headers and tree cells are a bit off. This mockup shows the Linux theme, but the issue is present in the Windows and Mac themes as well.
Assignee | ||
Comment 1•13 years ago
|
||
This patch aligns the text and the columnheader like in the mockup. I also aligned the AB headers. Under Mac AB this looks a little bit weird because the names with the icons have a large left margin. I aligned the title with the text. When the title should align with the icon, then a padding of 22px would do this. Under Windows I added a padding of 2px for the flaggedCol (the star) to align it better with the header.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #563955 -
Flags: ui-review?(nisses.mail)
Attachment #563955 -
Flags: review?(nisses.mail)
Assignee | ||
Comment 2•13 years ago
|
||
Oversaw the subject padding is different with threading enabled/disabled. New patch adapted to this. Mac AB padding question still present.
Attachment #563955 -
Attachment is obsolete: true
Attachment #563955 -
Flags: ui-review?(nisses.mail)
Attachment #563955 -
Flags: review?(nisses.mail)
Attachment #564032 -
Flags: ui-review?(nisses.mail)
Attachment #564032 -
Flags: review?(nisses.mail)
Reporter | ||
Comment 3•13 years ago
|
||
All the headers looks good, except the header Subject.
Assignee | ||
Comment 4•13 years ago
|
||
Sorry I haven't thought Mac is different to Linux Win and is using other paddings in main and search window :(. Starting my Mac VM is a pain with a 1.6GHz processor and 768MB for the VM. So I decided to change the padding according Linux and Win without checking.
Attachment #564032 -
Attachment is obsolete: true
Attachment #564032 -
Flags: ui-review?(nisses.mail)
Attachment #564032 -
Flags: review?(nisses.mail)
Attachment #564159 -
Flags: ui-review?(nisses.mail)
Attachment #564159 -
Flags: review?(nisses.mail)
Reporter | ||
Comment 5•13 years ago
|
||
There is also a bit of misalignment when using the HC theme (here at 150% zoom, zoom worked well with Aero and Aero Basic btw). Misalignment marked in red. Not sure it's really caused by this patch, since the text used to be very off, but it seems to affect the starring column as well. This issue is also present in the classic theme.
Reporter | ||
Comment 6•13 years ago
|
||
Comment on attachment 564159 [details] [diff] [review] Align the text v3 The mac alignment now works properly, but marking to review minus due to the Windows HC/Classic theme issue. Both LTR and increase text size to 125% works as it should btw.
Attachment #564159 -
Flags: ui-review?(nisses.mail)
Attachment #564159 -
Flags: ui-review-
Attachment #564159 -
Flags: review?(nisses.mail)
Attachment #564159 -
Flags: review-
Assignee | ||
Comment 7•13 years ago
|
||
The classic themes under Win7 make all a little bit complicated, but now it should work. Tested also under XP.
Attachment #564159 -
Attachment is obsolete: true
Attachment #564214 -
Flags: ui-review?(nisses.mail)
Attachment #564214 -
Flags: review?(nisses.mail)
Reporter | ||
Comment 8•13 years ago
|
||
Comment on attachment 564214 [details] [diff] [review] Align the text v4 Works excellent in both HC and Classic now!
Attachment #564214 -
Flags: ui-review?(nisses.mail) → ui-review+
Reporter | ||
Comment 9•13 years ago
|
||
Comment on attachment 564214 [details] [diff] [review] Align the text v4 Thanks to Neil on irc I now know that [primary="true"] is for the thread lines. From what I can see, an extra blank line got added in gnomestripe/mail/mailWindow1.css on line 283/284, review+ with that fixed.
Attachment #564214 -
Flags: review?(nisses.mail) → review+
Assignee | ||
Comment 10•13 years ago
|
||
Removed the unneeded line as in previous comment noted.
Attachment #564214 -
Attachment is obsolete: true
Attachment #564578 -
Flags: ui-review+
Attachment #564578 -
Flags: review+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
OS: Linux → All
Comment 11•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/ee2d1f664a9d
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
You need to log in
before you can comment on or make changes to this bug.
Description
•