Closed Bug 1322144 Opened 7 years ago Closed 7 years ago

[RTL] Lock & magnifier glass icons shown incorrectly under some circumstances

Categories

(Firefox for Android Graveyard :: Theme and Visual Design, defect, P5)

Unspecified
Android
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: itiel_yn8, Unassigned)

References

Details

(Keywords: rtl)

Attachments

(4 files)

Not sure whether this bug is a regression from bug 1319302 or not.

Environment:
Samsung Galaxy S5 G900F Android 6.0.1
Firefox Nightly 53.0a1 2016-12-04 Hebrew build

STR:
1. Open a new tab
2. a. Search for "hhhhhhhhhh" in the URL bar (it can also be other strings, but for some reason not all terms can make this bug reproducible)
   b. In the small amount of time Firefox has to search for this term, observe the green lock icon
3. a. Click the back button to go to the (old) new tab
   b. Observe the magnifying glass icon in the URL bar
4. Close Nightly completely, and re-open it
5. Search for "h" in the URL bar
6. a. Click the back button to go to the (old) new tab
   b. Observe the magnifying glass icon in the URL bar

Actual result:
(For 2b) The green lock icon gets stretched while searching for the term.
(For 3b) The magnifier glass icon gets 'thin'.

Expected result:
(For 2b) The green lock icon shouldn't get stretched while searching for the term.
(For 3b) The magnifier glass icon shouldn't get thin. In step 6b it looks okay.

The same behaviour can't be reproduced on Firefox 50 or Aurora.

Screenshots are attached for 2b, 3b, 6b and for the normal magnifying glass.
Attached image 2b
Attached image 3b
Attached image 6b
Attached image Normal magnifying glass
These are tricky issues, since the bar isn't controlled directly by the browser locale but Android locale.
I am not able to reproduce this on Pixel with Android 7.1.1 even if I use the exact term from scenario. 
Is this still reproducing on your device? (It might be device specific that is why I am asking)
Flags: needinfo?(itiel_yn8)
(In reply to Ioana Chiorean from comment #6)
> I am not able to reproduce this on Pixel with Android 7.1.1 even if I use
> the exact term from scenario. 
> Is this still reproducing on your device? (It might be device specific that
> is why I am asking)

Yes it is.
Using Samsung Galaxy S5 G500F Android 6.0.1, system locale is Hebrew.
Flags: needinfo?(itiel_yn8)
See Also: → 1331995
Hi ItielMan, 
According to your observation, Bug 1322144 is a side-effect of Bug1331995 which is not a RTL specific  defect.
Is my understanding correct ? Thank you !
 

Hi Sorina and Ioana,
Per comment6 and comment7, Could you help us further confirm again if it's reproducible ?and this bug is not a RTL bug ?

Thank you !
Flags: needinfo?(sorina.florean)
Flags: needinfo?(itiel_yn8)
Flags: needinfo?(ioana.chiorean)
Flags: needinfo?(chiorean.ioana)
(In reply to Rachelle Yang [:ryang][ryang@mozilla.com] from comment #8)
> Hi ItielMan, 
> According to your observation, Bug 1322144 is a side-effect of Bug1331995
> which is not a RTL specific  defect.
> Is my understanding correct ? Thank you !

I'd say these are RTL issues as I cannot reproduce them on LTR builds of Nightly. Just tried.
And yes, I think both of them are kinda related to one another.
Flags: needinfo?(itiel_yn8)
Hello,

With LG G4 (Android 6.0.1) the issue is reproducible on LTR - Nightly and RTL - AR build (2017-02-06).
With Huawei MediaPad M2 (Android 5.1.1) I wasn't able to reproduce the issue on latest Nightly and RTL build (Hebrew build).
Flags: needinfo?(sorina.florean)
Thanks Sorina!
Flags: needinfo?(ioana.chiorean)
Flags: needinfo?(chiorean.ioana)
Hi Ioana and Sorina,  

After monitoring the issue for some time, we suspect this is a device specific issue.
Have you ever encountered this issue again with the latest build of 53? 
if not, May I suggest to close the bug now?   If it happened again, feel free to file a bug again, Thank you very much !
Flags: needinfo?(sorina.florean)
Flags: needinfo?(ioana.chiorean)
Flags: needinfo?(chiorean.ioana)
On my device (Samsung Galaxy S5 G900F Android 6.0.1) using Firefox 53b4 the bug is still reproducible.
Rachelle - it is indeed only reproducing on one device from all tested. I see ItielMaN is reproducing on a second device. 
I am not sure how many usage we get from those (maybe we can inspect the Google Play store?)
Flags: needinfo?(sorina.florean)
Flags: needinfo?(ioana.chiorean)
Flags: needinfo?(chiorean.ioana)
This was fixed some time ago.
Closing as RESOLVED FIXED.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: