In the issue #1452318: Panels IPE Breaks When Region Style Wrapper Applied This issue is described. The title was copied since it covers this issue aswell.
When I apply a custom region style (eg. List) and enable Panels IPE as Panels renderer, the problem occur as soon as the second element is added to a region. (Look at screenshot)
In the JS function DrupalPanelsIPE.createSortContainers the .panels-ipe-sort-container wrapper is attached twice if there is an extra wrapper in the region style.
The class "panels-ipe-portlet-marker" on "panels-ipe-placeholder" dosen't do anything so I believe that it's safe to remove it.
For some reason I haven't figured out. The extra wrapper in the default pane style is also outcommented. http://drupalcode.org/project/panels.git/blob/2bdcf24e4248674a0e22b988ec...
Comment | File | Size | Author |
---|---|---|---|
#2 | 1838544-panels_ipe_region_style.patch | 1.33 KB | trogels |
#1 | 1838544-panels_ipe_region_style.patch | 1.33 KB | trogels |
Screen Shot 2012-11-12 at 5.29.03 PM.png | 60.17 KB | trogels |
Comments
Comment #1
trogels CreditAttribution: trogels commentedComment #2
trogels CreditAttribution: trogels commentedComment #3
trogels CreditAttribution: trogels commentedWell the above patch didn't work on list style as intended anyway.
It works on my custom region style.
Comment #4
mpotter CreditAttribution: mpotter commentedI ran into this same issue when trying to create a simple css-class wrapper custom style for Open Atrium 2. Whenever I would use the IPE to add a widget to a region with my wrapper style, the widget would get duplicated: once at the top in a panels-ipe-paneid-new-1 wrapper, and once down below within the proper custom style wrapper.
Applying this patch fixed this problem. I'm aware that the patch might need more work to fix the list style, but for basic wrapper styles it's good.
(btw, this bug also affects the Accordian style in Panopoly. Unfortunately this patch does not work with Accordian either and causes an Ajax error when a new Text widget is added to the region and saved. So, marking it as Needs Work)
Comment #5
mpotter CreditAttribution: mpotter commentedComment #6
hefox CreditAttribution: hefox commentedDuplicate of/related #1354572: IPE - multiple JS sort containers?
Comment #7
mpotter CreditAttribution: mpotter commentedNo, actually the patch in #1354572: IPE - multiple JS sort containers works in Open Atrium 2 for fixing this issue, but the patch in #2 here does not. I mispoke in #4 as we were using the other patch and not this one.
Maybe want to close this as a dup and point people to 1354572 instead??
Comment #7.0
mpotter CreditAttribution: mpotter commentedfixing link to other issue
Comment #8
elliotc CreditAttribution: elliotc commented