718722 - Icons passed to NativeWindow.menu.add() are not displayed

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Wladimir Palant]

Attachment: Restartless test extension

I'm not sure whether it's a general limitation but at least when I pass the Adblock Plus icon as a data: URL to NativeWindow.menu.add() I get the menu displayed correctly but the icon doesn't show up. I attached a restartless test extension that will run the following line and then remove itself:

> wnd.NativeWindow.menu.add("Test menu", "data:image/png;base64,iVBORw...");

When I then bring up the menu on Samsung Galaxy S II and go into Options I see a "Test menu" line but no icon. Tested on a 12.0a1 nightly build (20120116).

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Some parts of the menu do not show icons. Only the main menu area will show icons on Android >3.0 phones. The "More" list of items will not show a menu. Menus on honeycomb (v3) and ICS (v4) will not show icons in the menu at all. In that case, icons are only shown if the menu action is shown on the "action bar".

Comment 2

[Wladimir Palant]

Ok, looks like I have Android 2.3.4. If I understand correctly, the action bar can only be tested on tablets?

Comment 3

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to Wladimir Palant from comment #2)
> Ok, looks like I have Android 2.3.4. If I understand correctly, the action
> bar can only be tested on tablets?

Mainly, but it does exist on ICS (android 4) phones too. Just not enough room to show many menu actions on the action bar there.

Comment 4

[Wladimir Palant]

Feel free to resolve as WONTFIX. Too bad that I cannot test this feature.