[MediaWiki-commits] [Gerrit] mediawiki...VisualEditor[master]: Unify SVG markup

2017-11-06 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388933 )

Change subject: Unify SVG markup
..


Unify SVG markup

Align SVG markup across Foundation products by:
- unifying XML declaration,
- removing unnecessary `standalone="no"` attribute as it's default,
- removing all SVG editor metadata
- removing all `g` group elements except where necessary for transforms,
- putting attributes on `path`,
- removing unnecessary ids and
- unifying whitespace.

Saving up to 23% file size before gzipping.

Bug: T178867
Change-Id: If0339073a12f24df1a0f8dcc110c35e0ee2d38a7
---
M modules/ve-mw/ui/styles/images/icons/alienextension.svg
M modules/ve-mw/ui/styles/images/icons/image-frame.svg
M modules/ve-mw/ui/styles/images/icons/image-frameless.svg
M modules/ve-mw/ui/styles/images/icons/image-none.svg
M modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
M modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
M modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
M modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
M modules/ve-mw/ui/styles/images/icons/parameter-set-rtl.svg
M modules/ve-mw/ui/styles/images/icons/profile.svg
M modules/ve-mw/ui/styles/images/icons/public-domain.svg
M modules/ve-wmf/betafeatures-icon-VisualEditor-ltr.svg
M modules/ve-wmf/betafeatures-icon-VisualEditor-rtl.svg
M modules/ve-wmf/betafeatures-icon-WikitextEditor-ltr.svg
M modules/ve-wmf/betafeatures-icon-WikitextEditor-rtl.svg
15 files changed, 179 insertions(+), 295 deletions(-)

Approvals:
  jenkins-bot: Verified
  Jforrester: Looks good to me, approved



diff --git a/modules/ve-mw/ui/styles/images/icons/alienextension.svg 
b/modules/ve-mw/ui/styles/images/icons/alienextension.svg
index 8188e4f..cbaaed8 100644
--- a/modules/ve-mw/ui/styles/images/icons/alienextension.svg
+++ b/modules/ve-mw/ui/styles/images/icons/alienextension.svg
@@ -1,7 +1,5 @@
-
+
 http://www.w3.org/2000/svg"; 
xmlns:xlink="http://www.w3.org/1999/xlink"; width="24" height="24" viewBox="0 0 
24 24">
-
-
-
-
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-frame.svg 
b/modules/ve-mw/ui/styles/images/icons/image-frame.svg
index bec0e3e..75bbde0 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-frame.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-frame.svg
@@ -1,8 +1,6 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
-
+   
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-frameless.svg 
b/modules/ve-mw/ui/styles/images/icons/image-frameless.svg
index 0feaa9d..b88698d 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-frameless.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-frameless.svg
@@ -1,7 +1,5 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-none.svg 
b/modules/ve-mw/ui/styles/images/icons/image-none.svg
index 3c6d979..50a6965 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-none.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-none.svg
@@ -1,7 +1,5 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg 
b/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
index 142768c..8fbf492 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
@@ -1,10 +1,8 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
-
-
-
+   
+   
+   
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg 
b/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
index 813c540..7bc4094 100644
--- a/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
+++ b/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
@@ -1,6 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg 
b/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
index d66fc97..e257557 100644
--- a/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
+++ b/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
@@ -1,6 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg 
b/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
index 5e4a76b..86ea844 100644
--- a/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
+++ b/modules/ve-mw/ui/styles/images/icon

[MediaWiki-commits] [Gerrit] mediawiki...VisualEditor[master]: Unify SVG markup

2017-11-03 Thread VolkerE (Code Review)
VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/388933 )

Change subject: Unify SVG markup
..

Unify SVG markup

Align SVG markup across Foundation products by:
- unifying XML declaration,
- removing unnecessary `standalone="no"` attribute as it's default,
- removing all SVG editor metadata
- removing all `g` group elements except where necessary for transforms,
- putting attributes on `path`,
- removing unnecessary ids and
- unifying whitespace.

Saving up to 23% file size before gzipping.

Bug: T178867
Change-Id: If0339073a12f24df1a0f8dcc110c35e0ee2d38a7
---
M modules/ve-mw/ui/styles/images/icons/alienextension.svg
M modules/ve-mw/ui/styles/images/icons/image-frame.svg
M modules/ve-mw/ui/styles/images/icons/image-frameless.svg
M modules/ve-mw/ui/styles/images/icons/image-none.svg
M modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
M modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
M modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
M modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
M modules/ve-mw/ui/styles/images/icons/parameter-set-rtl.svg
M modules/ve-mw/ui/styles/images/icons/profile.svg
M modules/ve-mw/ui/styles/images/icons/public-domain.svg
M modules/ve-wmf/betafeatures-icon-VisualEditor-ltr.svg
M modules/ve-wmf/betafeatures-icon-VisualEditor-rtl.svg
M modules/ve-wmf/betafeatures-icon-WikitextEditor-ltr.svg
M modules/ve-wmf/betafeatures-icon-WikitextEditor-rtl.svg
15 files changed, 177 insertions(+), 291 deletions(-)


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

diff --git a/modules/ve-mw/ui/styles/images/icons/alienextension.svg 
b/modules/ve-mw/ui/styles/images/icons/alienextension.svg
index 8188e4f..9a02e94 100644
--- a/modules/ve-mw/ui/styles/images/icons/alienextension.svg
+++ b/modules/ve-mw/ui/styles/images/icons/alienextension.svg
@@ -1,4 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; 
xmlns:xlink="http://www.w3.org/1999/xlink"; width="24" height="24" viewBox="0 0 
24 24">
 
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-frame.svg 
b/modules/ve-mw/ui/styles/images/icons/image-frame.svg
index bec0e3e..75bbde0 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-frame.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-frame.svg
@@ -1,8 +1,6 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
-
+   
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-frameless.svg 
b/modules/ve-mw/ui/styles/images/icons/image-frameless.svg
index 0feaa9d..b88698d 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-frameless.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-frameless.svg
@@ -1,7 +1,5 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-none.svg 
b/modules/ve-mw/ui/styles/images/icons/image-none.svg
index 3c6d979..50a6965 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-none.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-none.svg
@@ -1,7 +1,5 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
+   
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg 
b/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
index 142768c..5fa8486 100644
--- a/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
+++ b/modules/ve-mw/ui/styles/images/icons/image-thumbnail.svg
@@ -1,10 +1,8 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
-
-
-
-
+
+
+
+
+
 
diff --git a/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg 
b/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
index 813c540..7bc4094 100644
--- a/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
+++ b/modules/ve-mw/ui/styles/images/icons/parameter-ltr.svg
@@ -1,6 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
+   
 
diff --git a/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg 
b/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
index d66fc97..039f370 100644
--- a/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
+++ b/modules/ve-mw/ui/styles/images/icons/parameter-rtl.svg
@@ -1,6 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
-
-
+
 
diff --git a/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg 
b/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
index 5e4a76b..ed7b8e8 100644
--- a/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
+++ b/modules/ve-mw/ui/styles/images/icons/parameter-set-ltr.svg
@@ -1,8 +1,6 @@