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.
At _layout--grid.sass you get:
#content
.no-sidebars &
// 100% width for small screens.
@include fill-parent
But you should get:
#content
.no-sidebar &
// 100% width for small screens.
@include fill-parent
to suit this line at html.html.twig template:
(not sidebar_first and not sidebar_second) ? 'no-sidebar',
Adding patch here...
Comment | File | Size | Author |
---|---|---|---|
#5 | basic_no-sidebar-wrong-class_2846367-5.patch | 451 bytes | leahtard |
_layout--grid.patch | 310 bytes | calbasi |
Comments
Comment #2
calbasiComment #3
calbasiComment #4
joelpittetWhere does the patch need to be ported to?
https://www.drupal.org/node/156119#patch-to-be-ported
Comment #5
leahtard CreditAttribution: leahtard at The Jibe commentedThanks Calbasi,
I have updated this patch to be generated from 8.x-1.x-dev. I had recently made some updates here as well. The usage of fill-parent was actually redundant so I removed it. It explains why we haven't run into this issue before.
Cheers, Leah
Comment #7
leahtard CreditAttribution: leahtard at The Jibe commentedComment #8
calbasiThanks @leahtard :-) This is my first credited issue at drupal.org :-p
Ps @joelpittet I'm sorry, I didn't get the right meaning of that status... Thanks you both, and other contrib, for this wonderful theme.
Comment #9
leahtard CreditAttribution: leahtard at The Jibe commentedThanks for taking the time to report @calbasi :)