466535 - Sunbird/Lightning should get RTL compatible theme

Status: RESOLVED FIXED

Description

[Tomer Cohen :tomer]

Attachment: screenshot of sunbird main screen

I am sorry to be the person to tell that, but sadly Sunbird has no RTL compatible theme at all. This blocks the release for every RTL locale. 

With a quick sanity check I've found the main screen has two issues. Other pages may also have some RTL issues, but this will require us to do a more deeply check (any script to get over every screen in the application?).


Issues on the main screen with default program configuration:
1. The day/hour table is not aligned correctly and is broken.
2. Arrows should be flipped. 

We done recently the fixes for Firefox 3.0 theme, and the linked page talks about how to do that. I will be happy to work on this, but will need some help from others, especially contacts from other RTL locales which are blocked by this issue in order to make sure solutions will also be helpful to them.

Comment 1

[Tomer Cohen :tomer]

https://developer.mozilla.org/en/Making_Sure_Your_Theme_Works_with_RTL_Locales
Making Sure Your [Firefox] Theme Works with RTL Locales


http://mxr.mozilla.org/l10n-central/source/he/toolkit/chrome/global/intl.css
Our current intl.css file. Keep in mind we prefer not to add more junk to this file, and instead solve the issues from its root, which is the application default theme.

Comment 2

[Simon Paquet [:sipaq]]

Tomer, the table issue happens in LTR locales as well and is not RTL specific. It goes away when you move the views (e.g. move to the next week) for the first time.

Comment 3

[Tomer Cohen :tomer]

Requesting blocking Calendar 1.0, as this issue is very important to RTL locales.

Comment 4

[Stefan Sitter [:ssitter]]

> Issues on the main screen with default program configuration:
> 1. The day/hour table is not aligned correctly and is broken.

Known issue, see Bug 344561.

Comment 5

[Philipp Kewisch [:Fallen] [:📆]]

Attachment: Fix - v1

This patch fixes the most prominent issues in the main view. It should greatly improve the situation. The only obvious thing I still saw was the cycler in the todaypane, but since this uses the same styles as thunderbird I'd say its better to fix the issue there.

When this bug is fixed, if needed please file a new one and pinpoint any remaining issues.

Comment 6

[Philipp Kewisch [:Fallen] [:📆]]

Please disregard the pinstripe/winstripe theme differences I accidentally flattened in today-pane.css, I've fixed this locally.

Comment 7

[Stefan Sitter [:ssitter]]

Comment on attachment 361332 [details] [diff] [review]
Fix - v1

>         <modehbox mode="mail,calendar" broadcaster="modeBroadcaster">
>-          <toolbarbutton id="folderview-cycler-left"
>+          <toolbarbutton id="folderview-cycler-prev"
>+                         chromedir="&locale.dir;"
>                          class="folderview-cycler"
>                          oncommand="TodayPane.cyclePaneView(-1);"/>
>-          <toolbarbutton id="folderview-cycler-right"
>+          <toolbarbutton id="folderview-cycler-next"
>+                         chromedir="&locale.dir;"

This change of the IDs makes the buttons invisible. Looks like Thunderbird defines style rules only for folderview-cycler-right and folderview-cycler-left. I suggest to keep the current IDs to actually see the buttons.

Otherwise r=ssitter. Only tested with Thunderbird 3.0b2pre on Windows XP.

Comment 8

[Philipp Kewisch [:Fallen] [:📆]]

Attachment: Fix - v2

This patch also fixes the mail/ code and makes the mail folderpane rtl compliant.

Comment 9

[Phil Ringnalda (:philor)]

Alas, poor Pinstripe. I'm not sure quite why yet, but in RTL that puts a space in between the two buttons, instead of between the buttons and the edge of the folderpane.

Comment 10

[Philipp Kewisch [:Fallen] [:📆]]

Attachment: Fix - v3

Blasted pinstripe! Instead of using a margin, I opted for a padding on the sidebarheader.

Comment 11

[Phil Ringnalda (:philor)]

Comment on attachment 362600 [details] [diff] [review]
Fix - v3

Sorry about the delay, that seems fine to me. (Other than the trailing whitespace after <!DOCTYPE bindings in minimonth.xml, anyway.)

Comment 12

[Philipp Kewisch [:Fallen] [:📆]]

This checkin needs to be postponed until after our 3.0b2 compatible milestone

Comment 13

[Stefan Sitter [:ssitter]]

(In reply to comment #12)

If I understand it correctly most Thunderbird theme extensions must be updated after landing the mail/ changes. Is this correct? In that case I think it should land rather soon, at least for Tb 3.0b3.

Comment 14

[Philipp Kewisch [:Fallen] [:📆]]

When is b3 planned? If we still want to provide a lightning version compatible with b2, this patch will break it. We could do the Thunderbird checkin asap and check in a modified version of the calendar patch, leaving this bug open for the rest of the changes, but is it really worth the effort?

Comment 15

[Phil Ringnalda (:philor)]

Schedule's still firming up, but roughly: freeze end of April, ship start of May.

Comment 16

[Philipp Kewisch [:Fallen] [:📆]]

I'm not going to hold this off any longer, its mythical that we will be ready before b3.

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/f7e27990b718>

-> FIXED

Comment 17

[Reed Loden [:reed] (use needinfo?)]

Not sure why this was put into the security group...

Comment 18

[Philipp Kewisch [:Fallen] [:📆]]

Sorry, that was definitely an accident.

Comment 19

[Philipp Kewisch [:Fallen] [:📆]]

These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".