Hello and thanks for the great usability module!
What is the status of the colorpicker CCK? I have installed it, and added fields to a content type but it doesn't appear to do anything on the node add form.

CommentFileSizeAuthor
#14 colorpicker_cck.tar_.gz1.33 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dwees’s picture

Testing required here, let me get back to you. One quick question, just to eliminate the obvious, do you have JavaScript enabled (required for this module)?

chrisroditis’s picture

Thanks for your quick reply!
yes, javascript is enabled. I am also getting this error when the module is enabled:

* warning: Invalid argument supplied for foreach() in F:\sitename\includes\menu.inc on line 258.
* warning: Invalid argument supplied for foreach() in F:\sitename\includes\menu.inc on line 258.

skilip’s picture

This is probably caused by a bug discussed here: #352761: warning: Missing argument 1

awolfey’s picture

subscribe

drawk’s picture

subscribe

domidc’s picture

I confirm having the same error

container is undefined
[Break on this error] return container.farbtastic || (contai...Query._farbtastic(container, callback));
prom3theus’s picture

Hello, this should be a great module, but all of 6.x versions don't displays the CCK field if I added to a content type, and then I going to create a content with this type.

Otherway, this will be really a significant module for CCK, so keep up the good work!

Thanks.

chales’s picture

subscribe

dwees’s picture

Sorry folks, I've been pretty busy with real life stuff outside of school. I haven't had a chance to look at this module for a while. I suspect it might be a while before I get back into active Drupal development. If you want to submit patches that you've tested, I'm sure Skilip (or myself if I get a free moment) can commit them.

Thanks,

Dave

iaminawe’s picture

same problem ... subscribe

defunctcitizen’s picture

and i thought i was losing my mind! the colorpicker and colortext fields also don't appear on edit node screens for me. subscribing...

defunctcitizen’s picture

Priority: Normal » Critical
skilip’s picture

Sorry guys, both dwees and me are too busy ATM for replieing faster in colorpockers issuequeue. I will do my best to fix this this WE.

alexpott’s picture

FileSize
1.33 KB

I needed a colorpicker cck field so I gave it a go - upgrading the cck module to work with Drupal 6. The module takes a slightly different approach than the orginal module.

I hope this helps get the module upgraded to work with Drupal 6 sooner.

Known bugs: only one field per node... not sure why multiple fields don't work. Perhaps someone has an idea?

Cheers,

Alex

eu.claudio’s picture

subscribe

skilip’s picture

I've committed a RC1 wich is ready for testing.

alexpott’s picture

The new RC1 works for me...

The only thing that you might consider changing is how default values are handled when a colorpicker textfield is added to an existing node. At the moment it'll always default to red regardless of how you set the default value in the widget settings.

changing this line

  $value = isset($element['#value'][$field_key]) ? $element['#value'][$field_key] : '#FF0000';

to this fixes it

  $value = isset($element['#value'][$field_key]) ? $element['#value'][$field_key] : $field['widget']['default_value'][$element['#delta']]['value'];

But this probably means we need to ensure that a default value is always set.

Thanks for the good work...

Alex

eu.claudio’s picture

Just one problem found on RC1:
Using multiple values you can't delete fields. So if you add 3 colors to your node, you cannot remove any, just add more.
And it even adds one value every time you edit your node.

Thanks for the work anyway!

Claudio

skilip’s picture

Thanks for reporting guys!

mattgilbert’s picture

subscribing