908318 - Dynamically added menuitems ignore acceltext

Status: RESOLVED WONTFIX

(Add-on SDK Graveyard :: General, defect, P2)

Description

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

In Bug 664361 we discovered issue when adding menuitems at runtime, it seems that acceltext won't show up at least on OSX.

Comment 1

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Oops I meant Bug 907016 instead.

Comment 2

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Attachment: Screen Shot 2013-08-22 at 14.26.49 .png

Steps to reproduce:

1. Open scratchpad
2. Switch to browser context
3. Execute code from: https://gist.github.com/Gozala/6313036

As you can see in attached image menuitems are added but they don't have
associated acceltext (keyboard shortcut text).

Note that opening tab with chrome://browser/content/browser.xul
and executing same code but in the context of tab will add menuitems and they will
have acceltext.

Comment 3

[Gabor Krizsanits (INACTIVE)]

(In reply to Irakli Gozilalishvili [:irakli] [:gozala] [@gozala] from comment #2)
> Created attachment 794270 [details]
> Screen Shot 2013-08-22 at 14.26.49 .png
> 
> Steps to reproduce:
> 
> 1. Open scratchpad
> 2. Switch to browser context
> 3. Execute code from: https://gist.github.com/Gozala/6313036
> 

This gives me:

/*
Exception: window is null
@Scratchpad/5:13
*/

Comment 4

[Gabor Krizsanits (INACTIVE)]

So I tried again. This time not the released version of ff but the latest one on mc (debug version). I don't get the annoying window is null anymore, but nothing happens. As in: no new menu items appear to me. I'm on windows, can that be the problem?

Comment 5

[Eddy Bruel [:ejpbruel]]

Clearing the needinfo request on this bug because it seems to be no longer relevant. Feel free to reset it if that's not the case.

Comment 6

[Erik Vold [:erikvold] (please needinfo? me)]

Hey Irakli, see comment 4, do we need to keep this open still?