62423 - URL bar: Alt+ArrowDown should open the list

Status: VERIFIED FIXED

(SeaMonkey :: Location Bar, defect, P2)

Description

[Tobias Burnus]

Alt+ArrowDown should open the list of entered URL items.
(The menu which pops up, when you click on the down arrow (blue triangle
pointing down) right of the URL bar.
This is obligatory if you want to use this feature without a mouse
("accessibility") and mimics the behaviour of Windows comboboxes (4x Win).

Comment 1

[Chris McAfee]

what about linux/mac?

Comment 2

[Chris McAfee]

nsbeta1

Comment 3

[Chris McAfee]

nsbeta1+, can minus if there is keyboard workaround

Comment 4

[Chris McAfee]

-

Comment 5

[Blake Ross]

The url bar isn't any type of widget, it's just a textfield + a button which 
opens a menu.  We currently have an editable menulist that we could be using 
instead, but I agree with Matthew that we should get rid of these.  On Windows, 
alt+down-arrow also drops non-editable dropdowns.  So I propose that we special 
case this for the urlbar and fix it in the (non-editable) menulist's binding.

Comment 6

[Blake Ross]

hewitt, are you interested in doing this (particularly since the urlbar is now 
indeed a specialized widget)?

Comment 7

[Joe Hewitt (gone)]

I'll take it.

Comment 8

[hangas]

moving to 0.9.2

Comment 9

[Tony Tovar]

What about pressing F6?  This would duplicate the functionality of IE on 
Windows...

Comment 10

[timeless]

No. Please don't confuse bugs, and don't mass spam a question.  The bug report 
'as is' is a windows parity report (and winie is a windows app)

Comment 11

[Aaron Leventhal]

Attachment: Non-working patch - please advise on how I can toggle the history popup programmatically

Joe, this patch make the Javascript engine complains about E_NOINTERFACE.

I cobbled this together as a start, but it's not close to working - any advice?

Comment 12

[Joe Hewitt (gone)]

The element you're trying to QI to nsIDOMXULPopupElement is an hbox.  The
history popup itself is not in the anon content, it would be the first explicity
menupopup child of the textbox.  You wouldn't need to QI the element anyway, its
just an xbl-defined property you are calling. Also, showPopup is not yet
properly implemented, it is still called openPopup in popup.xml

Comment 13

[Aaron Leventhal]

Attachment: Tested -- works

Comment 14

[Joe Hewitt (gone)]

Comment on attachment 61838 [details] [diff] [review]
Tested -- works

sr=hewitt, but remove the !historyPopup.open check, as "open" isn't implemented
in popup.xml yet, and you don't need that check anyway.

Comment 15

[Aaron Leventhal]

Attachment: Contains Hewitt's change - ready for r=

Comment 16

[Ben Goodger (use ben at mozilla dot org for email)]

Comment on attachment 61931 [details] [diff] [review]
Contains Hewitt's change - ready for r=

r=[e-mail]

Comment 17

[Aaron Leventhal]

-> fix checked in

Comment 18

[Donnie Cranford]

in Solaris CVS build as of 12/17/01 alt+arrowdown puts moz in the background...
Not sure if this is intended on Unix.

Comment 19

[sairuh (rarely reading bugmail)]

vrfy fixed using comm bits [2002.01.07.0x] on rh7.2, winnt and mac 10.1.2
--however, the list opens in the upperleft of the screen, offset from the
browser window. filed bug 118644 for that.