Closed Bug 1181944 Opened 5 years ago Closed 3 years ago

(Gaia RTL 2.5) Support RTL for System App

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(feature-b2g:2.5+)

RESOLVED WONTFIX
FxOS-S8 (02Oct)
feature-b2g 2.5+

People

(Reporter: nefzaoui, Unassigned, NeedInfo)

References

Details

RTL Support for the System App in Firefox OS 2.5
Blocks: rtl-2.5
feature-b2g: --- → 2.5+
Depends on: 1135196
Depends on: 1058757
Depends on: 1139154
Depends on: 1149450
Depends on: 1164368
Depends on: 1177049
Depends on: 1177051
Depends on: 1149454
Depends on: 1147240
Depends on: 1120549
Depends on: 1199000
Hi Loic, Hi Kaze,
Will RTL for system able to be review soon?
Thanks
Flags: needinfo?(loic)
Flags: needinfo?(fabien)
Target Milestone: --- → FxOS-S8 (02Oct)
(In reply to Josh Cheng [:josh] from comment #1)
> Hi Loic, Hi Kaze,
> Will RTL for system able to be review soon?
> Thanks

We are on it, the team is opening the bugs to push the shared component code asap, and we need that part before pushing any code on System app.

Plus we are on a bug with the rtl + taskswitcher. that should be tackle before pushing any code to you.
Flags: needinfo?(loic)
(In reply to Loïc Cuguen from comment #2)
> (In reply to Josh Cheng [:josh] from comment #1)
> > Hi Loic, Hi Kaze,
> > Will RTL for system able to be review soon?
> > Thanks
> 
> We are on it, the team is opening the bugs to push the shared component code
> asap, and we need that part before pushing any code on System app.
> 
> Plus we are on a bug with the rtl + taskswitcher. that should be tackle
> before pushing any code to you.

I thought we were going to land the CSS refactoring before diving into the RTL navigation stuff? For the task switcher, we should coordinate with :mcav as he has patches in-flight to add new sheet/window buttons and improve performance that will likely conflict.
css + rtl navigation in one PR per app with the associated shared components that is the plan we proposed last week.

easier for qa to test easier for reviewer to get a whole picture aswell.
(In reply to Loïc Cuguen from comment #4)
> css + rtl navigation in one PR per app with the associated shared components
> that is the plan we proposed last week.
> 
> easier for qa to test easier for reviewer to get a whole picture aswell.

Yeah I can see the sense in that, but it will make for a big PR for system at least. Lets see how it plays out - we can split the PR if necessary.
Depends on: 1208170
Depends on: 1208253
No longer depends on: 1208253
Depends on: 1209533
No longer depends on: 1209533
Depends on: 1209533
Depends on: 1213222
Depends on: 1215095
Depends on: 1217746
Depends on: 1218332
Depends on: 1218334
Depends on: 1218720
Depends on: 1212329
Depends on: 1219618
Depends on: 1220561
Depends on: 1220563
Depends on: 1221915
Depends on: 1226612
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.