48959 - need to define and implement focus behavior.

Status: VERIFIED INVALID

(Core Graveyard :: Embedding: APIs, enhancement, P3)

Description

[Judson Valeski]

An interface needs to provide notification of focus out of the content window
(maybe into the content window as well), programmatic transference of focus from
the embedding application back into content, query what window is currently in
focus, and query what element is currently in focus if any.

Comment 1

[Peter Trudelle]

->saari/M0.9, cc joki

Comment 2

[saari (gone)]

This is also known as moving focus management out of the command dispatcher.

Comment 3

[saari (gone)]

target mozilla 0.8

Comment 4

[Paul Wyskoczka]

Updating QA Contact

Comment 5

[saari (gone)]

nsIWebBrowserFocus
Proposed idl
       void activate();

       void deactivate();

        // Give the firstelementfocus within mozilla
       // (ie. TAB was pressedand focus should enter mozilla)
       void setFocusAtFirstElement();

       // Give the last element focus within mozilla
              // (ie ALT-TAB was pressed andfocus should enter mozilla)
       void setFocusAtLastElement();
       attribute nsIDOMWindowfocusedWindow;
       attribute nsIDOMElementfocusedElement;
nsWebBrowser.cpp has some Activate() impl that could/should be leveraged here.
[01/11/01]
embeddor must maintain focus memory up to our "boundary"/box, *we* maintain
focus memory inside our content area.
activate(): top level window getting focus via mouse click. maybe mouse click
events are enough.
activate/deactivate must be called by the embedding app in order for us to
maintain our internal focus memory. called when the top level window gets focus
*and* we are determined to receive focus, vise versa for deactivate.
hangs off of the webBrowser

Comment 6

[saari (gone)]

Attachment: patch for adding stubbed out support for nsIWebBrowserFocus.idl

Comment 7

[saari (gone)]

r=pavlov for checking in the stub implementation and idl

Comment 8

[Judson Valeski]

r=valeski

Comment 9

[Judson Valeski]

whoops. that's a sr=valeski.

Comment 10

[Asa Dotzler [:asa]]

this is targeted at mozilla 0.8  Is this something you all still want to try to
get in to 0.8?

Comment 11

[saari (gone)]

Attachment: implementations

Comment 12

[Peter Trudelle]

->moz0.9

Comment 13

[Peter Trudelle]

Can we get a new ETA for this work?

Comment 14

[Peter Trudelle]

->moz0.9.1

Comment 15

[Michael Dunn]

Correction: Changing QA contact for the Embed API bugs to David Epstein.

Comment 16

[saari (gone)]

out of date, invalidating

Comment 17

[David Epstein]

verified.