Bug 1410498 - Draw autoscroll icon on a 28x28 canvas. r? draft
authorIan Moody <moz-ian@perix.co.uk>
Wed, 13 Dec 2017 03:16:48 +0000
changeset 719288 f9f3014493f4fd63a429eef8161437e568a0c807
parent 717183 ca379fcca95b1f4a3744242ea8647004b99b3507
child 745744 f1bda1e17dface75088d610a688657a14b082c29
push id95200
push usermoz-ian@perix.co.uk
push dateThu, 11 Jan 2018 19:49:45 +0000
bugs1410498
milestone59.0a1
Bug 1410498 - Draw autoscroll icon on a 28x28 canvas. r? The icon is currently drawn with lines too thick because it's drawn on a 16x16 canvas like toolbar icons, but then blown up to 28x28. Also simplify by removing some redundant attributes, and using just stroke for the border rather than a separate circle. MozReview-Commit-ID: 5ZH70X6IGit
toolkit/themes/shared/icons/autoscroll-horizontal.svg
toolkit/themes/shared/icons/autoscroll-vertical.svg
toolkit/themes/shared/icons/autoscroll.svg
--- a/toolkit/themes/shared/icons/autoscroll-horizontal.svg
+++ b/toolkit/themes/shared/icons/autoscroll-horizontal.svg
@@ -1,9 +1,8 @@
-<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" height="48" width="48">
-  <circle id="compassbackground" r="6" cx="8" cy="8" fill="#f9f9fa" fill-opacity=".8"/>
-  <circle id="compassborder" r="7" cx="8" cy="8" fill="none" stroke="#0c0c0d" stroke-width="2" stroke-linejoin="round" stroke-miterlimit="0" stroke-opacity=".8"/>
-  <circle id="compasscenter" cx="8" cy="8" r="1" fill="#0c0c0d" fill-opacity=".8"/>
-  <g id="westeast" stroke="#0c0c0d" stroke-opacity=".8" fill="none" stroke-linecap="round" stroke-linejoin="round">
-    <path d="M4.5 7l-1 1 1 1"/>
-    <path d="M11.5 7l1 1-1 1"/>
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 28 28" width="28" height="28">
+  <circle id="compassbackground" cx="14" cy="14" r="12" fill="#f9f9fa" fill-opacity=".8" stroke="#0c0c0d" stroke-opacity=".8" stroke-width="2"/>
+  <circle id="compasscenter" cx="14" cy="14" r="1" fill="#0c0c0d" fill-opacity=".8"/>
+  <g id="westeast" fill="none" stroke="#0c0c0d" stroke-opacity=".8" stroke-linecap="round" stroke-linejoin="round">
+    <path d="M8 12l-2 2 2 2"/>
+    <path d="M20 12l2 2-2 2"/>
   </g>
 </svg>
--- a/toolkit/themes/shared/icons/autoscroll-vertical.svg
+++ b/toolkit/themes/shared/icons/autoscroll-vertical.svg
@@ -1,9 +1,8 @@
-<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" height="48" width="48">
-  <circle id="compassbackground" r="6" cx="8" cy="8" fill="#f9f9fa" fill-opacity=".8"/>
-  <circle id="compassborder" r="7" cx="8" cy="8" fill="none" stroke="#0c0c0d" stroke-width="2" stroke-linejoin="round" stroke-miterlimit="0" stroke-opacity=".8"/>
-  <circle id="compasscenter" cx="8" cy="8" r="1" fill="#0c0c0d" fill-opacity=".8"/>
-  <g id="northsouth" fill="none" fill-opacity=".8" stroke="#0c0c0d" stroke-opacity=".8" stroke-linecap="round" stroke-linejoin="round">
-    <path d="M7 4.5l1-1 1 1"/>
-    <path d="M7 11.5l1 1 1-1"/>
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 28 28" width="28" height="28">
+  <circle id="compassbackground" cx="14" cy="14" r="12" fill="#f9f9fa" fill-opacity=".8" stroke="#0c0c0d" stroke-opacity=".8" stroke-width="2"/>
+  <circle id="compasscenter" cx="14" cy="14" r="1" fill="#0c0c0d" fill-opacity=".8"/>
+  <g id="northsouth" fill="none" stroke="#0c0c0d" stroke-opacity=".8" stroke-linecap="round" stroke-linejoin="round">
+    <path d="M12 8l2-2 2 2"/>
+    <path d="M12 20l2 2 2-2"/>
   </g>
 </svg>
--- a/toolkit/themes/shared/icons/autoscroll.svg
+++ b/toolkit/themes/shared/icons/autoscroll.svg
@@ -1,13 +1,12 @@
-<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" height="48" width="48">
-  <circle id="compassbackground" r="6" cx="8" cy="8" fill="#f9f9fa" fill-opacity=".8"/>
-  <circle id="compassborder" r="7" cx="8" cy="8" fill="none" stroke="#0c0c0d" stroke-width="2" stroke-linejoin="round" stroke-miterlimit="0" stroke-opacity=".8"/>
-  <circle id="compasscenter" cx="8" cy="8" r="1" fill="#0c0c0d" fill-opacity=".8"/>
-  <g id="northsouth" fill="none" fill-opacity=".8" stroke="#0c0c0d" stroke-opacity=".8" stroke-linecap="round" stroke-linejoin="round">
-    <path d="M7 4.5l1-1 1 1"/>
-    <path d="M7 11.5l1 1 1-1"/>
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 28 28" width="28" height="28">
+  <circle id="compassbackground" cx="14" cy="14" r="12" fill="#f9f9fa" fill-opacity=".8" stroke="#0c0c0d" stroke-opacity=".8" stroke-width="2"/>
+  <circle id="compasscenter" cx="14" cy="14" r="1" fill="#0c0c0d" fill-opacity=".8"/>
+  <g id="northsouth" fill="none" stroke="#0c0c0d" stroke-opacity=".8" stroke-linecap="round" stroke-linejoin="round">
+    <path d="M12 8l2-2 2 2"/>
+    <path d="M12 20l2 2 2-2"/>
   </g>
-  <g id="westeast" stroke="#0c0c0d" stroke-opacity=".8" fill="none" stroke-linecap="round" stroke-linejoin="round">
-    <path d="M4.5 7l-1 1 1 1"/>
-    <path d="M11.5 7l1 1-1 1"/>
+  <g id="westeast" fill="none" stroke="#0c0c0d" stroke-opacity=".8" stroke-linecap="round" stroke-linejoin="round">
+    <path d="M8 12l-2 2 2 2"/>
+    <path d="M20 12l2 2-2 2"/>
   </g>
 </svg>