Steps to reproduce problem: 1. Open a colour picker e.g. Format/Text Colour 2. Double-click on a colour cell Expected results: dialog closes Actual results: JS error
Oops, forgot to set owner :-(
Assignee: syd → cmanske
Keywords: patch, polish, review, ui
Whiteboard: [FIX IN HAND][need sr=][adt2 RTM]
This was obviously caused by converting "onOk" methods to "onAccept" and the doublclick handler wasn't changed.
Status: NEW → ASSIGNED
Summary: Error: onOK is not defined → "onOK" should be "onAccept" in Color Picker for doubleclick color seletion
Target Milestone: --- → mozilla1.0.1
Comment on attachment 87492 [details] [diff] [review] Proposed patch sr=alecf
Attachment #87492 - Flags: superreview+
fixed in trunk Nominating to be fixed in branch.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Keywords: adt1.0.1, mozilla1.0.1
Resolution: --- → FIXED
Whiteboard: [FIX IN HAND][need sr=][adt2 RTM] → [fixed in trunk][adt2 RTM]
verified on trunk 0617.
Whiteboard: [fixed in trunk][adt2 RTM] → [fixed in trunk][adt2 RTM][verified-trunk]
adding adt1.0.1+. Please get drivers approval before checking into the branch.
Keywords: adt1.0.1 → adt1.0.1+
*** Bug 123790 has been marked as a duplicate of this bug. ***
Comment on attachment 87492 [details] [diff] [review] Proposed patch Approval granted for 1.0 branch checkin; please remove the mozilla1.0.1+ keyword when this is fixed and add the fixed 1.0.1 keyword.
Attachment #87492 - Flags: approval+
please checkin to the 1.0.1 branch. once there, remove the "mozilla1.0.1+" keyword and add the "fixed1.0.1" keyword.
Keywords: mozilla1.0.1 → mozilla1.0.1+
checked into mozilla1.0.1 branch
Keywords: mozilla1.0.1+ → fixed1.0.1
verified in 7/17 branch build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.