Jdlrobson has uploaded a new change for review.

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


Change subject: Rewrite collapsibleNav as LESS file
......................................................................

Rewrite collapsibleNav as LESS file

* Rewrote css file as a LESS file for better maintainability
* Make use of core embed mixin
** Add .background-image-svg mixin for rendering svgs as background images

Change-Id: I72a66553d6ecee5018d4efaabfd0fa0df6c4135f
---
M resources/Resources.php
D skins/vector/collapsibleNav.css
A skins/vector/collapsibleNav.less
3 files changed, 90 insertions(+), 95 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/91/86691/1

diff --git a/resources/Resources.php b/resources/Resources.php
index 2ba24e9..0e3b660 100644
--- a/resources/Resources.php
+++ b/resources/Resources.php
@@ -110,7 +110,7 @@
        ),
        'skins.vector.collapsibleNav' => array(
                'styles' => array(
-                       'vector/collapsibleNav.css',
+                       'vector/collapsibleNav.less',
                ),
                'scripts' => array(
                        'vector/collapsibleNav.js',
diff --git a/skins/vector/collapsibleNav.css b/skins/vector/collapsibleNav.css
deleted file mode 100644
index 8c43a2c..0000000
--- a/skins/vector/collapsibleNav.css
+++ /dev/null
@@ -1,94 +0,0 @@
-/**
- * Stylesheet for collapsible nav
- */
-
-#mw-panel.collapsible-nav .portal {
-       /* @embed */
-       background: url(images/portal-break.png) left top no-repeat;
-       padding: 0.25em 0 !important;
-       margin: -11px 9px 10px 11px;
-}
-
-#mw-panel.collapsible-nav .portal h3 {
-       color: #4D4D4D;
-       font-weight: normal;
-       /* @embed */
-       background: url(images/open.png) left center no-repeat;
-       /* SVG support using a transparent gradient to guarantee cross-browser
-        * compatibility (browsers able to understand gradient syntax support 
also SVG) */
-       /* @embed */
-       background-image: -webkit-linear-gradient(transparent, transparent), 
url(images/open.svg);
-       /* @embed */
-       background-image: linear-gradient(transparent, transparent), 
url(images/open.svg);
-       padding: 4px 0 3px 1.5em;
-       margin-bottom: 0;
-}
-
-#mw-panel.collapsible-nav .portal h3:hover {
-       cursor: pointer;
-       text-decoration: none;
-}
-
-#mw-panel.collapsible-nav .portal h3 a {
-       color: #4D4D4D;
-       text-decoration: none;
-}
-
-#mw-panel.collapsible-nav .portal .body {
-       background-image: none !important;
-       padding-top: 0;
-       display: none;
-}
-
-#mw-panel.collapsible-nav .portal .body ul li {
-       padding: 0.25em 0;
-}
-
-/* First */
-#mw-panel.collapsible-nav .portal.first h3 {
-       display: none;
-}
-
-#mw-panel.collapsible-nav .portal.first {
-       background-image: none;
-       margin-top: 0;
-}
-
-/* Persistent */
-
-#mw-panel.collapsible-nav .portal.persistent .body {
-       display: block;
-}
-
-#mw-panel.collapsible-nav .portal.persistent h3 {
-       background-image: none !important;
-       padding-left: 0.7em;
-       cursor: default;
-}
-
-#mw-panel.collapsible-nav .portal.persistent .body {
-       margin-left: 0.5em;
-}
-
-/* Collapsed */
-
-#mw-panel.collapsible-nav .portal.collapsed h3 {
-       color: #0645AD;
-       /* @embed */
-       background: url(images/arrow-collapsed-ltr.png) left center no-repeat;
-       /* SVG support using a transparent gradient to guarantee cross-browser
-        * compatibility (browsers able to understand gradient syntax support 
also SVG) */
-       /* @embed */
-       background-image: -webkit-linear-gradient(transparent, transparent), 
url(images/arrow-collapsed-ltr.svg);
-       /* @embed */
-       background-image: linear-gradient(transparent, transparent), 
url(images/arrow-collapsed-ltr.svg);
-       margin-bottom: 0;
-}
-
-#mw-panel.collapsible-nav .portal.collapsed h3 a {
-       color: #0645AD;
-}
-
-#mw-panel.collapsible-nav .portal.collapsed h3:hover {
-       text-decoration: underline;
-}
diff --git a/skins/vector/collapsibleNav.less b/skins/vector/collapsibleNav.less
new file mode 100644
index 0000000..c3779db
--- /dev/null
+++ b/skins/vector/collapsibleNav.less
@@ -0,0 +1,89 @@
+/**
+ * LESS Stylesheet for collapsible nav
+ */
+@import "mediawiki.mixins.less";
+
+#mw-panel.collapsible-nav {
+       .portal {
+               background-position: left top;
+               background-repeat: no-repeat;
+               .background-image('images/portal-break.png');
+               padding: 0.25em 0 !important;
+               margin: -11px 9px 10px 11px;
+
+               h3 {
+                       color: #4D4D4D;
+                       font-weight: normal;
+                       background-position: left center;
+                       background-repeat: no-repeat;
+                       .background-image-svg('images/open.svg', 
'images/open.png');
+                       padding: 4px 0 3px 1.5em;
+                       margin-bottom: 0;
+
+                       &:hover {
+                               cursor: pointer;
+                               text-decoration: none;
+                       }
+
+                       a {
+                               color: #4D4D4D;
+                               text-decoration: none;
+                       }
+               }
+
+               .body {
+                       background-image: none !important;
+                       padding-top: 0;
+                       display: none;
+
+                       ul {
+                               li {
+                                       padding: 0.25em 0;
+                               }
+                       }
+               }
+
+
+               /* First */
+               &.first {
+                       background-image: none;
+                       margin-top: 0;
+                       h3 {
+                               display: none;
+                       }
+               }
+
+               /* Persistent */
+               &.persistent {
+                       .body {
+                               display: block;
+                               margin-left: 0.5em;
+                       }
+
+                       h3 {
+                               background-image: none !important;
+                               padding-left: 0.7em;
+                               cursor: default;
+                       }
+               }
+
+               /* Collapsed */
+               &.collapsed {
+                       h3 {
+                               color: #0645AD;
+                               background-position: left center;
+                               background-repeat: no-repeat;
+                               
.background-image-svg('images/arrow-collapsed-ltr.svg', 
'images/arrow-collapsed-ltr.png');
+                               margin-bottom: 0;
+
+                               &:hover {
+                                       text-decoration: underline;
+                               }
+
+                               a {
+                                       color: #0645AD;
+                               }
+                       }
+               }
+       }
+}

-- 
To view, visit https://gerrit.wikimedia.org/r/86691
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I72a66553d6ecee5018d4efaabfd0fa0df6c4135f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to