Updated Colorpicker CCK module [#284337]

I couldn't get the Colorpicker CCK module to work at all because there is no relationship between the textfields and the color picker. I reworked the module so there is only one colorpicker CCK type, which consists of a colorpicker and an associated textfield.

I added a couple other formatters - one to display the plain hex value of the field, one to display a box in the selected color, and one to display the hex value inside a colored box.

I got it set up so you can select the multiple option on the field to create several textfields that use the same colorpicker. Now when you select colors with the colorpicker they are stored in the textfields.

I also added dependencies to the .info file.

Comment	File	Size	Author
	colorpicker.patch	7.56 KB	KarenS

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

thinkyhead CreditAttribution: thinkyhead commented 2 September 2008 at 03:55

Assigned:

Unassigned

» thinkyhead

Comment #2

dwees CreditAttribution: dwees commented 3 September 2008 at 01:40

If this patch does all that it advertises, then it's a shoo-in to be committed. Have you tested this patch Scott? Do we need to create an update in the install file so that users can 'upgrade'?

Also, this list includes enough features that we may wish to create an 'official version' of this module.

What do you think?

Comment #3

rooby CreditAttribution: rooby commented 3 September 2008 at 05:40

I have used the patch and it works well. I haven't seen any bugs yet.

Comment #4

KarenS CreditAttribution: KarenS commented 3 September 2008 at 11:21

FWIW, in CCK we don't usually worry about creating upgrade paths from -dev versions that have never had official releases, we only worry about creating upgrade paths after there's been an official release (even if it's a beta or rc release).

You obviously can do whatever you want, though.

Comment #5

dwees CreditAttribution: dwees commented 3 September 2008 at 14:07

Well that's good then. That will make our lives easier. Also the upgrade path would be exceedingly difficult for this module given the big change in where the data is stored. Previous to this, the data was stored in the 'colortextfield' or whatever it used to be called, and the 'colorpicker' was just an element to allow users to be able to change the position of the colorpicker JS widget more easily. However, this should really be the role of of the theme for that particular content type.

Dave

Comment #6

brady53 CreditAttribution: brady53 commented 17 October 2008 at 21:56

There seems to be a problem with two color pickers on one node -- neither of them really work. The color wheel can be moved but the text fields never get updated.

Comment #7

dwees CreditAttribution: dwees commented 17 October 2008 at 22:07

This worked before this patch. You just had to manually associate the colorpicker with the colortextfields.

Dave

Comment #8

brady53 CreditAttribution: brady53 commented 31 October 2008 at 17:02

Can you give me some steps to "manually associate the colorpicker with the colortextfields" or point me to a node with those steps outlined?

Thanx in advance.

Comment #9

thinkyhead CreditAttribution: thinkyhead commented 23 July 2009 at 04:26

I take it this issue is closed? Also, are the D5 and D6 versions going to maintain any kind of parity or is the D5 version retiring? Just curious, as I'd love to see this get final polish and an official release. skilip seems to be on top of it, but if there's anything I can do to clean it up...