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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

calbasi created an issue. See original summary.

calbasi’s picture

Status: Active » Needs review
calbasi’s picture

Status: Needs review » Patch (to be ported)
joelpittet’s picture

Where does the patch need to be ported to?

https://www.drupal.org/node/156119#patch-to-be-ported

leahtard’s picture

Version: 8.x-1.2 » 8.x-1.x-dev
Status: Patch (to be ported) » Needs review
FileSize
451 bytes

Thanks 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

  • leahtard authored 40d7c19 on 8.x-1.x
    Issue #2846367 by calbasi, leahtard: wrong class for no sidebars at...
leahtard’s picture

Status: Needs review » Fixed
calbasi’s picture

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

leahtard’s picture

Thanks for taking the time to report @calbasi :)

Status: Fixed » Closed (fixed)

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