Closed Bug 306486 Opened 19 years ago Closed 6 years ago

Consolidate *Element::SetFocus()

Categories

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

Product:
Core
Component:
DOM: UI Events & Focus Handling
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: MatsPalmgren_bugz, Unassigned)

Details

Just so it's not lost... in bug 303620 comment 13, Aaron wrote:

I wonder why we really need separate impls for nsHTMLAnchorElement::SetFocus()
and nsHTMLAreaElement::SetFocus() apart from nsGenericHTMLElement::SetFocus(). I
see that they FlushPendingNotifications(Flush_Layout) which seems like it should
be done for everything or not at all. The anchor impl has checks for a link
handler but the generic SetFocus() already does a better IsFocusable().
Assignee: events → nobody
QA Contact: ian → events
This seems can be resolved at Bug 778654.
Not relevant anymore.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.