Closed Bug 1102496 Opened 7 years ago Closed 7 years ago

Keypad delete button is pointed the wrong way and padded incorrectly

Categories

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

defect
Not set
normal

Tracking

(b2g-v2.2 verified)

VERIFIED FIXED
2.2 S1 (5dec)
Tracking Status
b2g-v2.2 --- verified

People

(Reporter: drs, Assigned: drs)

References

Details

(Whiteboard: [planned-sprint c=1])

Attachments

(4 files)

Attached image Screenshot
In RTL mode, the keypad delete button is too far to the left, and is pointing the wrong way.
Ahmed, here is my proposal for how this will look after being fixed. Does this look correct to you, other than minor imperfections in pixels of padding?
Flags: needinfo?(nefzaoui.ahmed)
Well, I think the best way to do this is what's proposed in Bug 1011731, Forcing the whole Dialer keypad view to be rendered LTR even with RTL languages. That is because the direction people write numbers with in RTL languages is the same of non-RTLs (i.e English, French...), all from left towards right.
Competitors does that too :) Which actually makes sense.. :)
Flags: needinfo?(nefzaoui.ahmed)
Target Milestone: --- → 2.2 S1 (5dec)
This might end up not being any work based on comment 2, but I need to talk with Stephany about this.
Assignee: nobody → drs.bugzilla
Whiteboard: [planned-sprint c=?] → [planned-sprint c=1]
See Also: → 1011731
PR: https://github.com/mozilla-b2g/gaia/pull/26565

Needinfo on myself for rebase and demo.
Flags: needinfo?(drs.bugzilla)
Attachment #8530465 - Flags: review?(gsvelto)
Bug 1011731 partially fixed this, but not entirely.
Status: NEW → ASSIGNED
Depends on: 1011731
Comment on attachment 8530465 [details] [diff] [review]
Force render the keypad delete button in LTR, even in RTL mode.

Good to see that this one ended up being a one-liner :)
Attachment #8530465 - Flags: review?(gsvelto) → review+
Master: https://github.com/mozilla-b2g/gaia/commit/65e97a2624638c726cce27f0b7af81952a5f312b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: needinfo?(drs.bugzilla)
Resolution: --- → FIXED
Keywords: verifyme
Accordding to  Comment 2 
Forcing the whole Dialer keypad view to be rendered LTR even with RTL languages
This issue verified successfully on Flame 2.2
Gaia-Rev        f5b3d1b6cfa3e702033f613915ae637cb735cbfb
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/8067c111ddff
Build-ID        20150118002501
Version         37.0a2
Device-Name     flame
FW-Release      4.4.2
Refer to video
Status: RESOLVED → VERIFIED
Keywords: verifyme
Attached video VIDEO0255.mp4
Flags: in-moztrap+
You need to log in before you can comment on or make changes to this bug.