From 7e7e532bfbb4563a71cbb3b557d642f2e1a6c8e0 Mon Sep 17 00:00:00 2001 From: jdlrobson Date: Tue, 1 Aug 2017 11:52:11 -0700 Subject: [PATCH] Relax navbox hack Per discussion in T172078 let's relax these rules so navbox shows up in the desktop mode of the site. Note: this will not impact the mobile site as navboxes are stripped from the HTML. Bug: T172078 Change-Id: I59d229d75dbcb790eb75dc4b838b2afafd2f3980 --- .../skins.minerva.content.styles/hacks.less | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/resources/skins.minerva.content.styles/hacks.less b/resources/skins.minerva.content.styles/hacks.less index f1a954a..0f07ee6 100644 --- a/resources/skins.minerva.content.styles/hacks.less +++ b/resources/skins.minerva.content.styles/hacks.less @@ -17,6 +17,11 @@ FIXME: Review all of these hacks to see if they still apply. } .content { + .vertical-navbox, + .navbox { + display: none; + } + // Hide cleanup templates by default to non-javascript users as these stop them from reading the article itself // Talk page informational boxes .tmbox, @@ -24,7 +29,7 @@ FIXME: Review all of these hacks to see if they still apply. .ambox, #coordinates, // Hide article badges, clean-up notices, stub notices, and navigation boxes - .navbox, .vertical-navbox, .topicon, .metadata { + .topicon, .metadata { // It's important as some of these are tables which become display: table on larger screens display: none !important; } @@ -112,6 +117,16 @@ FIXME: Review all of these hacks to see if they still apply. width: auto !important; } +@media all and ( min-width: @deviceWidthTablet ) { + // When in the HTML these should be revealed at tablet resolution (T172078) + .content { + .vertical-navbox, + .navbox { + display: inherit; + } + } +} + // Hacks to render galleries and multicol tables better on mobile @media all and ( max-width: @deviceWidthTablet ) {