Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Using Bootstrap-based Kalatheme 3 (https://www.drupal.org/project/kalatheme)
Updated Panopoly to 1.14, which is now using Radix layouts instead of the old Panopoly theme layouts. The layouts are now filling the entire viewport of my dev site. Something in the radix css is not kosher with Kalatheme, but after experimenting I am still not sure what's going on.
Looking at the issues for including Radix in Panopoly, it looks like someone may have been aware of this issue, but I don't see any steps to solving it...?
Comments
Comment #1
dsnopekHmm. Actually, I don't think anyone tested Kalatheme (at least I don't recall hearing about anyone testing it). I'll try it when I get a chance!
However, in the meantime, there is a sandbox module to return the layouts to their original templates:
https://www.drupal.org/sandbox/dsnopek/2378979
Because it's a sandbox module (as opposed to a full project), you'll need to use Git to download it. But it should get you back to the old state of your site while we look into the problem.
Comment #2
dsnopekAn idea just came to me - can you try adding
<div class="container">...</div>
around the main content area in your page template? This needed to be added to Radix to make it be fixed width. If this works, we can submit a patch to Kalatheme.Comment #3
katyrose CreditAttribution: katyrose commentedThanks, that did work!
Comment #4
dsnopek@katyrose: That's great! Which thing did you do: use the sandbox module, or add the
<div>
?Comment #5
katyrose CreditAttribution: katyrose commentedAdded the div in Kalatheme's page template.
Comment #6
shadcn CreditAttribution: shadcn commentedHi Katy, the
.container
class was added to Radix as well. See the related issue: #2334871: Radix Layouts breaks Panopoly for nested panels (e.g. teaser lists).Comment #7
dsnopekActually, let's keep this open until we have a patch merged into Kalatheme!
Comment #8
tintoNot using Kalatheme, but I'm experiencing the exact same issue with my custom theme (using Radix as base theme). The "content region" suddenly is filling the entire width of the browser window.
#2 does not work for me, because I end up with double padding form both container classes.
I see this in the HTML:
When I change
<div class="container-fluid">
to<div class="container">
, my layout switches back to normal.Now I need to decide where I'm going to override this, so that I don't run into trouble with future updates.
Comment #9
tintoI've done some more research and noticed that a patch was committed here (http://www.drupal.org/node/2334871#comment-9325097) that replaces all instances of class container with container-fluid in all Radix layouts, causing its contents to fill the entire window.
Is this really a Kalatheme-related issue? I suspect anyone using any Radix (sub)theme with a Radix layout would run into this problem.
Comment #10
dsnopek@tinto: Try the latest -dev version of Radix. This change should fix it. I've also opened an issue in the Radix queue asking the maintainer to make a new release so that the latest Radix is compatible with the latest Radix Layouts: #2389859: Make new release (3.0-beta3) for compatibility with latest radix_layouts?
Comment #11
tintoThanks dsnopek!
The patch seemed to mess up some of my layouts, so I ended up overriding the .container-fluid class with everything in .container. Not pretty, but it'll have to do for now.
Comment #12
thekenshow CreditAttribution: thekenshow commentedUsing Kalatheme and have same problem after updating from Panopoly 1.13 to 1.15. After wandering around trying to sort out the overlap between radix, radix_layouts, kalatheme and panopoly, I settled on #11 as the quickest way to get back to work.
Comment #13
thiagodemellobueno CreditAttribution: thiagodemellobueno commented@sonictruth
@labboy:
We should role this in to 3.x so this ticket can get closed, and incorporate it into futureversions™
Comment #14
thiagodemellobueno CreditAttribution: thiagodemellobueno commentedThe patch is here http://cgit.drupalcode.org/radix/patch/?id=4171d8d5351b5ceb898566e56bba3...
Comment #15
aubjr_drupal CreditAttribution: aubjr_drupal commentedCheck out our alternate fix to this older issue.