Skip to content

Commit 08389c6

Browse files
committed
Bug 1700124 - de-duplicate add (+) icons r=thecount,harry,desktop-theme-reviewers
* Rename toolkit's add.svg to plus.svg, use it for the zoom-in buttons and about:newtab's add icon Differential Revision: https://phabricator.services.mozilla.com/D118627
1 parent 90efefc commit 08389c6

File tree

15 files changed

+10
-27
lines changed

15 files changed

+10
-27
lines changed

browser/components/newtab/content-src/styles/_icons.scss

+1-1
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@
189189
}
190190

191191
&.icon-add {
192-
background-image: url('chrome://activity-stream/content/data/content/assets/glyph-add-16.svg');
192+
background-image: url('chrome://global/skin/icons/plus.svg');
193193
}
194194

195195
&.icon-minimize {

browser/components/newtab/css/activity-stream-linux.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -346,7 +346,7 @@ body[lwt-newtab-brighttext] {
346346
background-image: url("chrome://global/skin/icons/arrow-up.svg");
347347
}
348348
.icon.icon-add {
349-
background-image: url("chrome://activity-stream/content/data/content/assets/glyph-add-16.svg");
349+
background-image: url("chrome://global/skin/icons/plus.svg");
350350
}
351351
.icon.icon-minimize {
352352
background-image: url("chrome://activity-stream/content/data/content/assets/glyph-minimize-16.svg");

browser/components/newtab/css/activity-stream-mac.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -350,7 +350,7 @@ body[lwt-newtab-brighttext] {
350350
background-image: url("chrome://global/skin/icons/arrow-up.svg");
351351
}
352352
.icon.icon-add {
353-
background-image: url("chrome://activity-stream/content/data/content/assets/glyph-add-16.svg");
353+
background-image: url("chrome://global/skin/icons/plus.svg");
354354
}
355355
.icon.icon-minimize {
356356
background-image: url("chrome://activity-stream/content/data/content/assets/glyph-minimize-16.svg");

browser/components/newtab/css/activity-stream-windows.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -346,7 +346,7 @@ body[lwt-newtab-brighttext] {
346346
background-image: url("chrome://global/skin/icons/arrow-up.svg");
347347
}
348348
.icon.icon-add {
349-
background-image: url("chrome://activity-stream/content/data/content/assets/glyph-add-16.svg");
349+
background-image: url("chrome://global/skin/icons/plus.svg");
350350
}
351351
.icon.icon-minimize {
352352
background-image: url("chrome://activity-stream/content/data/content/assets/glyph-minimize-16.svg");

browser/components/newtab/data/content/assets/glyph-add-16.svg

-6
This file was deleted.

browser/components/touchbar/MacTouchBar.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ var gBuiltInInputs = {
120120
},
121121
NewTab: {
122122
title: "new-tab",
123-
image: "chrome://global/skin/icons/add.svg",
123+
image: "chrome://global/skin/icons/plus.svg",
124124
type: kInputTypes.BUTTON,
125125
callback: () => execCommand("cmd_newNavigatorTabNoEvent"),
126126
},

browser/installer/allowed-dupes.mn

-4
Original file line numberDiff line numberDiff line change
@@ -118,10 +118,6 @@ browser/chrome/browser/skin/classic/browser/customize.svg
118118
browser/features/[email protected]/icons/lightbulb.svg
119119
chrome/toolkit/skin/classic/global/icons/lightbulb.svg
120120

121-
# Bug 1700124 - De-duplicate browser/themes/shared icons
122-
browser/chrome/browser/content/activity-stream/data/content/assets/glyph-add-16.svg
123-
browser/chrome/browser/skin/classic/browser/zoom-in.svg
124-
125121
# Bug 1710546 - Bundle Firefox extension as a builtin addon Nighly only
126122
# We plan to remove this duplicity after Firefox Translations become pref'd on
127123
browser/chrome/browser/builtin-addons/translations/wasm/cld-worker.js.mem

browser/themes/osx/browser.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ moz-input-box > menupopup .context-menu-add-engine > .menu-iconic-left {
259259
min-height: 24px;
260260
height: 24px;
261261
color: inherit;
262-
list-style-image: url("chrome://global/skin/icons/add.svg");
262+
list-style-image: url("chrome://global/skin/icons/plus.svg");
263263
}
264264

265265
#editBMPanel_newFolderButton:hover {

browser/themes/shared/icons/zoom-in.svg

-6
This file was deleted.

browser/themes/shared/jar.inc.mn

-1
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,6 @@
185185
skin/classic/browser/tab.svg (../shared/icons/tab.svg)
186186
skin/classic/browser/whatsnew.svg (../shared/icons/whatsnew.svg)
187187
skin/classic/browser/window.svg (../shared/icons/window.svg)
188-
skin/classic/browser/zoom-in.svg (../shared/icons/zoom-in.svg)
189188
skin/classic/browser/zoom-out.svg (../shared/icons/zoom-out.svg)
190189
skin/classic/browser/ion.svg (../shared/icons/ion.svg)
191190

browser/themes/shared/tabs.inc.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -690,7 +690,7 @@
690690

691691
#tabs-newtab-button,
692692
#TabsToolbar #new-tab-button {
693-
list-style-image: url(chrome://global/skin/icons/add.svg);
693+
list-style-image: url(chrome://global/skin/icons/plus.svg);
694694
}
695695

696696
/* All tabs button and menupopup */

browser/themes/shared/toolbarbutton-icons.inc.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,7 @@ toolbar[brighttext]:-moz-lwtheme {
335335
}
336336

337337
#zoom-in-button {
338-
list-style-image: url("chrome://browser/skin/zoom-in.svg");
338+
list-style-image: url("chrome://global/skin/icons/plus.svg");
339339
}
340340

341341
#zoom-in-button > .toolbarbutton-icon {

toolkit/components/aboutconfig/content/aboutconfig.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ td.cell-value > form > input[type="number"] {
228228
}
229229

230230
.button-add {
231-
background-image: url("chrome://global/skin/icons/add.svg");
231+
background-image: url("chrome://global/skin/icons/plus.svg");
232232
}
233233

234234
.button-save {
File renamed without changes.

toolkit/themes/shared/jar.inc.mn

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ toolkit.jar:
3030
skin/classic/global/numberinput.css (../../shared/numberinput.css)
3131
skin/classic/global/in-content/info-pages.css (../../shared/in-content/info-pages.css)
3232
skin/classic/global/in-content/toggle-button.css (../../shared/in-content/toggle-button.css)
33-
skin/classic/global/icons/add.svg (../../shared/icons/add.svg)
3433
skin/classic/global/icons/arrow-left.svg (../../shared/icons/arrow-left.svg)
3534
skin/classic/global/icons/arrow-right.svg (../../shared/icons/arrow-right.svg)
3635
skin/classic/global/icons/autoscroll.svg (../../shared/icons/autoscroll.svg)
@@ -63,6 +62,7 @@ toolkit.jar:
6362
skin/classic/global/icons/performance.svg (../../shared/icons/performance.svg)
6463
skin/classic/global/icons/plugin.svg (../../shared/icons/plugin.svg)
6564
skin/classic/global/icons/plugin-blocked.svg (../../shared/icons/plugin-blocked.svg)
65+
skin/classic/global/icons/plus.svg (../../shared/icons/plus.svg)
6666
skin/classic/global/icons/pocket.svg (../../shared/icons/pocket.svg)
6767
skin/classic/global/icons/print.svg (../../shared/icons/print.svg)
6868
skin/classic/global/icons/undo.svg (../../shared/icons/undo.svg)

0 commit comments

Comments
 (0)