Catrope has uploaded a new change for review.

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

Change subject: Hack around OOUI's icon pack being too large by creating our own
......................................................................

Hack around OOUI's icon pack being too large by creating our own

Bug: T112401
Change-Id: Id2a527dc1f4846985beb2cc4d5562a27d5636d55
---
M Hooks.php
M Resources.php
A modules/icons/apex/badgeicons.json
A modules/icons/bell.svg
A modules/icons/bellOn-ltr.svg
A modules/icons/bellOn-rtl.svg
A modules/icons/mediawiki/badgeicons.json
A modules/icons/speechBubble-ltr.svg
A modules/icons/speechBubble-rtl.svg
9 files changed, 70 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/39/238239/1

diff --git a/Hooks.php b/Hooks.php
index 3d26498..62be237 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -591,7 +591,7 @@
                        // Load the styles for the Notifications badge
                        $out->addModuleStyles( array(
                                'ext.echo.nojs',
-                               'oojs-ui.styles.icons-alerts'
+                               'ext.echo.badgeicons'
                        ) );
                }
 
diff --git a/Resources.php b/Resources.php
index ed261ef..a9f2d3d 100644
--- a/Resources.php
+++ b/Resources.php
@@ -153,6 +153,17 @@
                        'echo-feedback',
                ),
        ),
+
+       // HACK: OOUI has an icon pack for these, but it's unhelpfully large 
and we don't
+       // want to load more as render-blocking CSS than we have to (T112401)
+       'ext.echo.badgeicons' => $echoResourceTemplate + array(
+               'class' => 'ResourceLoaderOOUIImageModule',
+               'position' => 'top',
+               'name' => 'badgeicons',
+               'rootPath' => 'icons',
+               'selectorWithoutVariant' => '.oo-ui-{name}',
+               'selectorWithVariant' => 
'.oo-ui-image-{variant}.oo-ui-icon-{name}',
+       ),
 );
 
 unset( $echoResourceTemplate );
diff --git a/modules/icons/apex/badgeicons.json 
b/modules/icons/apex/badgeicons.json
new file mode 100644
index 0000000..4182a1a
--- /dev/null
+++ b/modules/icons/apex/badgeicons.json
@@ -0,0 +1,14 @@
+{
+       "prefix": "oo-ui-icon",
+       "images": {
+               "bell": { "file": "../bell.svg" },
+               "bellOn": { "file": {
+                       "ltr": "../bellOn-ltr.svg",
+                       "rtl": "../bellOn-rtl.svg"
+               } },
+               "speechBubble": { "file": {
+                       "ltr": "../speechBubble-ltr.svg",
+                       "rtl": "../speechBubble-rtl.svg"
+               } }
+       }
+}
diff --git a/modules/icons/bell.svg b/modules/icons/bell.svg
new file mode 100644
index 0000000..787ed14
--- /dev/null
+++ b/modules/icons/bell.svg
@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
+<svg xmlns="http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
+    <path d="M17.5 14V9c0-3-2.3-5-5.5-5S6.5 6 6.5 9v5c0 2 0 3-2 3v1h15v-1c-2 
0-2-1-2-3zM12 20H9c0 1 1.6 2 3 2s3-1 3-2h-3z"/>
+</svg>
diff --git a/modules/icons/bellOn-ltr.svg b/modules/icons/bellOn-ltr.svg
new file mode 100644
index 0000000..f68a8e0
--- /dev/null
+++ b/modules/icons/bellOn-ltr.svg
@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
+<svg xmlns="http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
+    <path d="M17.8 14.7l1.7-4.7c1-2.8-.5-5.5-3.5-6.6s-5.9 0-6.9 2.8l-1.7 
4.7c-.7 1.9-1 2.8-2.9 2.1l-.3 1 14.1 5.1.3-.9c-1.9-.7-1.5-1.6-.8-3.5zM12 
19.8l-2.8-1c-.3.9.8 2.4 2.1 2.9s3.2.1 3.5-.9l-2.8-1z"/>
+</svg>
diff --git a/modules/icons/bellOn-rtl.svg b/modules/icons/bellOn-rtl.svg
new file mode 100644
index 0000000..ae03529
--- /dev/null
+++ b/modules/icons/bellOn-rtl.svg
@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
+<svg xmlns="http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
+    <path d="M6.21 14.7L4.51 10c-1-2.8.5-5.5 3.5-6.6 3-1.1 5.9 0 6.9 2.8l1.7 
4.7c.7 1.9 1 2.8 2.9 2.1l.3 1-14.1 5.1-.3-.9c1.9-.7 1.5-1.6.8-3.5zm5.8 
5.1l2.8-1c.3.9-.8 2.4-2.1 2.9s-3.2.1-3.5-.9l2.8-1z" id="path56"/>
+</svg>
diff --git a/modules/icons/mediawiki/badgeicons.json 
b/modules/icons/mediawiki/badgeicons.json
new file mode 100644
index 0000000..21c8220
--- /dev/null
+++ b/modules/icons/mediawiki/badgeicons.json
@@ -0,0 +1,20 @@
+{
+       "prefix": "oo-ui-icon",
+       "variants": {
+               "invert": {
+                       "color": "#FFFFFF",
+                       "global": true
+               }
+       },
+       "images": {
+               "bell": { "file": "../bell.svg" },
+               "bellOn": { "file": {
+                       "ltr": "../bellOn-ltr.svg",
+                       "rtl": "../bellOn-rtl.svg"
+               } },
+               "speechBubble": { "file": {
+                       "ltr": "../speechBubble-ltr.svg",
+                       "rtl": "../speechBubble-rtl.svg"
+               } }
+       }
+}
diff --git a/modules/icons/speechBubble-ltr.svg 
b/modules/icons/speechBubble-ltr.svg
new file mode 100644
index 0000000..6e5eacd
--- /dev/null
+++ b/modules/icons/speechBubble-ltr.svg
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<svg xmlns="http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
+    <g>
+        <path d="M19 20H2l3-3V6h17v11c0 1.7-1.3 3-3 3z"/>
+    </g>
+</svg>
diff --git a/modules/icons/speechBubble-rtl.svg 
b/modules/icons/speechBubble-rtl.svg
new file mode 100644
index 0000000..139c34e
--- /dev/null
+++ b/modules/icons/speechBubble-rtl.svg
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<svg xmlns="http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
+    <g id="g586">
+        <path d="M5 20h17l-3-3V6H2v11c0 1.7 1.3 3 3 3z" id="path588"/>
+    </g>
+</svg>

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id2a527dc1f4846985beb2cc4d5562a27d5636d55
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Catrope <roan.katt...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to