Add accesskeys to the xp filepicker. Setting OS to Linux, but this affects anywhere that doesn't have a native filepicker. (Is it possible to force use of the xp picker?)

Could someone using the xp filepicker check this patch for me? (i've only got Windows here).

Tested the patch using solaris nightly 2003012622 on solaris 8/sparc. I created a new user profile for the test. Upon selecting File->Open File I get a filepicker dialog. The labels "_L_ook in:", "File _n_ame:", and "Files of _t_ype:" contain underlined access keys corresponding to what is listed in filepicker.dtd. The checkbox labelled "Show hidden files and directories" contains no such underline. Pressing alt-S toggles the "show hidden..." checkbox. But none of the other access keys seem to work. Mozilla appears to ignore L, N, or T by themselves or in any combination with ctrl/alt/shift. I also tried moving the focus to different controls and then trying the access keys, but no joy.

The checkbox not being underlined is expected behaviour (bug 68841). The rest is not expected. Aaron, i couldn't find any other bugs that would cause this, any thoughts? :)

I checked out the patch. The problem is that the <label> elements are missing a control="targetid" attribute. For example: <label value="&textInput.label;" id="textInputLabel" control="textInput" accesskey="&textInput.accesskey;"/> <textbox id="textInput" flex="1" oninput="doEnabling()" onfocus="onTextFieldFocus();"/> You need the control attribute on the label, so that it knows what it's pointing to.

Actually add control attributes this time. Thanks Aaron :)

I don't see the xp picker on Windows. Can you post a screenshot w/ the patch?

Dean, i don't have access to unix at the moment, but you can just open filepicker.xul in the browser for the general idea. Is that enough, or do you still need a screenshot?

Managed to get my hands on a solaris machine for a screenshot...

Comment on attachment 113641 [details] [diff] [review] Add accesskeys works for me. r=me

Comment on attachment 113641 [details] [diff] [review] Add accesskeys sr=dmose. sorry for taking so long to get to this bug.

Checking in mozilla/xpfe/components/filepicker/res/locale/en-US/filepicker.dtd; /cvsroot/mozilla/xpfe/components/filepicker/res/locale/en-US/filepicker.dtd,v <-- filepicker.dtd new revision: 1.7; previous revision: 1.6 done Checking in mozilla/xpfe/components/filepicker/res/content/filepicker.xul; /cvsroot/mozilla/xpfe/components/filepicker/res/content/filepicker.xul,v <-- filepicker.xul new revision: 1.49; previous revision: 1.48 done I checked in for piersc@cogs.susx.ac.uk, fixed.

spiffy! vrfy'd fixed, 2003.02.27.09 comm trunk, linux rh8.0.