Sibling issue: #2334871: Radix Layouts breaks Panopoly for nested panels (e.g. teaser lists)

So according to the bootstrap docs it looks like the .container class can not be nested. This broke in bootstrap 3.0.1 when the .container changed from a max-width property to a width property.

This issue is apparent when a View displays teaser nodes and the teaser view mode renders as a panel layout.

To fix this, I've moved the .container class to page.tpl.php all the time instead of checking to see if we're not using panels. In the other issue I linked to for radix_layouts I'll add a patch that removes the .container from each layout. This should effectively have the same effect for theming but remove the nested divs.

Patch coming shortly.

CommentFileSizeAuthor
#1 radix-add_page_content_container_class-2353825-1.patch633 bytescaschbre

Comments

caschbre’s picture

caschbre commented
Title: Add container class to content div that wraps panel layouts » Avoid nested container class by moving it to page.tpl.php instead of layouts.
Status: Active » Needs review
StatusFileSize
new radix-add_page_content_container_class-2353825-1.patch633 bytes

And here's the patch. This goes along with the patch in the related issue.

shadcn’s picture

shadcn commented
Status: Needs review » Fixed

This is fixed in another ticket. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.