johannesjo / super-productivity

Super Productivity is an advanced todo list app with integrated Timeboxing and time tracking capabilities. It also comes with integrations for Jira, Gitlab, GitHub and Open Project.
http://super-productivity.com
MIT License
11.66k stars 903 forks source link

Invalid characters in alert on startup #2619

Open dahaupt opened 1 year ago

dahaupt commented 1 year ago

image

Your Environment

Can you reproduce this reliably?

Yes, on every startup.

github-actions[bot] commented 1 year ago

Thank you very much for opening up this issue! I am currently a bit overwhelmed by the many requests that arrive each week, so please forgive me, if I fail to respond personally. I am still very likely to at least skim read your request and I'll probably try to fix all (real) bugs if possible and I will likely review every single PR being made (please, give me a heads up if you intent to do so) and I will try to work on popular requests (please upvote via thumbs up on the original issue) whenever possible, but trying to respond to every single issue over the last years has been kind of draining and I need to adjust my approach for this project to remain fun for me and to make any progress with actually coding new stuff. Thanks for your understanding!

github-actions[bot] commented 1 year ago

Hello there dahaupt! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀

For more open ended discussions and/or specific questions, please visit the discussions page. 💖

johannesjo commented 1 year ago

I am not able to reproduce this. Maybe anybody knows how to?

dahaupt commented 1 year ago

I was able to get rid of the alert message by manually editing my JSON data.

  1. Changed globalConfig.sync.syncProvider to null (previously set to GoogleDrive)
  2. Removed globalConfig.sync.googleDriveSync

The alert was working fine since upgrading Ubuntu to 23.04.

johannesjo commented 1 year ago

Glad that it works for you! Thanks for reporting back. Likely the problem will reappear with other native dialogs though I'd assume :/

github-actions[bot] commented 1 year ago

This issue has not received any updates in 90 days. Please comment, if this still relevant!

KatrinIhler commented 9 months ago

I've been seeing invalid characters as well for a while now, in my case if the local data conflicts with the remote. Luckily, by now I know which option does what, even without letters. ;)

I'm running a Ubuntu 23.10 with Gnome 45 and Wayland, currently on SP 7.16.0 installed with snap. If I remember correctly, my Fedora is not experiencing those issues.

Edit: Also happened when reproducing #2940:

image

See that issue for further information on my system.

KatrinIhler commented 9 months ago

Would be grateful if we could reopen this. :)

github-actions[bot] commented 5 months ago

This issue has not received any updates in 90 days. Please comment, if this still relevant!

KatrinIhler commented 5 months ago

Just saw this issue last week :sweat_smile: Only occurs on my Ubuntu, not on my Fedora. If you need more information, I will gladly provide it.

johannesjo commented 5 months ago

Does executing

sudo rm /var/cache/fontconfig/*
rm ~/.cache/fontconfig/*
fc-cache -r

help?

(https://github.com/electron-userland/electron-builder/issues/5217)

Also the logs from executing superproductivity from the command line would be helpful!

KatrinIhler commented 4 months ago

That unfortunately didn't help. Will provide logs when I can.

johannesjo commented 4 months ago

Could you maybe do another test for me? If you open up the console (Ctrl+Shift+i) and then type alert("something") and then press enter, how will the characters appear inside the dialog that opens? And what happens if you repeat the same steps for confirm("something")?

dahaupt commented 4 months ago

I have not noticed this issue again after upgrading Ubuntu to 24.04 LTS.

mvalim commented 3 months ago

I can confirm that with alert('ok') and confirm('ok') the alert dialog show wrong characters.

I'm using Manjaro Linux, and installed the app via snapcraft, found this link searching for related problems with electron https://github.com/electron-userland/electron-builder/issues/5217 don't know if the comments in there can be usefull

image image

mvalim commented 3 months ago

Maybe the easiest way to solve this problem is by using an alternative alerts library. I've noticed in your package.json that you import the angular material library, maybe using the angular material dialog component would be a good alternative https://material.angular.io/components/dialog/examples.

I have experience with angular and if you want, when I have some spare time, I can make a pull request changing the places where confirmation and alerts are shown to use the material dialog component.

johannesjo commented 3 months ago

@mvalim the behavior of alert and confirm is different since they (1) pause the execution of all code (which is kind of what we want for some of these dialogs and (2) they can appear beyond the bounds of the application. So simply replacing them won't work :)

mvalim commented 3 months ago

After using the app a bit more, I noticed that just replacing would not work in some cases. I hope you find a solution soon, I will fork the project and will do some research and tests, if I find something useful that could help solve the problem I'll tell you

johannesjo commented 3 months ago

Thank @mvalim ! Much appreciated! I hope you find something!

justAnotherAnotherUser commented 3 days ago

Does executing

sudo rm /var/cache/fontconfig/*
rm ~/.cache/fontconfig/*
fc-cache -r

help?

(electron-userland/electron-builder#5217)

Also the logs from executing superproductivity from the command line would be helpful!

i am here to report that executing these commands seemed to have worked for me