Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editor Icon Missing #17

Closed
nilshoerrmann opened this issue Aug 1, 2014 · 17 comments
Closed

Editor Icon Missing #17

nilshoerrmann opened this issue Aug 1, 2014 · 17 comments
Milestone

Comments

@nilshoerrmann
Copy link
Contributor

I don't see the icon in Chrome: Version 36.0.1985.125 on OSX Mavericks
I see the icon in: Firefox v30
I don't see the icon in: Safari 7.0.5 (9537.77.4)

See #16.

Reference how it should look like:

bildschirmfoto 2014-07-31 um 09 12 35

@nilshoerrmann
Copy link
Contributor Author

Strange thing: the icons display correctly here using the same browsers on the same OS.

@andrewminton
Copy link

Hi :)

Anything else you can thing it may be?

@nilshoerrmann
Copy link
Contributor Author

Maybe something's wrong with the SVG file.

@andrewminton
Copy link

Opens fine here.. Mac and PC.

@nilshoerrmann
Copy link
Contributor Author

Does the inspector highlight any issues?
Something related to background positioning or another rule that overrides the background property?

@andrewminton
Copy link

screen shot 2014-08-01 at 11 27 15

This is the only thing I see as may be causing an issue? styles overriding it?

@nilshoerrmann
Copy link
Contributor Author

Which rule is overriding the background-position?

@andrewminton
Copy link

media="screen" .field[data-editor^='aui-editor'] .multi .item { padding-left: 1.3rem; background-position: 1rem center; }

@andrewminton
Copy link

Weirdly though, disabling these overriding rules has no effect :(

Also don't understand why Safari is experiencing the same issue too.

@nilshoerrmann
Copy link
Contributor Author

@andrewminton
Copy link

That's what I was just lookng at.. mime type of text/xml.. I'll add that HTACCESS and check now.

@andrewminton
Copy link

Bingo! that did it.

@nilshoerrmann
Copy link
Contributor Author

Great! So we should add a note about that problem to the readme (#15).

@andrewminton
Copy link

Yep! good idea.

It didn't make a difference in Safari at the moment.. will do a cache refresh and test some more.. but Chrome is fine.

@andrewminton
Copy link

Safari fine now too after a cache refresh.

@andrewminton
Copy link

Case closed?

@nilshoerrmann
Copy link
Contributor Author

Kind of, yes. Thanks for debugging this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants