Hi!

Steps to reproduce:

  • Add field to entity
  • Go to entity Manage Form Display
  • Here the default value is shown, Default Color: #FFFFFF
  • Press edit, add another value (ff00ff in my case) and press update

Result:

  • Default Color: # is shown instead of Default Color: #ff00ff
  • Dumping out the value in JQueryColorpickerDefaultWidget::settingsSummary(), the $this->getSetting('color') is NULL
  • Pressing edit again, the color field is also empty

Edit: The value is NULL also in the exported entity_form_display config.

CommentFileSizeAuthor
#6 jquery_color_2888430-6.patch840 bytesaudriusb
#4 Color_picker_manage_form_display.png58.58 KBAardWolf
#4 Color_picker_manage_field.png22.82 KBAardWolf
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mhavelant created an issue. See original summary.

mhavelant’s picture

mhavelant CreditAttribution: mhavelant at Brainsum for Tieto commented
Issue summary: View changes
Jaypan’s picture

Jaypan CreditAttribution: Jaypan commented
Status: Active » Closed (cannot reproduce)

I am unable to reproduce this. If someone else is able to reproduce it, please re-open this thread.

AardWolf’s picture

AardWolf CreditAttribution: AardWolf at ADCI Solutions commented
Version: 8.x-1.3 » 8.x-1.4
Status: Closed (cannot reproduce) » Active
FileSize
Color_picker_manage_field.png22.82 KB
Color_picker_manage_form_display.png58.58 KB

Hi.
I've installed module yesterday.
There are some screenshots.
I added some fields and set default values to #000000 value.
Field edit screenshot
On a "manage form display" form I can see it has #FFFFFF values.
But selected by red fields must be #000000.
Manage display screenshot
Please check again and fix it please.
Reopened.

Jaypan’s picture

Jaypan CreditAttribution: Jaypan commented

Thanks for providing additional information. I'm a little pressed for time these days, but next chance I have to look at it, I will.

audriusb’s picture

audriusb CreditAttribution: audriusb commented
FileSize
jquery_color_2888430-6.patch840 bytes

here's a quick fix

audriusb’s picture

audriusb CreditAttribution: audriusb commented
Status: Active » Needs review
Jaypan’s picture

Jaypan CreditAttribution: Jaypan commented
Status: Needs review » Closed (outdated)

Marking as outdated, as version 8.x-2.x is a major re-write, and should have dealt with any issues like this. Please re-open if the issue still stands after upgrade.