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

Icon / text will not change on toggle occasionally #6

Open
jokervsbatman opened this issue Nov 25, 2017 · 10 comments
Open

Icon / text will not change on toggle occasionally #6

jokervsbatman opened this issue Nov 25, 2017 · 10 comments
Assignees
Labels
bug needs info Waiting on more information for debugging

Comments

@jokervsbatman
Copy link

There a notable bug I’ve come across when you’ve set the add-on to block all images, it occasionally doesn’t block all the images and essentially requires a refresh, but overall out of all the image block add-ons I’ve tested, it’s the best one for Firefox Quantum thus far. I’ll keep you posted on any other possible bugs I encounter.

@ckuhl
Copy link
Owner

ckuhl commented Nov 27, 2017

Hmm, that's odd. Are there specific actions that cause this bug to happen? I am trying to replicate the bug but can't.

@ckuhl ckuhl added the bug label Nov 27, 2017
@ckuhl ckuhl self-assigned this Nov 27, 2017
@jokervsbatman
Copy link
Author

jokervsbatman commented Nov 28, 2017

As far as I can tell, nah nothing out of the ordinary except so far it manifests on Facebook which materializes once in a blue moon. After an indirect trial and error, the effect isn't limited to Facebook.

@ohhai
Copy link
Contributor

ohhai commented Dec 12, 2017

Side question: I see that this addon's button doesn't switch permissions.default.image preference.
Is it OK?

@ckuhl
Copy link
Owner

ckuhl commented Dec 12, 2017

@ohhai that shouldn't be an issue -- it gets stored settings (see code here) at launch and sets the image from there.

@ckuhl
Copy link
Owner

ckuhl commented Dec 12, 2017

@jokervsbatman I'm still trying to replicate the behaviour but have been having a hard time. Are you still facing issues?

@jokervsbatman
Copy link
Author

Yeah, the issue still exists. What perplexes me is the cause.

@ohhai
Copy link
Contributor

ohhai commented Dec 18, 2017

I see something similar (maybe) on Firefox for Android. Once switched, can't switch back sometimes (at least the text in menu--I mean "images are allowed" or "images are blocked").
Can provide more detailed flow if necessary.

Off-topic request: is it possible to add the addon's button to browser's address field (only for Android)? I mean, some addons do that and it saves one extra click for every usage.

@ckuhl
Copy link
Owner

ckuhl commented Dec 18, 2017

That sounds interesting -- I'll have to look into it when I get some spare time. The challenge for me will be getting a device that has Firefox Android on it, as I have an iPhone right now.

I haven't forgotten, I've just been really busy the past month!

@ohhai
Copy link
Contributor

ohhai commented Dec 18, 2017

OK, I can try to debug it if you give me some highlights. How to monitor addon's state on a mobile device (variables, function calls, like usual debugger), remotely maybe, like that. There is something similar, but I never tried it.

@ckuhl ckuhl changed the title Block all images bug Icon / text will not change on toggle occasionally Mar 28, 2018
@ckuhl
Copy link
Owner

ckuhl commented Sep 1, 2018

Hi, I just realized that I never followed up on this after I made a few changes. Are you still having these issues?

@ckuhl ckuhl added the needs info Waiting on more information for debugging label Sep 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs info Waiting on more information for debugging
Projects
None yet
Development

No branches or pull requests

3 participants