Extend configuration schema for CTools blocks with generic CSS styles (CSS ID, class and inline styles).

Related issues:
#2849867: Custom attributes in panels blocks and variants
#2947714: Add classes, IDs, and styles to blocks
#2947720: Custom CSS classes for ctools entity view blocks.

Please test and review.
Thanks.

CommentFileSizeAuthor
#5 interdiff_3001713_2-5.txt470 bytesnevergone
#5 3001713-css_styles_for_blocks-5.patch658 bytesnevergone
#2 3001713-css_styles_for_blocks-2.patch655 bytesnevergone

Issue fork ctools-3001713

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:

About issue forks

Comments

nevergone created an issue. See original summary.

nevergone’s picture

nevergone
Primary language Hungarian
commented
StatusFileSize
new 3001713-css_styles_for_blocks-2.patch655 bytes
nevergone’s picture

nevergone
Primary language Hungarian
commented
Issue summary: View changes
nevergone’s picture

nevergone
Primary language Hungarian
commented

Related issue and patch:
https://www.drupal.org/project/panels/issues/2849867#comment-12786362

nevergone’s picture

nevergone
Primary language Hungarian
commented
StatusFileSize
new 3001713-css_styles_for_blocks-5.patch658 bytes
new interdiff_3001713_2-5.txt470 bytes

Rename CSS Id.

denes.szabo’s picture

denes.szabo commented
Status: Needs review » Reviewed & tested by the community

I tested it, works! Thank you!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

How tests this feature?

nevergone’s picture

nevergone
Primary language Hungarian
commented

This only complements the schema. Related issue: #2849867: Custom attributes in panels blocks and variants

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Thanks!
I want learn more about this module.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Issue summary: View changes
alphawebgroup’s picture

alphawebgroup
he/him
Primary language English
Location Kyiv / Toronto
commented

Is there any chance to get it merged?

kunal_sahu made their first commit to this issue’s fork.

kunal_sahu opened merge request !45

kunal_sahu’s picture

kunal_sahu commented

Hi , i tried the patch. LGTM.

kunal@LAPTOP-G4FTKS5H MINGW64 /c/xampp/htdocs/contributions/web/modules/contrib/ctools-3001713 (3001713-css-style-properties)
$ git apply -v 3001713-css_styles_for_blocks-5.patch
Checking patch config/schema/ctools.schema.yml...
Applied patch config/schema/ctools.schema.yml cleanly.

RTBC++ for #5.

Created and MR for helping the maintainer.
Please review and merge this MR.

Thanks

vetchneons made their first commit to this issue’s fork.

vetchneons opened merge request !70

vetchneons opened merge request !71

liam morland made their first commit to this issue’s fork.

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Version: 8.x-3.x-dev » 4.1.x-dev
Status: Reviewed & tested by the community » Needs review

Re-rolled merge request 71. The other merge requests should probably be closed since they target branches that are not supported.