Regression: Back button icon missing in menu on some small screen (?) hardware (?) devices

VERIFIED FIXED in Firefox 33

Status

()

defect
VERIFIED FIXED
5 years ago
3 years ago

People

(Reporter: aaronmt, Assigned: mcomella)

Tracking

({regression, reproducible})

Trunk
Firefox 35
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox32 wontfix, firefox33 fixed, firefox34 fixed, firefox35 verified, fennec+)

Details

Attachments

(3 attachments)

Reporter

Description

5 years ago
Posted image screenshot.png
Yikes. This has been missing for quite some time apparently on some devices running Android 4.0?

This is reproducible on my Samsung S-B5330 small screen device.

  ro.build.id=JZO54K
  ro.build.display.id=JZO54K.B5330LUBUBMC2
  ro.build.version.incremental=B5330LUBUBMC2
  ro.build.version.sdk=16
  ro.build.version.codename=REL
  ro.build.version.release=4.1.2
  ro.product.model=GT-B5330L

See screenshot.

The functionality is still there, the icon is missing. The hardware back button on the keyboard works. The button still works in the menu.

Last good revision: 862cb6a1cc88 (2013-12-18)
First bad revision: 5c7fa2bfea8b (2013-12-19)
Pushlog:

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=862cb6a1cc88&tochange=5c7fa2bfea8b

My guess is one of these 

90699ba761fb	Sriram Ramasubramanian — Bug 940997: Remove icons from main menu. [r=mfinkle]

63cac76f5f7f	Sriram Ramasubramanian — Bug 940997: Support showAsAction "ifRoom" in custom menu. [r=mfinkle]

0e84f8ba1333	Sriram Ramasubramanian — Bug 940997: Reorganize primary controls in main menu. [r=mfinkle]
Reporter

Comment 1

5 years ago
Posted image historic-menu.png
History screenshot 

What the old menu used to look like prior to the changes in the above bugs.
Reporter

Comment 2

5 years ago
Device this is filed against has a hardware keyboard: http://www.samsung.com/latin_en/support/model/GT-B5330ZKLTTT
Reporter

Comment 3

5 years ago
>   ro.build.version.release=4.1.2

Forgot, I updated the device. Reflecting summary change.
Summary: Regression: Back button icon missing in menu in Android 4.0? (some devices?) → Regression: Back button icon missing in menu on some small screen (?) hardware (?) devices
Reporter

Comment 4

5 years ago
The device is a 320x240 LDPI device. Perhaps there's no provided drawable ic_menu_back.png?
(In reply to Aaron Train [:aaronmt] from comment #4)
> The device is a 320x240 LDPI device. Perhaps there's no provided drawable
> ic_menu_back.png?

We don't provide ldpi drawables for anything. But I'd expect android to just use mdpi (or hdpi even) in this case.
Reporter

Comment 6

5 years ago
Would it help if there a way for me to find the drawables in use from a hierarchy view dump? How would I do that?
(In reply to Aaron Train [:aaronmt] from comment #6)
> Would it help if there a way for me to find the drawables in use from a
> hierarchy view dump? How would I do that?

What about tinting the drawable images with different colours and seeing which one is displayed in the UI? :-)
Assignee: nobody → michael.l.comella
tracking-fennec: ? → +
Does this work, Aaron? https://people.mozilla.org/~mcomella/apks/mcomella-1062338_01.apk
Status: NEW → ASSIGNED
Flags: needinfo?(aaron.train)
Reporter

Comment 9

5 years ago
(In reply to Michael Comella (:mcomella) from comment #8)
> Does this work, Aaron?
> https://people.mozilla.org/~mcomella/apks/mcomella-1062338_01.apk

Yep! I see the back button now in this build.
Flags: needinfo?(aaron.train)
Attachment #8487364 - Flags: review?(lucasr.at.mozilla) → review+
Comment on attachment 8487364 [details] [diff] [review]
Remove useless ic_menu_back drawable xml

Approval Request Comment
[Feature/regressing bug #]: Unknown

[User impact if declined]:
  Users on a small subset of devices (some Android 4.0?) cannot see the back button in the options menu. Note that back is still accessible (and more highly discoverable) via the back button, but it looks really unpolished.

[Describe test coverage new/current, TBPL]:
  None

[Risks and why]: 
  Low risk - we are removing an essentially unused drawable file, which allows the image drawables already in the tree to be used by all devices.

[String/UUID change made/needed]: None
Attachment #8487364 - Flags: approval-mozilla-beta?
Attachment #8487364 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/6ba92c715ffe
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Aaron - As the reporter, can you verify that this is fixed in the latest nightly build?
Flags: needinfo?(aaron.train)
Reporter

Updated

5 years ago
Status: RESOLVED → VERIFIED
Flags: needinfo?(aaron.train)
Comment on attachment 8487364 [details] [diff] [review]
Remove useless ic_menu_back drawable xml

Thanks Aaron!
Attachment #8487364 - Flags: approval-mozilla-beta?
Attachment #8487364 - Flags: approval-mozilla-beta+
Attachment #8487364 - Flags: approval-mozilla-aurora?
Attachment #8487364 - Flags: approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.