Viewport issues with Radix panels layouts

[katypool-zz]

So the update to Panopoly 1.14 switched panels layouts from the old Panopoly theme to Radix. But the Radix css isn't playing nice with Kalatheme and messes with fixed width. Read all about it here: https://www.drupal.org/node/2379315

Very easily fixable. There needs to be another container div in kalatheme's page.tpl.php in order for it to integrate seamlessly with the Radix layouts.
Now you know!

[shadcn]

We had to add the .container wrapper to Radix as well.

See the related issue here: https://www.drupal.org/node/2334871

[pirog]

thanks @arshad!

@katypool any chance we can coax a pull request from you with the container class added as per https://www.drupal.org/node/2334871?

[katypool-zz]

will do

[pirog]

THX

On Fri, Nov 21, 2014 at 11:49 AM, katypool notifications@github.com wrote:

will do

—
Reply to this email directly or view it on GitHub
#155 (comment)
.

Cheers,

Mike Pirog
Kalamuna
www.kalamuna.com

[labboy0276]

@pirog @reynoldsalec PR put in

[pirog]

not super familiar with the implications here so i defer to others.

if this is relevant lets make sure it gets into the 4x branch (ie kalatheme-source) as well.

[labboy0276]

fixes all things

[dariogcode]

I think this change is valid if we use the Radix layout direcly, without trying to override in the Kalatheme tpls.

From the last Panopoly update I noticed the Kalatheme tpl files for layouts (kalatheme\templates\panopoly) aren't overriding the Panopoly theme layouts (radix layouts)

Previous
In the kalatheme tpl files for layouts you will notice all use the ".container" class. which make unnecesary to add an additional container in page.tpl. This approach also let you to create a fuild container if you need it, in the panel layout. But as mentioned, this isn't working now.

Actual
With the latest Panopoly changes, Radix layout have the ".container-fluid" class which make everything fuild bcause our page.tpl don't have a container class. Adding a container in tpl will increase padding because we will have 2 container in the tree.

Hope this is clear, my apologize for my english.

Thanks!

[dariogcode]

Which works pretty good for me is change the name of the tpl in templates/panopoly adding "radix-" as prefix, for example bryant.tpl.php change to radix-bryant.tpl.php

[maxplus]

Hi tilon,

The old Panopoly 1.13 layouts worked perfect together with Kalatheme and so I was a little bit lost with the new Radix layouts because they have built in the default 12 columns grid and do not support to use a (for example) 24 columns grid.

=> now by changing the tpl name of the layout files (included in the templates folder of my custom theme, copied from the Kalatheme base folder) by adding "radix-", I can still keep using the current radix-layouts.

Thanks!

[gboudrias]

This is also occurring in v4. See PR 174.

[andrewmallis]

@madeofpeople – take a look svp?

[thiagodemellobueno]

Seems like using the radix layouts in panopoly is the way to go.
So @tilon 's suggestion is valid, but we shouldnt really duplicate those files so much as make the templates available to kalatheme?