I tried using the predefined color box widget but it is not using the default colors I set. It just uses the ones that the module comes with. It does save my defaults but it just does not use them. See attached image.

CommentFileSizeAuthor
#5 patched2.png35.34 KBsdstyles
#5 patched.png150.57 KBsdstyles
#5 no_patch2.png38.01 KBsdstyles
#5 no_patch.png159.27 KBsdstyles
#1 predefined-colors-2477681-1.patch836 bytessdstyles
Screen Shot 2015-04-24 at 7.51.11 PM.png181.3 KBcaspervoogt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sdstyles’s picture

sdstyles CreditAttribution: sdstyles at FFW commented
Status: Active » Needs review
FileSize
predefined-colors-2477681-1.patch836 bytes

Changed to use colors from field instance settings.

targoo’s picture

targoo CreditAttribution: targoo at MariqueCalcus commented

sorry guys but I can't reproduce it. Works for me.

targoo’s picture

targoo CreditAttribution: targoo at MariqueCalcus commented
Status: Needs review » Closed (cannot reproduce)
caspervoogt’s picture

caspervoogt CreditAttribution: caspervoogt at Plethora commented

@targoo; the patch fixed the issue for me .. I just tested it. Maybe this should be committed and status set to Solved / Patch to be ported? Just sayin' - two people had this issue and the patch did fix it.

sdstyles’s picture

sdstyles CreditAttribution: sdstyles at FFW commented
FileSize
no_patch.png159.27 KB
no_patch2.png38.01 KB
patched.png150.57 KB
patched2.png35.34 KB

Hi guys,
@caspervoogt, you are right, the issue is still present.

@targoo, I've changed the "Default colors" (added my colors), and saved field settings after field settings edit, "COLOR FIELD SETTINGS" are not displaying my new colors, also I can't see my colors on node add/edit page.
See screenshots, tested with 7.x-2.x branch.
Thanks

sdstyles’s picture

sdstyles CreditAttribution: sdstyles at FFW commented
Status: Closed (cannot reproduce) » Needs review
targoo’s picture

targoo CreditAttribution: targoo at MariqueCalcus commented

ok i'll give it another try. Thanks for the screenshots.

  • targoo committed 9a54a08 on 7.x-2.x 
    Issue #2477681 by sdstyles, caspervoogt, targoo: Predefined color boxes...
targoo’s picture

targoo CreditAttribution: targoo at MariqueCalcus commented
Status: Needs review » Closed (fixed)
caspervoogt’s picture

caspervoogt CreditAttribution: caspervoogt at Plethora commented

sdtyles should get credit for the patch. We just reported the issue.

sdstyles’s picture

sdstyles CreditAttribution: sdstyles at FFW commented

I don't see this commit on my profile. Any thoughts?

targoo’s picture

targoo CreditAttribution: targoo commented
targoo’s picture

targoo CreditAttribution: targoo commented

trying to attribute to sdstyles..