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
Issue fork ui_patterns_settings-3200469
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
Comment #2
nedjoI'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.
Comment #3
nedjo#3200449: Support CSS classes to set widget background color was applied, so this is no longer blocked.
Comment #4
christian.wiedemann commentedcan you provide an MR than I will merge it.
Comment #5
christian.wiedemann commented