449948 - Disable snav on xul content.

Status: RESOLVED DUPLICATE of bug 444801

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[Antonio Gomes (tonikitoo)]

currently, spatial navigation works on page content only, but not xul content: e.g. if you go to 'about:config' , snav will just fails oddly.

just disable it in such cases ?

Comment 1

[Antonio Gomes (tonikitoo)]

Attachment: v0.1 - disabile snav if on xuldocument

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 339915 [details] [diff] [review]
v0.1 - disabile snav if on xuldocument

are there other documents the we should also avoid?  svg?

Comment 3

[Antonio Gomes (tonikitoo)]

(In reply to comment #2)
> (From update of attachment 339915 [details] [diff] [review])
> are there other documents the we should also avoid?  svg?

yes, and probably in chrome content (google search bar, find bar) as well ... i will take a look

Comment 4

[Antonio Gomes (tonikitoo)]

(In reply to comment #2)
> (From update of attachment 339915 [details] [diff] [review])
> are there other documents the we should also avoid?  svg?

actually disabling snav for xul docs would break our mochitests.

hum, maybe snav mochitests could be .html not .xul ...

doug?

Comment 5

[Doug Turner (:dougt)]

you could set a pref in mochitests

Comment 6

[Doug Turner (:dougt)]

but ideally, maybe we just want snav to work in xul content.

Comment 7

[Antonio Gomes (tonikitoo)]

it is probably going to be fixed by bug 444801 at some point, marking as dup of it, then.

Comment 8

[Doug Turner (:dougt)]

Comment on attachment 339915 [details] [diff] [review]
v0.1 - disabile snav if on xuldocument

this will break mochitests.  Also this is fixed "better" in bug 444801.