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

Thunderbird version #49

Closed
nclm opened this issue Jun 16, 2019 · 14 comments · Fixed by #126
Closed

Thunderbird version #49

nclm opened this issue Jun 16, 2019 · 14 comments · Fixed by #126
Assignees
Labels
blocked Cannot be resolved, because we need to wait for something else to be resolved first. enhancement New feature or request

Comments

@nclm
Copy link

nclm commented Jun 16, 2019

Background

Since Firefox and Thunderbird extensions are probably made the same way and since Thunderbird currently has a antiquated emoticon picker, it would be really nice to have a version of Awesome Emoji Picker for Thunderbird.

Proposed solution

Insert the emoji picking window in the editing bar of the composing window, possibly replacing the current emoticon picker:

Mockup

Distribute it on https://addons.thunderbird.net

@nclm nclm added the enhancement New feature or request label Jun 16, 2019
@rugk
Copy link
Owner

rugk commented Jun 16, 2019

Yeah, I need to look into it (whether Thunderbird supports WebExtensions)…

@tdulcet
Copy link
Contributor

tdulcet commented Oct 30, 2019

Thunderbird does support WebExtensions: https://thunderbird-webextensions.readthedocs.io/en/latest/

@rugk
Copy link
Owner

rugk commented Oct 31, 2019

Thanks for the pointer. That's great to hear, but it needs to be looked into whether it can be made compatible/needs adjustments.

@tdulcet
Copy link
Contributor

tdulcet commented May 28, 2020

I just filed a bunch of bugs with Thunderbird that would need to be fixed to get the Awesome Emoji Picker fully working:

this to fix #45, #100 and #106:

and this would make debugging easier:

Even with these bugs/missing features, I was able to get the emoji picker and autocorrect (#93) to work in Thunderbird's compose window with very minimal changes.

@rugk
Copy link
Owner

rugk commented May 31, 2020

Wow thanks. Good work! 😃

@rugk
Copy link
Owner

rugk commented Jul 13, 2020

Issue #1641577 has a workaround posted, does this fix the problem, @tdulcet?
If so, I've converted your list above to a checklist, so let's tick the issues that are solved…

@tdulcet
Copy link
Contributor

tdulcet commented Jul 14, 2020

Issue #1641577 has a workaround posted, does this fix the problem

No. With the workaround, the options page will load, but changing the options has no effect.

If so, I've converted your list above to a checklist, so let's tick the issues that are solved…

OK, I marked the bugs that have been fixed. At a minimum, we only need 1641577 and 1641573 to be fixed to get this working. The rest either are missing features (1641575 and 1641576) or will eliminate Thunderbird specific code that is currently required (1641580).

As soon as #93 is merged, I will create a PR for this, so that people can test it out. Note that Thunderbird 78, the next major version, is currently scheduled for release on July 17. Thunderbird follows the Firefox ESR release schedule, so there is only a major release every year.

@rugk rugk added the hacktoberfest Hacktoberfest 2020, https://hacktoberfest.digitalocean.com label Sep 29, 2020
@rugk rugk added blocked Cannot be resolved, because we need to wait for something else to be resolved first. and removed hacktoberfest Hacktoberfest 2020, https://hacktoberfest.digitalocean.com help wanted Extra attention is needed labels Nov 17, 2020
@rugk rugk removed the good first issue Good for newcomers label Nov 17, 2020
@tdulcet
Copy link
Contributor

tdulcet commented Feb 24, 2021

Bug 1641577 has finally been fixed, which was the last blocking bug (see #49 (comment))! Unfortunately, they will not uplift the fix to the current Thunderbird 78 release, but it is now available in Thunderbird Beta and will be available in the next major version, Thunderbird 91, which is currently scheduled for release on August 10.

@tdulcet
Copy link
Contributor

tdulcet commented Aug 20, 2021

@rugk and I published an experimental version of the Awesome Emoji Picker for Thunderbird: https://addons.thunderbird.net/en-US/thunderbird/addon/awesome-emoji-picker/. It requires Thunderbird 91, which was just released last week. Feedback is welcome.

It includes the emoji autocorrect feature from #93, as well as a context menu item to fix #45. Note that the emoji picker and emoji autocorrect feature currently only work in the message body of the compose window.

@dvergeylen
Copy link

I've tested with Thunderbird 91.1.0, under Debian Testing Wayland.

I'm experiencing a weird flickering of the menu when clicking on it. Does not appear 100% of the time, I'd say roughly 90%.

output.mp4

That might not be related to the addon at all (which I extensively use with Firefox 🙏).

@rugk
Copy link
Owner

rugk commented Sep 10, 2021

Oh gosh, this looks like some (P)ReactJS rendering issue or whatever… Does it matter, where on the screen you place the window before opening the popup?

Because in your video/GIF (thanks BTW for that, this is very helpful!) I can see it that the height you have available is not much, the popup just fits on the screen.
So maybe move the window up a bit and then the popup has enough space to be shown? (I'm just speculating here…)

@dvergeylen
Copy link

Does it matter, where on the screen you place the window before opening the popup?

It doesn't, but that might be a hint: whatever the window position and size, the menu tries to get as much height as it can (resulting in too big emojis, proportionally with the rest of the interface) until it collides with the bottom of the screen. I guess it can't find a correct "non colliding" position and starts flickering by repositioning itself again and again... 🤔

I don't experience the issue in Firefox 91.1.0esr.

Hope this helps!

@rugk
Copy link
Owner

rugk commented Sep 17, 2021

Given the same engine etc. should be underlying Firefox and Thunderbird, maybe we should consider opening a Thunderbird bug for that? Or do you have any other ideas, @tdulcet?

@tdulcet
Copy link
Contributor

tdulcet commented Sep 18, 2021

Given the same engine etc. should be underlying Firefox and Thunderbird, maybe we should consider opening a Thunderbird bug for that? Or do you have any other ideas, @tdulcet?

No, I definitely agree that someone should create a bug on BMO with the above video, as this is most likely a Thunderbird/Gecko issue since the whole popup flickers/flashes.

@dvergeylen - Thanks for reporting this! I noticed in your video that there is another format toolbar icon for a different add-on. Do you have any issues with the popup for that? I agree that the popup in your video is a little bigger than I would expect:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Cannot be resolved, because we need to wait for something else to be resolved first. enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants