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 |
Comments
Comment #1
thinkyhead CreditAttribution: thinkyhead commentedComment #2
dwees CreditAttribution: dwees commentedIf 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 commentedI have used the patch and it works well. I haven't seen any bugs yet.
Comment #4
KarenS CreditAttribution: KarenS commentedFWIW, 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 commentedWell 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 commentedThere 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 commentedThis worked before this patch. You just had to manually associate the colorpicker with the colortextfields.
Dave
Comment #8
brady53 CreditAttribution: brady53 commentedCan 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 commentedI 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...