Closed
Bug 24645
Opened 25 years ago
Closed 24 years ago
[XBL] Convert DOM window keybindings to XBL
Categories
(Core :: XBL, defect, P3)
Tracking
()
Future
People
(Reporter: hyatt, Assigned: dr)
References
(Blocks 1 open bug)
Details
(Keywords: arch, perf, Whiteboard: [nsbeta2-],[nsbeta3-])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Convert the key binding files (browsers, textareas, enders, and inputs) over to
XBL. This needs to happen sooner rather than later. We have broken basic
functionality (like keyboard scrolling) in the embedded case.
Reporter | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M14
Comment 1•25 years ago
|
||
putting on beta1 radar
Reporter | ||
Comment 2•25 years ago
|
||
I have converted inputs and textareas over. Next come the browser and editor
DOM windows.
Whiteboard: 50% complete
Comment 3•25 years ago
|
||
Cc'ing myself so that I can add the Unix-specific bindings (if you don't do it
for me) as soon as the framework is in place.
The XBL bindings in the text fields/areas are SO much faster than XUL -- it'll
be great to have them in the editor as well!
Comment 5•25 years ago
|
||
Does this include the mail/news window? Mail compose?
Putting on PDT- radar for beta1. Concerned about regression risk at this time.
Whiteboard: 50% complete → [PDT-]50% complete
Reporter | ||
Comment 8•25 years ago
|
||
No, this HAS to be PDT+. Other PDT+ bugs depend on this.
Comment 9•25 years ago
|
||
wiped out pdt- so they will reconsider, though you should probably list the pdt+
bugs that depend on this, since they will ask :-)
Whiteboard: [PDT-]50% complete → 50% complete
Comment 10•25 years ago
|
||
Please put an estimated completion date on this.
Whiteboard: 50% complete → [PDT+] 50% complete
Reporter | ||
Updated•25 years ago
|
Whiteboard: [PDT+] 50% complete → [PDT+] 2/16 Estimated Fix Date
Reporter | ||
Comment 11•25 years ago
|
||
Well, if we don't care about the embedding case, I guess we could go to beta
without finishing this conversion.
Your call. If it is acceptable for viewer to remain in the busted state it's in
now regarding window key bindings, then I'm comfortable shoving this to M15.
Reporter | ||
Updated•25 years ago
|
Summary: [XBL] Convert Ender keybindings to XBL → [XBL] Convert DOM window keybindings to XBL
Reporter | ||
Comment 12•25 years ago
|
||
Also, composer performance is adversely impacted by leaving the bindings in XUL.
If you notice, typing in text fields is much faster than typing in the composer
window. If an XBL conversion gets done, then the composer window will be much
faster.
There would undoubtedly be some regressions that would occur as fallout from
this conversion however.
Reporter | ||
Comment 13•25 years ago
|
||
Trying to be pragmatic here. I think we can get by without this being in for
beta. If I start getting bugs that become PDT+ that would depend on this, we
can reconsider it, but I think for now we're ok.
Whiteboard: [PDT+] 2/16 Estimated Fix Date
Target Milestone: M14 → M15
Reporter | ||
Comment 15•25 years ago
|
||
*** Bug 27739 has been marked as a duplicate of this bug. ***
Reporter | ||
Updated•25 years ago
|
Target Milestone: M15 → M16
Comment 16•25 years ago
|
||
Mass-moving all M16 non-feature bugs to M17, which we still consider to be
part of beta2
Target Milestone: M16 → M17
Comment 17•25 years ago
|
||
spam, open xptoolkit qa contact moving over to jrgm
QA Contact: paulmac → jrgm
Comment 18•25 years ago
|
||
nominating for nsbeta2
Comment 20•25 years ago
|
||
Adding mlk keyword because of bug 27739 (which was marked as a duplicate of this
one).
Keywords: mlk
Comment 22•25 years ago
|
||
Comment 23•25 years ago
|
||
*** Bug 40291 has been marked as a duplicate of this bug. ***
Comment 24•25 years ago
|
||
*** Bug 41232 has been marked as a duplicate of this bug. ***
Comment 25•25 years ago
|
||
It's actually not too hard to fix the keybindings leak without fixing this bug
(which is probably hard). I wrote code to do this a few weeks ago, but I
couldn't test very easily because I ran into bug 42895.
However, I realize now I could use the steps in bug 41232 to test my fix...
I'll try doing this soon.
Comment 26•25 years ago
|
||
Updated•25 years ago
|
Comment 27•25 years ago
|
||
Let bug 17390 cover the leak issues from this point on. Removing mlk keyword
from this bug but adding perf keyword since perfomance issues were discussed for
the real issue in this bug.
Comment 28•24 years ago
|
||
So hyatt, can we get somebody else to help you do this? or do this for you? The
current keybinding stuff has significant problems with leaks and cleaning itself
up. (Although dbaron has tracked down most of those issues...)
Comment 29•24 years ago
|
||
nsbeta3-, not worth the risk for seamonkey.
Whiteboard: [nsbeta2-] → [nsbeta2-],[nsbeta3-]
Comment 30•24 years ago
|
||
Workaround for embedding is to use a XUL layer. Otherwise this is huge and risky.
Updated•24 years ago
|
Target Milestone: M20 → Future
Comment 31•24 years ago
|
||
Adding nsbeta2 keyword to bugs with nsbeta2 triage value in status field so the
queries don't get screwed up
Keywords: nsbeta2
Comment 32•24 years ago
|
||
Reassign to dr, since he just filed bug 52359.
Assignee: hyatt → dr
Status: ASSIGNED → NEW
Assignee | ||
Comment 33•24 years ago
|
||
Hmm. Can I mark this as a dup of bug 52359, or vice-versa? The only thing that's
left to do here is the browserBindings and editorBindings, right?
Status: NEW → ASSIGNED
Assignee | ||
Comment 34•24 years ago
|
||
this is a dup of bug 52359, which will be fixed rsn as part of 48758.
*** This bug has been marked as a duplicate of 52359 ***
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•