Last modified: 2012-09-27 01:14:30 UTC
Created attachment 6110 [details] include commonPrint.css The modern skin is severely broken when printing, because it doesn't include commonPrint.css (which sets styling for thumbs etc). Although some of the defines of commonPrint.css are totally useless, several others are highly critical. Since it seems that the modern/print.css was never really finished, I think that we should include commonPrint.css instead, at the very least until the modern skin is finished.
Updating bug summary.
i can't reproduce any severe brokenness. please describe the problem in more detail, and if possible, provide an example of a page that demonstrates it.
I'll attach three "print to pdf" files (Of the OS X print core, not "the mediawiki PDF version"). All the same article, with monobook skin, modern skin and a "patched" modern skin. Observe the differences in rendering of thumbs, links, TOC and the appending of href element to external links. Mostly dealt with by commonPrint.css I have no problem with Modern providing an additional print skin, but I don't see why it should ignore the "shared" one that sets some very reasonable options. There are several elements in commonPrint.css that are not used in modern (#content), but these can be fixed either later, or implemented in modern/print.css at a later time. Please ignore the articles headers (metadata extension in place there), and underlines (which i add myself to redirect links).
Created attachment 6112 [details] Print with Monobook skin
Created attachment 6113 [details] print with Modern skin
Created attachment 6114 [details] print with patched Modern skin
This problem with thumbnails in printed pages of the modern skin still exist. Either my patch needs applying, or the rules for thumb styling need to be copied from commonPrint.css to modern/print.css
Why don't you apply your patches? ;)
patch is super-obsolete (Resource Loader FTW!) but the problem remains.
Unassigning default assignments. http://article.gmane.org/gmane.science.linguistics.wikipedia.technical/54734
Comment on attachment 6110 [details] include commonPrint.css Obsolete, already applied (or otherwise changed as such)
This issue has been solved now.
