Closed Bug 7001 Opened 25 years ago Closed 25 years ago

focus events don't handle embedded webshells

Categories

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

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: buster, Assigned: rods)

References

Details

While implementing ender-backed text controls, I noticed that I don't get focus
change notifications when switching between text controls.  Each ender text
control has it's own webshell, very similar to IFRAME.  Talked to Tom about
this, he says it's a known problem. Submitting this bug as a nag to get a fix in
for M7.
Severity: normal → major
Priority: P3 → P2
Target Milestone: M7
set severity, priority, and milestone.
Blocks: 7403
No longer blocks: 7403
Status: NEW → ASSIGNED
Target Milestone: M7 → M8
Unfortunately I won't get to this tonight with my other fixes.  M8
Tom, can I get an update on any progress on this bug?  Still on target for M8?
Target Milestone: M8 → M9
buster comment:
7001 is not worth holding up M8 over.  make it an early M9 deliverable and I'm
happy.
Blocks: 7595
Priority: P2 → P1
Summary: focus events don't handle embedded webshells → [BLOCKER] focus events don't handle embedded webshells
7595 is blocked on this bug.  OnChange notifications are triggered by focus
and blur events.  If I don't get focus and blur, I can't fire OnChange
notifications.  We can't enable GFX text widgets until we can get this solved.
added [BLOCKER] to summary.
Blocks: 9595
Summary: [BLOCKER] focus events don't handle embedded webshells → focus events don't handle embedded webshells
Target Milestone: M9 → M10
I believe we are past the blocker phase on this.  I know I still have a couple
of issues to fix here for the more subtle forms of focus shifting (tab vs
click) but I won't hit them for M9. Removing blocker status and moving to M10.
Whiteboard: rods looking at this, joki says
Assignee: joki → rods
Status: ASSIGNED → NEW
Hey Rod, the remaining part of this was left open because I knew we didn't deal
correctly with focus from js, focus from shifting, etc. with the new gfx text
fields.  If you've finished working on that and that all works you can mark
this fixed.
Target Milestone: M10 → M12
Changing to M12
Whiteboard: rods looking at this, joki says → I have the fix
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
you can set focus from js - fixed
Thanks buster,
Verified fixed per reporter.
Status: RESOLVED → VERIFIED
URL: any
Whiteboard: I have the fix
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.