1543379 - Make accesskey work in MozMenuPopup custom elements

Status: RESOLVED WORKSFORME

(Toolkit :: UI Widgets, defect, P3)

Description

[Paul Morris [:pmorris]]

When using the MozMenuPopup class (introduced recently in bug 1531870) the accesskey functionality for menu items is not working. For example, see bug 1531296. I looked into where and how to fix this, but I was not able to figure it out.

Edit: The accesskey is rendered correctly (there are underlines under the accesskey characters in the menu items), but pressing those keys has no effect.

Comment 1

[alexander :surkov (:asurkov)]

I'm not familiar with accesskey handling code, all I know it all lives in c++ (afaik). One suspect I can see in code (bug 1531870) is you change the hierarchy: menuitems are now children of explicit arrowscrollbox (which was used to be an anonymous child of menupoup), and thus menuitems were direct children of a menupopup. My guess would be that accesskey handling code might have a dependency on DOM hierarchy. Cc'ing masayuki and neil for ideas.

Comment 2

[Brian Grinstead [:bgrins]]

It it only not working when you press the key, or is the accesskey also not rendering in the label within the menuitems?

Comment 3

[Paul Morris [:pmorris]]

The accesskey is rendered correctly (there are underlines under the accesskey characters in the menu items), but pressing those keys has no effect.

Comment 4

[Brian Grinstead [:bgrins]]

So if we go with Shadow DOM will the accesskey start working again? This is the approach being taken in Bug 1539651. Before we can use it more generally I think we'll need to get Shadow Parts (bug 1505489), but if that does fix it that's probably our best bet.

Comment 5

[Jorg K (CEST = GMT+2)]

(In reply to Paul Morris [:pmorris] from comment #0)

When using the MozMenuPopup class (introduced recently in bug 1531870) the accesskey functionality for menu items is not working. For example, see bug 1531296. I looked into where and how to fix this, but I was not able to figure it out.

Please don't insert bugs as links. That's not necessary since BMO will linkify "bug 1531870". It's even harmful since the link tooltip doesn't show the bug summary where the link from "bug 1531870" does. Oh, and the links don't have strike-though either.

Comment 6

[Mark Striemer [:mstriemer]]

So I just poked around at a few menupopups on Windows, and they appear to work with accesskeys. Brian, do you know if this was fixed by something else?

Comment 7

[Brian Grinstead [:bgrins]]

(In reply to Mark Striemer [:mstriemer] from comment #6)

So I just poked around at a few menupopups on Windows, and they appear to work with accesskeys. Brian, do you know if this was fixed by something else?

I have no idea, but I'm not seeing specific STR here or in i.e. bug 1531296. I'd be fine to close this as worksforme and then we can reopen if it can still be reproduced somewhere. Magnus, are you aware of this still being an issue in TB?

Comment 8

[Magnus Melin [:mkmelin]]

AFAICT, it's working.