Open Bug 269488 Opened 20 years ago Updated 2 years ago

Refactor: how to move tree-specific code from nsEventListenerManager?

Categories

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

Product:
Core
Component:
DOM: UI Events & Focus Handling
Type:
defect

Tracking

()

People

(Reporter: aaronlev, Unassigned)

Details

The fix for bug 175568 makes sure keyboard-activated context menus show up in an
appropriate place on the screen.

nsEventListenerManager::GetCoordinatesFor has to special-case trees, in order to
get the location of the focused tree item. Otherwise, the context menu would
show below the entire tree. This is because the entire tree is one frame.

nsEventListenerManager shouldn't have to know about trees.

I suggest something like nsINodelessChildCreator -- in the future there could be
other widgets that do something similar. It could have a method called
GetFocusRect() or something.
Assignee: events → nobody
QA Contact: ian → events
Component: Event Handling → User events and focus handling
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.