Problem/Motivation

Follow-up to #3186163: Add a color settings type (widget). Would require #3200449: Support CSS classes to set widget background color, therefore postponed.

A second use case that didn't make it into the fix for [#] is:

Selecting a class from a list. Typically in the corresponding component this class will be added to a wrapper.

This would be useful for CSS frameworks like Bulma and Bootstrap where a set of predefined CSS classes are used to apply color.

Proposed resolution

Extend the value of colorwidget options to include a third segment in addition to the current two. The third segment defaults to css_color. But if it's set to css_class, the resulting widget uses the css_class rather than css_color property. This will only be supported in colorwidget if #3200449: Support CSS classes to set widget background color goes in.

Remaining tasks

User interface changes

API changes

Data model changes

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

nedjo created an issue. See original summary.

nedjo’s picture

Issue summary: View changes

I've pushed a commit with a proposed fix: extend the value of colorwidget options to include a third segment in addition to the current two. The third segment defaults to css_color. But if it's set to css_class, the resulting widget uses the css_class rather than css_color property. This will only be supported in colorwidget if #3200449: Support CSS classes to set widget background color goes in.

nedjo’s picture

Status: Postponed » Needs review

#3200449: Support CSS classes to set widget background color was applied, so this is no longer blocked.

christian.wiedemann’s picture

can you provide an MR than I will merge it.

christian.wiedemann’s picture

Status: Needs review » Needs work