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

dsnopek’s picture

Title: New Radix layouts fill entire viewport » New Radix layouts fill entire viewport in Kalatheme

Hmm. 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.

dsnopek’s picture

An 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.

katyrose’s picture

Thanks, that did work!

dsnopek’s picture

@katyrose: That's great! Which thing did you do: use the sandbox module, or add the <div>?

katyrose’s picture

Added the div in Kalatheme's page template.

shadcn’s picture

Status: Active » Fixed

Hi 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).

dsnopek’s picture

Status: Fixed » Active

Actually, let's keep this open until we have a patch merged into Kalatheme!

tinto’s picture

Not 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:

<div id="content" class="">
  <div class="region region-content">
    <div id="block-system-main" class="block block-system">
      <div class="content">
        <div id="panels-ipe-display-panelizer-node-240-page-manager" class="panels-ipe-display-container panels-ipe-processed">
          <div class="panel-display burr-flipped clearfix radix-burr-flipped">
            <div class="container-fluid">
              <div class="row">
                <div class="col-md-8 radix-layouts-content panel-panel">

                [...]

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.

tinto’s picture

I'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.

dsnopek’s picture

@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?

tinto’s picture

Thanks 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.

thekenshow’s picture

Using 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.

thiagodemellobueno’s picture

@sonictruth
@labboy:

We should role this in to 3.x so this ticket can get closed, and incorporate it into futureversions™

thiagodemellobueno’s picture

aubjr_drupal’s picture

Check out our alternate fix to this older issue.