(photon) (RTL) Back/forward buttons aren't Photon style on 4.2 tablet when RTL is enabled

VERIFIED FIXED in Firefox 57

Status

()

defect
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: jwu, Assigned: jwu)

Tracking

(Blocks 2 bugs)

Trunk
Firefox 58
Unspecified
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox57 verified, firefox58 verified)

Details

(Whiteboard: [FNC][SPT58.2][INT])

Attachments

(2 attachments)

Devices:
Nexus 10 emulator (Android 4.2)

Build: Nightly

Steps to reproduce:
1. On a tablet device, set the browser language to an RTL supported language.

Expected result:
The back/forward buttons aren't Photon style.

Actual result:
The back/forward buttons should be Photon style.
Comment on attachment 8916493 [details]
Bug 1406828 - Update back/forward icons for RTL on tablet.

https://reviewboard.mozilla.org/r/187636/#review192638
Attachment #8916493 - Flags: review?(cnevinchen) → review+
Pushed by topwu.tw@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/71d6058e889b
Update back/forward icons for RTL on tablet. r=nechen
https://hg.mozilla.org/mozilla-central/rev/71d6058e889b
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 58
Comment on attachment 8916493 [details]
Bug 1406828 - Update back/forward icons for RTL on tablet.

Approval Request Comment
[Feature/Bug causing the regression]:
Update back/forward icons for RTL on tablet

[User impact if declined]:
This is a visual enhancement as part of Photon related UI bug.

[Is this code covered by automated tests?]:
No

[Has the fix been verified in Nightly?]:
Not yet

[Needs manual test from QE? If yes, steps to reproduce]: 
No, we just update several back/forward icons. No code logic or UI layout changed.

[List of other uplifts needed for the feature/fix]:
None

[Is the change risky?]:
No

[Why is the change risky/not risky?]:
We just change images for back/forward icons on tablet.

[String changes made/needed]:
No
Attachment #8916493 - Flags: approval-mozilla-beta?
Comment on attachment 8916493 [details]
Bug 1406828 - Update back/forward icons for RTL on tablet.

Fennec photon, Beta57+
Attachment #8916493 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Whiteboard: [FNC][SPT58.2][INT]
Verified as fixed on both latest Nightly and beta (57.0b9).
Devices:
Lenovo Yoga Tablet 2 (Android 4.4.2)
Asus ZenPad 8.0 Z380KL (Android 6.0.1)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.