ComboBox CellEditor problem in FF2 and FF3

[mshillin]

The ComboBox CellEditor has an event handling problem in FF2 and FF3.  If the combobox dropdown list overlays rows in the table and you select an item, the focus indicator in the table (darkblue bar) will move to the row under the item which was selected.  A lightblue focus indicator remains on the row containing the ComboBox cell editor.

If the item you select is beyond the end of the table, the control works as expected.

You can see this behavior in the Table_CellEditor_1 demo.

I have traced this to the _onmouseup() function in qx.ui.form.Combobox.  The _onmouseup() function does not stop propogation of the event and it looks like the undelaying table gets the event and moves the dark blue  focus indicator.

Adding a "e.stopPropagation()" call in _onmouseup() in qx.ui.form.ComboBox seems to fix the problem, but don't know if it breaks anything else.

This problem does not exist in IE7.

[Hugh Gibson]

I suggest that you report this as a bug so it doesn't get lost. There's
been no official response that I can see.

Hugh

> The ComboBox CellEditor has an event handling problem in FF2 and
> FF3.  If the combobox dropdown list overlays rows in the table and
> you select an item, the focus indicator in the table (darkblue bar)
> will move to the row under the item which was selected.  A lightblue
> focus indicator remains on the row containing the ComboBox cell
> editor.
>
> If the item you select is beyond the end of the table, the control
> works as expected.
>
> You can see this behavior in the Table_CellEditor_1 demo.
>
> I have traced this to the _onmouseup() function in
> qx.ui.form.Combobox.  The _onmouseup() function does not stop
> propogation of the event and it looks like the undelaying table gets
> the event and moves the dark blue focus indicator.
>
> Adding a "e.stopPropagation()" call in _onmouseup() in
> qx.ui.form.ComboBox seems to fix the problem, but don't know if it
> breaks anything else.
>
> This problem does not exist in IE7.

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@...
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

[Hugh Gibson]

Looks like there's still a problem here in 0.8. Looking at current source,
there is in qx.ui.form.AbstractSelectBox.js:

          control.addListener("mouseup", this.close, this);
         
and this.close doesn't have any special stop propogation calls.        

Again, it seems that this report has been missed by the qooxdoo devs and
you should raise a bug report.

Hugh

> *Subject:* Re: [qooxdoo-devel] ComboBox CellEditor problem in FF2
> and FF3
> *From:* "Hugh Gibson" <hgibson@...>
> *To:* qooxdoo-devel@...
> *Date:* Thu, 24 Jul 2008 09:25 +0100 (BST)
>
> I suggest that you report this as a bug so it doesn't get lost.
> There's
> been no official response that I can see.
>
> Hugh
>
> > The ComboBox CellEditor has an event handling problem in FF2 and
> > FF3.  If the combobox dropdown list overlays rows in the table and
> > you select an item, the focus indicator in the table (darkblue
> > bar)
> > will move to the row under the item which was selected.  A
> > lightblue
> > focus indicator remains on the row containing the ComboBox cell
> > editor.
> >
> > If the item you select is beyond the end of the table, the
> > control works as expected.
> >
> > You can see this behavior in the Table_CellEditor_1 demo.
> >
> > I have traced this to the _onmouseup() function in
> > qx.ui.form.Combobox.  The _onmouseup() function does not stop
> > propogation of the event and it looks like the undelaying table
> > gets
> > the event and moves the dark blue focus indicator.
> >
> > Adding a "e.stopPropagation()" call in _onmouseup() in
> > qx.ui.form.ComboBox seems to fix the problem, but don't know if it
> > breaks anything else.
> >
> > This problem does not exist in IE7.

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@...
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel