323251 - Colorpicker doesn't work anymore from web content

Status: RESOLVED DUPLICATE of bug 298665

(Toolkit :: UI Widgets, defect)

Description

[Martijn Wargers (dead)]

See upcoming testcase.
You can't choose a color anymore with this xul colorpicker, while this still works in current trunk SeaMonkey builds.
Firefox colorpicker:
http://lxr.mozilla.org/seamonkey/source/toolkit/content/widgets/colorpicker.xml
SeaMonkey colorpicker:
http://lxr.mozilla.org/seamonkey/source/xpfe/global/resources/content/bindings/colorpicker.xml

Maybe it works because SeaMonkey uses this.onselect(); in selectCell, while Firefox uses this._fireEvent(this, "select");.

This regressed between 2005-04-28 and 2005-04-29, very likely a regression from bug 289940

Comment 1

[Martijn Wargers (dead)]

Attachment: testcase

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Note also bug 298665 (doesn't seem related).

Comment 3

[:Gavin Sharp [email: [e-mail]]]

(well, semi-related, it also deals with fireEvent problems)

Comment 4

[Martijn Wargers (dead)]

This is a duplicate of bug 298665, isn't it?

Comment 5

[:Gavin Sharp [email: [e-mail]]]

Ah, I hadn't looked at the testcase. I guess it's a dupe, though it might be a good idea to test with the patch in that bug.

Comment 6

[Martijn Wargers (dead)]

Yes, tested the patch and that makes it work.
Note to self: search before you file.

*** This bug has been marked as a duplicate of 298665 ***