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

Messenger blocked when using Facebook's instantaneous messages #37

Closed
Prometheos2 opened this issue May 20, 2019 · 10 comments
Closed

Messenger blocked when using Facebook's instantaneous messages #37

Prometheos2 opened this issue May 20, 2019 · 10 comments
Labels
bug Something isn't working site-issue This issue is about a specific incompatible website.

Comments

@Prometheos2
Copy link
Contributor

Trying to insert an emoji in the instantaneous messages results in the blockage of the Messenger in the current page.

It only does that bug when no character are into the dialog box beforehand.

I assume it's on Fb's side, like an non-handled error or an anti-bot/flood/spam measure; so it might be hard to fix that.
I'm not sure we can access the message field to determine whether or not the field is empty through the add-on.

@rugk rugk added bug Something isn't working site-issue This issue is about a specific incompatible website. labels May 20, 2019
@rugk
Copy link
Owner

rugk commented May 20, 2019

Thanks for the report. Yes, it's likely specific to that site.
Maybe they don't even have a real input field when there is no character entered yet.

Anyway, there is indeed hardly anything I can do. But good that it's tracked. Just as the other site issues.

@Prometheos2
Copy link
Contributor Author

Would it be possible to detect when the cursor is in the Messenger "box" and block the emoji insertion?

Or, if that is the problem, detecting a loss of focus from the Messenger box when the add-on dialog box opens?

@rugk rugk changed the title Bug with Facebook's instantaneous messages Messenger blocked when using Facebook's instantaneous messages Sep 19, 2019
@rugk
Copy link
Owner

rugk commented Sep 19, 2019

Anything is possible, but this sounds like a very specific cumbersome workaround (which does not even actually fix the problem of insertion, just prevent bad side effects). Though it may be worth it for such a big site.

I am not completly oppoosed to it, but it has to be well-thought. E.g. make it such generic that one can later add similar/other workarounds for other sites.
That said, I'd appreciate concrete development ideas or even pull requests on that feature. 😃

Also, BTW, do you have a screenshot on how that "blockage" looks like?

@Prometheos2
Copy link
Contributor Author

Prometheos2 commented Sep 19, 2019

"That said, I'd appreciate concrete development ideas or even pull requests on that feature. 😃"
Oh no 😥

Well, it's complicated, since it's more about not being able to reopen some dialog rather than something more "screenshot-able"

I will send you the JS console logs instead

Is "browser-polyfill.min.js" your thing?

@Prometheos2
Copy link
Contributor Author

@rugk
Copy link
Owner

rugk commented Sep 20, 2019

Hmm, okay, I don't see any error related to this bug/add-on or so.

Is "browser-polyfill.min.js" your thing?

No, that has to be a different add-on. You can look up the add-on ID in about:debugging.

@Prometheos2
Copy link
Contributor Author

Prometheos2 commented Sep 20, 2019

That's weird, then.
It would truly be on Fb's side?


No, that has to be a different add-on.

I've checked, and yes, it's another extension.
It probably attempts to access the (now-crashed) message box.

QA is hard ^^

@Prometheos2
Copy link
Contributor Author

Addendum: It also concerns the Messenger page itself, as it blocks the text area until refreshment (cf. screenshot & JS console extract), but the emoji is inserted and can be seen on refresh.


Screenshot

image

JS Console extracts

ErrorUtils caught an error: "React reported an error from NotFoundError: Node was not found". Subsequent errors won't be logged; see https://fburl.com/debugjs. 8gEnQK1P6Dl.js:51:246 Source map error: Error: NetworkError when attempting to fetch resource. Resource URL: moz-extension://660ce86a-a063-4dab-8aea-184db93e38bb/browser-polyfill.min.js Source Map URL: browser-polyfill.min.js.map

extended JS console extract.txt

@Prometheos2
Copy link
Contributor Author

Prometheos2 commented Apr 7, 2020

With the new version of the website, the issue seems to be solved.

However, like in #38, inserting two (or more) emojis results in very awkward events.
That's hard to pinpoint what it gives.

Edit: it somehow broke again with the awkward insertion.
But the core problem seems to be solved; trying to insert an emoji without focusing on the Messenger's tab leads to a copy of the smiley, not an insert.

@rugk
Copy link
Owner

rugk commented Apr 7, 2020

Okay, so if this problem is fixed, that's good, please open a new issue.

If the other issue is really the same as #38, we could expand #38 to also track the Messenger problem, but this one here seems fixed so I'm closing this.

@rugk rugk closed this as completed Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working site-issue This issue is about a specific incompatible website.
Projects
None yet
Development

No branches or pull requests

2 participants