Last modified: 2013-11-04 23:01:38 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T58366, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56366 - [Regression] Missing "media => screen" breaks Vector styles for printing
[Regression] Missing "media => screen" breaks Vector styles for printing
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.22.0
All All
: High normal (vote)
: 1.22.0 release
Assigned To: Jon
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-30 10:03 UTC by Michael M.
Modified: 2013-11-04 23:01 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Michael M. 2013-10-30 10:03:49 UTC
https://gerrit.wikimedia.org/r/#/c/79948/ removed the "media => screen" for vector/styles.less (See http://bits.wikimedia.org/en.wikipedia.org/load.php?debug=true&lang=en&modules=skins.vector&only=styles&skin=vector, the second part isn't wrapped in any @media clause). These styles now override the styles for printing, which is bad. Most noticeably:

* The font changed from serif (browser default) to sans-serif.
* The font size is smaller than before.
* Bulleted lists use colored bullets.

Please add the "media => screen" for these styles, or - if there are styles that should explicitly apply for printing, too, split them into a separate file.
Comment 1 Gerrit Notification Bot 2013-10-30 16:25:42 UTC
Change 92658 had a related patch set uploaded by Jdlrobson:
Add missing media screen

https://gerrit.wikimedia.org/r/92658
Comment 2 Gerrit Notification Bot 2013-10-30 17:40:36 UTC
Change 92672 had a related patch set uploaded by Bartosz Dziewoński:
Add missing media screen

https://gerrit.wikimedia.org/r/92672
Comment 3 Bartosz Dziewoński 2013-10-30 17:42:18 UTC
Fixed by Jon, backported to 1.22 release. Thanks Michael, we'd love to have more bug reporters like you :D
Comment 4 Krinkle 2013-10-30 17:45:23 UTC
It was removed because Jon merged the vector-hd styles into the same less file. That means the module style file *must not* have 'media' set, otherwise it will end up beig nested, which browsers do not support.
Comment 5 Jon 2013-10-30 18:12:53 UTC
I'm confused Krinkle why have you reopened this? Your patch looks identical..
Comment 6 Jon 2013-10-30 18:16:00 UTC
(me now super confused at the 3 identical patches)
Comment 7 Bartosz Dziewoński 2013-10-30 18:38:18 UTC
The patches were not merged, and there are only two, one for master and one for REL1_22.
Comment 8 DaB. 2013-11-01 12:11:52 UTC
The german-speaking community is concerned that printing wikipedia articles is now much harder then before (at least the reading). Could this bug please fixed soon or the patch, that introduce the problem, reverted?
Comment 9 Gerrit Notification Bot 2013-11-03 22:07:21 UTC
Change 93408 had a related patch set uploaded by Jdlrobson:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93408
Comment 10 Gerrit Notification Bot 2013-11-03 23:01:36 UTC
Change 92672 abandoned by Bartosz Dziewoński:
Add missing media screen

Reason:
Superseded by https://gerrit.wikimedia.org/r/#/c/93408/1

https://gerrit.wikimedia.org/r/92672
Comment 11 Gerrit Notification Bot 2013-11-03 23:01:58 UTC
Change 92658 abandoned by Bartosz Dziewoński:
Vector: Set media=screen on styles.less

Reason:
Superseded by https://gerrit.wikimedia.org/r/#/c/93408/1

https://gerrit.wikimedia.org/r/92658
Comment 12 Gerrit Notification Bot 2013-11-03 23:03:31 UTC
Change 93412 had a related patch set uploaded by Bartosz Dziewoński:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93412
Comment 13 Gerrit Notification Bot 2013-11-03 23:05:58 UTC
Change 93408 merged by jenkins-bot:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93408
Comment 14 Gerrit Notification Bot 2013-11-03 23:09:23 UTC
Change 93412 merged by jenkins-bot:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93412
Comment 15 Bartosz Dziewoński 2013-11-03 23:11:09 UTC
All merged and backported. (So many automated comments.)

Jon said he will try to get the patch deployed to Wikimedia wikis tomorrow.
If that doesn't happen, it will be deployed during regular deployments (within about two weeks).
Comment 16 Gerrit Notification Bot 2013-11-04 22:29:37 UTC
Change 93612 had a related patch set uploaded by Mwalker:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93612
Comment 17 Gerrit Notification Bot 2013-11-04 22:31:12 UTC
Change 93613 had a related patch set uploaded by Mwalker:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93613
Comment 18 Gerrit Notification Bot 2013-11-04 22:35:07 UTC
Change 93613 merged by Mwalker:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93613
Comment 19 Gerrit Notification Bot 2013-11-04 22:35:11 UTC
Change 93612 merged by Mwalker:
Vector: Set media screen on styles.less

https://gerrit.wikimedia.org/r/93612

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links