1292727 - Consolidate context menu navigation icon styling

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Dão Gottwald [:dao]]

Attachment: patch

Comment 1

[Dão Gottwald [:dao]]

Attachment: patch v2

Turns out this can be simplified further.

Comment 2

[:Paolo Amadini]

Comment on attachment 8778519 [details] [diff] [review]
patch v2

Review of attachment 8778519 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/themes/shared/contextmenu.inc.css
@@ +13,5 @@
> +  height: 16px;
> +  margin: 7px;
> +  filter: url(chrome://browser/skin/filters.svg#fill);
> +  fill: currentColor;
> +}

Looks great! Thanks for this unification.

::: browser/themes/windows/content-contextmenu.svg
@@ -1,4 @@
> -<?xml version="1.0"?>
> -<!-- This Source Code Form is subject to the terms of the Mozilla Public
> -   - License, v. 2.0. If a copy of the MPL was not distributed with this
> -   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->

Please use "hg mv" on one of the platform files so we can keep track of the origin of the shapes using hg blame, or link this bug to bug 1016405.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5734f8d9f39c
Consolidate context menu navigation icon styling. r=paolo

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5734f8d9f39c