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

Notifications and freeze on Linux: add a pause #7

Open
maxnikulin opened this issue Jun 10, 2022 · 2 comments
Open

Notifications and freeze on Linux: add a pause #7

maxnikulin opened this issue Jun 10, 2022 · 2 comments
Labels
lazy Delay because the repo owner have no free time and feel tired so want to be lazy.

Comments

@maxnikulin
Copy link

I am not a user of the add-on, but I have noticed a mention of a problem I was struggling with some time ago.

README.org

Notification may freeze the whole Firefox when using on Linux (Test on Firefox 92, Ubuntu & Manjaro), but such issue seems not occurs on Firefox for Windows.

You may add a pause for a fraction of second between clipboard.writeText and sending a notification. Likely the issue will be significantly alleviated.

Is it KDE in your case? If not, please add description of your environment to https://bugzilla.mozilla.org/1769930 I suspect, KDE is doing something is a wrong way as well, so the issue should be reported to them too. On the other hand I have enough bugs to report in my backlog.

A minor unrelated note that does not deserve a separate issue.

TODOs

  • Customizable keyboard shortcuts.

From manifest.json I expect it is already configurable through Firefox UI.

@kuanyui
Copy link
Owner

kuanyui commented Jun 19, 2022

  1. Sorry for late, thanks for providing this information!
  2. Yes, I'm using KDE.
  3. This solution may not be applied until someone make a PR, or I have free time and a good mood because I urgently need a vacation.
  4. And I found that the in-page notification is much more readable (e.g. I can make it larger and wider, set font to monospace...etc) than OS's one.

@kuanyui kuanyui added the lazy Delay because the repo owner have no free time and feel tired so want to be lazy. label Jun 19, 2022
@maxnikulin
Copy link
Author

Kuanyui, thank you for the information that you met the problem in KDE as well.

I am reluctant in respect to blind pull requests especially to projects that I do not use. I hope, some time later after a period of hard work you will be recovered and will add a single line with setTimeout at your spare time. Or maybe the Firefox bug will be fixed earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lazy Delay because the repo owner have no free time and feel tired so want to be lazy.
Projects
None yet
Development

No branches or pull requests

2 participants