Closed
Bug 1029646
Opened 11 years ago
Closed 8 years ago
RTL locale support in locale picker
Categories
(Firefox for Android Graveyard :: Locale switching and selection, defect, P4)
Firefox for Android Graveyard
Locale switching and selection
All
Android
Tracking
(firefox53 fixed)
RESOLVED
FIXED
Firefox 53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: rnewman, Assigned: maliu, Mentored)
References
Details
Attachments
(1 file)
This bug covers displaying RTL locales in the locale picker itself. Making our settings fragments themselves RTL-aware is Bug 928663.
This bug *might* be a no-op, depending on how smart Android's list picker is.
Comment 1•10 years ago
|
||
I'll have a go at this as soon as I set-up my development environment.
Reporter | ||
Comment 2•10 years ago
|
||
Aymen: be aware that this is a very ambitious bug, and there is an implicit prerequisite: Fennec's UI doesn't support RTL locales at all right now, and to the best of my knowledge we don't ship any.
You should sync up with :gueroJeff to make sure that RTL/bidi efforts are all working together.
Flags: needinfo?(jbeatty)
Comment 3•10 years ago
|
||
(In reply to Richard Newman [:rnewman] from comment #2)
Well, maybe I ought to first try to fix a less tasking and difficult bug before thinking about this, in order to acquaint myself with the code and development process.
Comment 4•10 years ago
|
||
Hi Aymen,
Yes, I'm organizing a group of interested localizers/Android developers to work on RTL in Fennec. Let me add you to my list of interested people and I'll be in touch this week.
Richard, thanks for CC-ing me. There are about three-four people interested in being mentored through these RTL bugs. I'll have an email sent out by the end of the day with everyone (you, Finkle, and localizers) on it.
Flags: needinfo?(jbeatty)
Updated•8 years ago
|
Priority: -- → P4
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → max
Updated•8 years ago
|
Attachment #8823190 -
Flags: review?(s.kaspari) → review?(ahunt)
Comment 7•8 years ago
|
||
ahunt: You recently reviewed a RTL patch - can you take this one too?
Comment 8•8 years ago
|
||
mozreview-review |
Comment on attachment 8823190 [details]
Bug 1029646 - Update layout direction of configuration and DecorView,
https://reviewboard.mozilla.org/r/101776/#review105210
This looks sane to me!
Sorry for the delay, I forgot about my vacation backlog...
Attachment #8823190 -
Flags: review?(ahunt) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Pushed by ihsiao@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4e763552b6cf
Update layout direction of configuration and DecorView, r=ahunt
Keywords: checkin-needed
Comment 10•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•