Closed Bug 1062338 Opened 7 years ago Closed 7 years ago
Regression: Back button icon missing in menu on some small screen (?) hardware (?) devices
32.81 KB, image/png
20.57 KB, image/png
971 bytes, patch
|Details | Diff | Splinter Review|
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]
History screenshot What the old menu used to look like prior to the changes in the above bugs.
Device this is filed against has a hardware keyboard: http://www.samsung.com/latin_en/support/model/GT-B5330ZKLTTT
> 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
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.
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
(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.
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
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Aaron - As the reporter, can you verify that this is fixed in the latest nightly build?
Comment on attachment 8487364 [details] [diff] [review] Remove useless ic_menu_back drawable xml Thanks Aaron!
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.