41235 - XBL bindings don't work

Status: VERIFIED WORKSFORME

(Core :: XBL, defect, P1)

Description

[Akkana Peck]

In the 6/1 build on Linux, XBL bindings don't work in text fields or textareas -
- neither XP bindings (e.g. left/right arrow) nor platform-specific ones (e.g.
^U on Unix), either in the urlbar or form controls.

They work in the editor (which I think still uses XUL bindings).  Mike says they
work in text fields on Windows.  I'm fairly sure they were working on Linux
yesterday.

Comment 1

[Akkana Peck]

Adding keywords: Recent regression, makes data entry in text fields painful.

Ah, here's a clue: this bugzilla page is wide enough to show a horizontal
scrollbar, and when I typed a rightarrow, the whole page scrolled.  So this may
be an event problem, with the events going to the page rather than to the
widget.  Likewise, if I type enough here to get a vertical scrollbar in the
textarea, up/downarrows cause a scroll, rather than moving the caret as they should.

Comment 2

[John Morrison]

*** Bug 41231 has been marked as a duplicate of this bug. ***

Comment 3

[David Hyatt]

ben probably didn't patch the xbl inheritance chain when he moved these files.

Comment 4

[leger]

Putting on [nsbeta2+][dogfood-] radar.  Does not need a fix ASAP for daily work, 
but we should fix this for beta2.

Comment 5

[Akkana Peck]

Aha: in the release builds, I notice that htmlBindings is there but
platformHTMLBindings is not.  I bet this is a Makefile or installer problem,
that the Unix platform file isn't being installed into the right place.

Requesting reconsideration as dogfood, because it's really been getting in the
way as I try to use dogfood today: aside from making it difficult to type, I
can't edit any urls longer than the urlbar, because I can't arrow forward.  And
the fix should be very simple.

Comment 6

[leger]

Putting on [dogfood+] radar.

Comment 7

[Akkana Peck]

Ben removed all the xml and xbl bindings files in version 1.78 of packages-unix
(and corresponding changes to the other platforms).  The comment just says
"Updating packager files" so I'm not sure why the files were removed, or whether
these files were intended to be referenced from somewhere else.  Ben, can we put
them back?

So this is probably XP after all, and only happens in release builds, not in
people's debug builds.

Comment 8

[don]

Make this P1 and now I'm off to find Ben ...

Comment 9

[Akkana Peck]

Ben, Don and I figured this out.  Turns out that it was operator error: I have a
user.css which I use for testing/demonstration which adds some keybindings,
which means that it has to point back to the existing keybinding files.  That
file had to be changed when the keybinding files moved (which I didn't know they
had).

So, mea culpa, it's working fine now.  But it does illustrate that after we
start to get lots of outsiders using mozilla and making skins, we should try
hard not to move important files like this, if we want people to be able to
write skins or user.css files which inherit from existing UI.

Comment 10

[John Morrison]

verified worksforme