bug 334997 introduced a menubar to suiterunner's toolkit-based JS console, but now accesskeys of the console mode toolbar clash with menu accesskeys. The overlay could fix that easily (attachment following in a minute), but bug 335375 blocks that approach from working (only the braodcasters get their accesskeys changed, but that change doesn't propagate to the toolbar buttons).
Created attachment 219757 [details] [diff] [review] overlay broadcasters with new accesskeys Here's the patch I talked about in comment #0 - it's quite straight forward, and DOMI shows that the broadcasters get their new accesskeys assigned like intended. The toolbar buttons still have the pre-overlay accesskeys though :(
Created attachment 277383 [details] [diff] [review] Remove broadcasters We can resolve the problem with the broadcasters by eliminating them ;-)
Created attachment 277388 [details] [diff] [review] Overlay new access keys Requires attachment 277283 [details], obviously! Note: I used the old names for the entities, because I hadn't read attachment 219757 [details] [diff] [review] first. I also had to replace the "V" access key for "Evaluate".
Comment on attachment 277383 [details] [diff] [review] Remove broadcasters This part looks OK, but you need to also change the toolkit console.css files to not style the toolbarbutton based on the observes attribute :(
Created attachment 277404 [details] [diff] [review] Fix up CSS too We get to use id selectors :-)
Comment on attachment 277404 [details] [diff] [review] Fix up CSS too Hmm, my Mac build really doesn't like it when I change the sort order (hangs), but it works fine on Windows. This patch doesn't impact it, anyways. r=me.
Comment on attachment 277388 [details] [diff] [review] Overlay new access keys This variant works fine, thanks for looking into that. r=me
Id selectors are indeed nicer, but why do have the id selectors have an ':'? It doesn't seem to be needed and requires ugly escaping in CSS.
(In reply to comment #8) >Id selectors are indeed nicer, but why do have the id selectors have an ':'? I retained the existing ids for backwards compatibility.
The use of escape's in id's make them inefficient in parsing, and from a themers point of view there is no 'backwards compatility' in this way. We have to have both selectors anyway (both observer and the id).
I was referring to backwards compatibility of extensions (including SeaMonkey).
Seems like I forgot to mark this fixed last month, sorry.