SO-Close-Vote-Reviewers / UserScripts

Various user scripts that add features to the review queue or to the chat room
Other
57 stars 40 forks source link

Remove dependency on i.stack.imgur.com, so icon will show when that domain is blocked #124

Closed GrumpyCrouton closed 6 years ago

GrumpyCrouton commented 6 years ago

I thought that my Magic Editor was just not working (on Chrome), because the icon was not showing. I haven't had the icon in a few days, but I was just hovering around my toolbar and noticed I could still click the button, but the icon does not show.

I am on version 1.6.0.2.

tiny

makyen commented 6 years ago

I'm not able to easily duplicate this issue. I'm going to need some more information. Could you tell me:

I did see something like this when working on the previous version, but that was resolved with CSS/style changes. I have not seen this once that was resolved (in 1.6.0.0).

GrumpyCrouton commented 6 years ago
GrumpyCrouton commented 6 years ago

Any update? I still have this issue.

makyen commented 6 years ago

Sorry for not leaving a comment earlier.

I've attempted to duplicate the issue multiple times, but had no success in doing so. The icon is showing fine for me on Win 10x64 using Chrome/Tampermonkey, Firefox 52ESR/Greasemonkey 3.

What browser and version are you using?

GrumpyCrouton commented 6 years ago

I'm using Tampermonkey on Google Chrome version 65.0.3325.181 and version v1.6.0.2 of Magic Editor.

EDIT: I think I've figured out what the issue is by viewing the icon in Inspect Element. I've noticed that the image is hosted on i.stack.imgur.com which unfortunately has been blocked on the network I am currently using. Has this image always been hosted here?

EDIT 2: Confirmed, I fixed this by hosting the image on my own site: https://grumpycrouton.com/other/magic.png

makyen commented 6 years ago

Blocking that domain would certainly cause the problem you are seeing.

Yes, it has been that way for the last 2 years. Prior to that it was in the i.imgur.com domain for more than a year prior to that (i.e. from almost 4 years ago to 2 years ago).

makyen commented 6 years ago

FYI: I've converted the icon to SVG and included it within the script. This will be fixed in the next version I push.