Closed Bug 1336634 Opened 7 years ago Closed 7 years ago

no color in context menu icons

Categories

(Firefox :: Theme, defect)

51 Branch
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: vitalik.perevertun, Unassigned)

Details

(Keywords: regression)

Attachments

(3 files)

Attached image context_menu.png
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:50.0) Gecko/20100101 Firefox/50.0
Build ID: 20161208153507

Steps to reproduce:

1. create userChrome.css in user profile with content
/* Firefox userChrome.css */

@namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");

#context-bookmarkpage image
{
    -moz-image-region: rect(48px, 16px, 64px, 0);
    list-style-image: url("Silk.png");
}
/* (or just install Menu Icons Plus addon) */

2. open new tab on any site(or empty)
3. open context menu (right click mouse)




Actual results:

"back", "next", "reload", "bookmark" buttons have no color(see screenshot after)

firefox 51


Expected results:

"back", "next", "reload", "bookmark" buttons must have color(see screenshot before)

firefox 50
Narrowed down the regression range to the following set of changes:
Last good revision: 417d1e27c8eaf63a737abd95cc26d912332f232f
First bad revision: a1727a7c7f2f5cb3d732c475567e421aed2f0ba8
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=417d1e27c8eaf63a737abd95cc26d912332f232f&tochange=a1727a7c7f2f5cb3d732c475567e421aed2f0ba8

Dao, would you please take a look?
Status: UNCONFIRMED → NEW
Component: Untriaged → Theme
Ever confirmed: true
Flags: needinfo?(dao+bmo)
Keywords: regression
[Tracking Requested - why for this release]:

browser:Firebox 51 

In my browser context menu icons are appearing with color.

It works properly in latest Version.
Attached image 50 vs 51.jpg
I've used the default settings from the comment 0 addon, with no custom css for context, the issue reproduces in today's release build as well. (see attachment)
This isn't a Firefox bug, it needs to be fixed in your userChrome.css code or in the add-on, respectively.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(dao+bmo)
Resolution: --- → INVALID
Try this:

#context-navigation > .menuitem-iconic > .menu-iconic-left > .menu-iconic-icon {
  filter: none;
}
thanks It works.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: