136.17 KB, image/png
94.01 KB, image/png
27.51 KB, image/png
11.02 KB, patch
|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.
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.
Oversaw the subject padding is different with threading enabled/disabled. New patch adapted to this. Mac AB padding question still present.
All the headers looks good, except the header Subject.
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.
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.
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.
The classic themes under Win7 make all a little bit complicated, but now it should work. Tested also under XP.
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+
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+
Removed the unneeded line as in previous comment noted.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 10.0
You need to log in before you can comment on or make changes to this bug.