Esanders has uploaded a new change for review.

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

Change subject: Icons: Remove 'source' icon in favour of identical OOUI 
'wikiText' icon
......................................................................

Icons: Remove 'source' icon in favour of identical OOUI 'wikiText' icon

Depends on the next release of OOUI after Ia84f0594c.

Bug: T111038
Change-Id: Idd5d1499ac1499cde40271597a03c97e01c21925
---
M modules/ve-mw/ui/dialogs/ve.ui.MWTransclusionDialog.js
M modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js
D modules/ve-mw/ui/styles/images/icons/source.svg
M modules/ve-mw/ui/styles/ve.ui.Icons.css
M modules/ve-mw/ui/tools/ve.ui.MWEditModeTool.js
5 files changed, 4 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/72/250972/1

diff --git a/modules/ve-mw/ui/dialogs/ve.ui.MWTransclusionDialog.js 
b/modules/ve-mw/ui/dialogs/ve.ui.MWTransclusionDialog.js
index bede4f0..a6de7c7 100644
--- a/modules/ve-mw/ui/dialogs/ve.ui.MWTransclusionDialog.js
+++ b/modules/ve-mw/ui/dialogs/ve.ui.MWTransclusionDialog.js
@@ -331,7 +331,7 @@
        } );
        this.addContentButton = new OO.ui.ButtonWidget( {
                framed: false,
-               icon: 'source',
+               icon: 'wikiText',
                title: ve.msg( 'visualeditor-dialog-transclusion-add-content' )
        } );
        this.addParameterButton = new OO.ui.ButtonWidget( {
diff --git a/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js 
b/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js
index fcecd1b..92a1722 100644
--- a/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js
+++ b/modules/ve-mw/ui/pages/ve.ui.MWTransclusionContentPage.js
@@ -44,7 +44,7 @@
                .connect( this, { click: 'onRemoveButtonClick' } );
        this.valueFieldset = new OO.ui.FieldsetLayout( {
                label: ve.msg( 'visualeditor-dialog-transclusion-content' ),
-               icon: 'source',
+               icon: 'wikiText',
                $content: this.textInput.$element
        } );
 
@@ -69,7 +69,7 @@
 
        if ( this.outlineItem ) {
                this.outlineItem
-                       .setIcon( 'source' )
+                       .setIcon( 'wikiText' )
                        .setMovable( true )
                        .setRemovable( true )
                        .setLabel( ve.msg( 
'visualeditor-dialog-transclusion-content' ) );
diff --git a/modules/ve-mw/ui/styles/images/icons/source.svg 
b/modules/ve-mw/ui/styles/images/icons/source.svg
deleted file mode 100644
index 0aef5b9..0000000
--- a/modules/ve-mw/ui/styles/images/icons/source.svg
+++ /dev/null
@@ -1,9 +0,0 @@
-<?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="source">
-        <path id="opening-bracket-inner" d="M7 19h3v-2H9V7h1V5H7z"/>
-        <path id="closing-bracket-inner" d="M17 19h-3v-2h1V7h-1V5h3z"/>
-        <path id="closing-bracket-outer" d="M21 19h-3v-2h1V7h-1V5h3z"/>
-        <path id="opening-bracket-outer" d="M3 19h3v-2H5V7h1V5H3z"/>
-    </g>
-</svg>
diff --git a/modules/ve-mw/ui/styles/ve.ui.Icons.css 
b/modules/ve-mw/ui/styles/ve.ui.Icons.css
index b59be6d..2cba38a 100644
--- a/modules/ve-mw/ui/styles/ve.ui.Icons.css
+++ b/modules/ve-mw/ui/styles/ve.ui.Icons.css
@@ -147,11 +147,6 @@
        background-image: url( images/icons/ref-cite-web.svg );
 }
 
-.oo-ui-icon-source {
-       /* @embed */
-       background-image: url( images/icons/source.svg );
-}
-
 .oo-ui-icon-template {
        /* @embed */
        background-image: url( images/icons/template.svg );
diff --git a/modules/ve-mw/ui/tools/ve.ui.MWEditModeTool.js 
b/modules/ve-mw/ui/tools/ve.ui.MWEditModeTool.js
index 2f8e197..a4c74ce 100644
--- a/modules/ve-mw/ui/tools/ve.ui.MWEditModeTool.js
+++ b/modules/ve-mw/ui/tools/ve.ui.MWEditModeTool.js
@@ -56,7 +56,7 @@
 };
 OO.inheritClass( ve.ui.MWEditModeSourceTool, ve.ui.MWEditModeTool );
 ve.ui.MWEditModeSourceTool.static.name = 'editModeSource';
-ve.ui.MWEditModeSourceTool.static.icon = 'source';
+ve.ui.MWEditModeSourceTool.static.icon = 'wikiText';
 ve.ui.MWEditModeSourceTool.static.title =
        OO.ui.deferMsg( 'visualeditor-mweditmodesource-tool' );
 /**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idd5d1499ac1499cde40271597a03c97e01c21925
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders <[email protected]>

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

Reply via email to