Needs work
Project:
Page Manager
Version:
8.x-4.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
20 Apr 2016 at 15:02 UTC
Updated:
3 Apr 2025 at 17:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
omarlopesinoComment #3
omarlopesinoAttach patch with a possible solution.
Settings are added in 'Operations' button, and styles added in buildBlock method.
Can you check this please?
Thanks!
Comment #4
omarlopesinoChanging status to Needs review.
Comment #6
omarlopesinoSeeing the tests failed i found some errors in my patch.
Attach new patch with correction.
Comment #7
omarlopesinoComment #9
omarlopesinoAttach updated patch adding schema info.
Comment #10
omarlopesinoComment #11
omarlopesinoComment #12
jwineichen commentedI attempted to install this but I still don't see any options to add classes and ids.
Comment #13
andypostThis fixes regression otoh maybe better to implement more generic on top of https://www.drupal.org/project/styles_api
Comment #14
dobrzyns commentedI also do not see any options to add classes and ids. If I have some time, I'll take a closer look.
Comment #15
andypostDon't hack related issues
Comment #16
dobrzyns commented@andypost Sorry about that. The removal of the related issue was unintentional. Thanks for catching that and adding it back.
Comment #17
hypertext200This appeared to be not working.
Comment #18
samerali commentedThis appears not to be working per @heshnlk said.
The problem seems that the settings does actually exist at the "Edit Block" level.
But when you load them through PageBlockDisplayVariant->buildRegions it builds the configuration array without the css (or the new custom) attributes.
any idea why this is going on?
Comment #19
samerali commentedActually this page_manager-block-css-attributes-9.patch does work.
I totally missed that i should have clicked "update & save" to save the tempstore into the configuration.
Thanks for the efforts @mistermope
Comment #20
samerali commentedComment #21
bbujisic commentedPatch page_manager-block-css-attributes-9.patch worked nice for new module installations. However, it required
hook_update_N()for existing page manager config.New patch loops through existing page variant config and amends block settings where needed.
Comment #22
samerali commentedI have been using #9 for a while and it's working great for me (real help), I think @bbujisic #21 just perfected it. Any reason we are not merging this yet?
Comment #23
andypostI'm sure, resave should happens only when both properties is unset.
+ needs newline at the end file
Comment #24
jian he commentedFix #23 and reroll to 8.x-4.x
Comment #25
max-kuzomko commentedComment #26
max-kuzomko commentedFor some reason there is no option to run 4.x tests on drupal.org
Comment #27
jlballes commentedI have applied the last patch and I don't see the fields Css classes and Css ID
I have tried to run update.php at it don´t find any update. Of course I´ve deleted cache with drush cr
My Page Manager version is 8.x-4.0-beta2
Thanks in advanced
Comment #28
ericmulder1980 commentedIf i understand correctly this issue is about adding custom attributes in the Panelizer UI forms. I don't see any reference however to being able to add / edit custom attributes while using panels IPE. Would it be an option to expand the scope of this issue to also implement this?
Comment #29
ptsimard commentedReroll #24 on top of latest commit (8.x-4.0-rc3)