PolymerElements / iron-icons

A set of icons for use with iron-icon
https://www.webcomponents.org/element/PolymerElements/iron-icons
105 stars 61 forks source link

Iron icons in chrome do not work #63

Closed erinhong17 closed 7 years ago

erinhong17 commented 7 years ago

Some icons (seems to be the icons under the "icons:" set of icons whereas icons in "communications:" work) In the chrome inspect tool, the icon is detectable but is not visible.

DOES NOT work on Chrome

Version 51.0.2704.79 (64-bit) on Mac Book Pro OS X El Captain Version 10.11.4

DOES work on Safari

Version 9.1 (11601.5.17.1)

Expected Results:

notwaldorf commented 7 years ago

Can you provide a screenshot? Here's the iron-icons demo on my Chrome 51.0.2704.103 on 10.11.6:

screen shot 2016-08-02 at 5 30 30 pm

It's super zoomed out to get as many of them in, but they're all there.

erinhong17 commented 7 years ago

@notwaldorf I can see them all on the demo, I should have clarified. I downloaded the app demo template, started to play around with the icons until I noticed that some showed up and some dont! So I noticed that "icons:menu" works perfectly fine.. and if I swap out that icon for another one such as "icons:gesture" or "icons:check" or many others within the "Icons" library, they just don't show up.

My current Chrome version: Version 51.0.2704.106 (64-bit) on 10.11.4

As mentioned earlier, detectable in the Chrome inspect tool:

Image below is where the search icon is not there but the filter shows up! This happens in my own custom components as well as the demo project that is usable from the polymer-project website.

image

notwaldorf commented 7 years ago

I think there might be something wrong in your demo. Here's a jsbin where the icons show up fine in Chrome: http://jsbin.com/sokaza/edit?html,output

screen shot 2016-08-03 at 10 37 02 am

If you can provide a similar JSBin with your repro steps that would be great

Moggers commented 7 years ago

I ran into the same issue (can render arrow-back, menu, chevron-right, and close, but no others). The app-drawer demo imports my-icons.html which is a locally defined very slim subset of iron-icons. If it comes after the import iron-icons I believe it overwrites the config? So if you try to go from the out of the box template and import iron-icons you seem to end up just having it overwritten by my-icons.html.

asoseil commented 7 years ago

With the starter project "my-icons.html" overwrite the import of iron-icons

SecareLupus commented 7 years ago

I was having the same problem, and can confirm that swapping the order of the imports of my-icons and iron-icons fixed the issue for me. I was using the polymer init project as a base, presumably OP is too?

tony19 commented 7 years ago

The iconset name conflict is fixed in https://github.com/PolymerElements/polymer-starter-kit/issues/934

arthurevans commented 7 years ago

As @tony19 said, this appears to be a PSK issue that's now fixed in PSK. Closing here.