Opened 12 years ago

Closed 11 years ago

#6627 closed defect (invalid)

[dojox] dojox.form._ColorPicker should use dojox.color

Reported by: dante Owned by: dante
Priority: high Milestone: 1.4
Component: DojoX Form Version: 1.1.0
Keywords: Cc: Tom Trenka
Blocked By: Blocking:

Description

There is duplicated code in _ColorPicker ported from 0.4 before dojox.color had all the conversion utilities, making _ColorPicker a little bloaty. Rework _Picker to use dojox.color, and add in the ability to set a color by hex value (at least), and eventually from any of the available inputs (all set readonly for the time being)

Attachments (1)

dojox_widget_ColorPicker_attr_Value.patch​ (3.2 KB) - added by Shane O'Sullivan 11 years ago.
Patch to add attr function that handles setting the hex value

Download all attachments as: .zip

Change History (4)

comment:1 Changed 12 years ago by bill

Milestone: → tbd

mark all (open) tickets w/blank milestones to be "tbd"; their milestones need to be set to a version number or to "future"

comment:2 Changed 12 years ago by dante

Milestone: tbd → 1.4
Status: new → assigned

Changed 11 years ago by Shane O'Sullivan

Attachment: dojox_widget_ColorPicker_attr_Value.patch​ added

Patch to add attr function that handles setting the hex value

comment:3 Changed 11 years ago by dante

Resolution: → invalid
Status: assigned → closed

closing in light of newfound interest in dojox colorpicker ala #8169 - will merge this patch and that to bring picker up to date.

Note: See TracTickets for help on using tickets.