Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#14 | colorpicker_cck.tar_.gz | 1.33 KB | alexpott |
Comments
Comment #1
dwees CreditAttribution: dwees commentedTesting 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)?
Comment #2
chrisroditis CreditAttribution: chrisroditis commentedThanks 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.
Comment #3
skilip CreditAttribution: skilip commentedThis is probably caused by a bug discussed here: #352761: warning: Missing argument 1
Comment #4
awolfey CreditAttribution: awolfey commentedsubscribe
Comment #5
drawk CreditAttribution: drawk commentedsubscribe
Comment #6
domidc CreditAttribution: domidc commentedI confirm having the same error
Comment #7
prom3theus CreditAttribution: prom3theus commentedHello, 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.
Comment #8
chales CreditAttribution: chales commentedsubscribe
Comment #9
dwees CreditAttribution: dwees commentedSorry 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
Comment #10
iaminawe CreditAttribution: iaminawe commentedsame problem ... subscribe
Comment #11
defunctcitizen CreditAttribution: defunctcitizen commentedand i thought i was losing my mind! the colorpicker and colortext fields also don't appear on edit node screens for me. subscribing...
Comment #12
defunctcitizen CreditAttribution: defunctcitizen commentedComment #13
skilip CreditAttribution: skilip commentedSorry 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.
Comment #14
alexpottI 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
Comment #15
eu.claudio CreditAttribution: eu.claudio commentedsubscribe
Comment #16
skilip CreditAttribution: skilip commentedI've committed a RC1 wich is ready for testing.
Comment #17
alexpottThe 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
to this fixes it
But this probably means we need to ensure that a default value is always set.
Thanks for the good work...
Alex
Comment #18
eu.claudio CreditAttribution: eu.claudio commentedJust 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
Comment #19
skilip CreditAttribution: skilip commentedThanks for reporting guys!
Comment #20
mattgilbert CreditAttribution: mattgilbert commentedsubscribing