please add:

#comments .comment,
#comments .comment .content,
#preface, #preface > div,
#primary .inside,
#primary .node,
#primary .node .content,
#content-pre-title,
#content-post-title,
#secondary,
#secondary > div,
#tertiary,
#tertiary > div,
#postscript,
#postscript > div,
#content,
#container,
#footer,
#closure,
#page {
    overflow: visible;
}

to the print.css

When all those divs are changed to 'overflow:hidden' in the default stylesheet they cut off half the content when attempting to print.

I guess an alternate method would be to add the overflow:hidden section to the screen.css

Comments

rhache’s picture

Part of my thinking was to reduce the following rule in default.css:

#comments .comment,
#comments .comment .content,
#preface,
#preface > div,
#primary .inside,
#primary .node,
#primary .node .content,
#content-pre-title,
#content-post-title,
#secondary,
#secondary > div,
#tertiary,
#tertiary > div,
#postscript,
#postscript > div,
#content,
#container,
#footer,
#closure,
#page {
	overflow: hidden;
}

to:

.block
#comments .comment,
#primary .node,
#container {
	overflow: hidden;
}

That would mean that the print stylesheet would only need this:

.block
#comments .comment,
#primary .node,
#container {
	overflow: visible;
}

Or maybe the reduction on items that are set to overflow hidden might make unnecessary to add the print rule?

Let me know what you think.

rhache

theshanergy’s picture

I've just tested something similar:

defaults.css

#container,
.region,
.node,
.block,
.comment {
  overflow: hidden;
}

and it seems to work well, with no override in print.css necessary.

Along with that css, we should add a 'region' class to the #primary div.inside - so that it follows the same convention as the secondary, tertiary, etc.

rhache’s picture

Change the theme to use the clear-block class instead in latest dev commit. Please test and let me know if it works out.

rhache

rhache’s picture

Status: Active » Fixed
rhache’s picture

Status: Fixed » Closed (fixed)