-
Notifications
You must be signed in to change notification settings - Fork 874
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
Taskbar icon not working #2627
Comments
Thank you for submitting your first issue to this repository! A maintainer will be here shortly to triage and review.
Finally, remember to use https://discuss.ipfs.io if you just need general support. |
Try to change My guess is that this is another one of those bugs in the translation configuration.
Seems like the language system has been screwed up since version 0.30.0 |
Worked for me. |
Worked for me too! Thanks. |
Thanks for submitting this issue @Dominiquini and thanks for providing a resolution @Azulan, this sounds like the correct assumption. IPFS-Desktop started lazy-lazy loading app-context which makes the loads much more faster and reduces the amount of initial load tasks. However, this transition has not been as smooth as we'd hoped for, especially on the taskbar issues as testing taskbar in a cross-env fashion is hard. Taskbar was not covered with tests and hence it has seen a regression. There is an issue to tracking this. This issue is currently not reproducible on a mac, and maybe is limited to windows only. Based on the logs you provided, looks like
@SgtPooki I think there is a weird race condition (on windows) where the |
@whizzzkid to be clear, it was @yatounoneko who provided the work around and analysis. But thank you all for your hard work! |
Thanks for the correction @Azulan, thanks @yatounoneko I confused the names Thanks! |
thanks for the clarification on ipfs-desktop versions and where you're seeing problems, also, providing the logs is super helpful. I'll get on my windows machine today to start diving in, and possibly look into getting a remote windows VM that both @whizzzkid and I can look at when deploying & testing ipfs-desktop updates. |
I was not able to reproduce this error on my windows 10 machine. Right clicking the taskbar icon works fine no matter what language I have selected.
I'm guessing an error is occurring on some machines preventing the systray menu from being set up properly.. but it's hard to find a solution if i can't reproduce. |
note: I was able to attempt to right click the icon while ipfs-desktop was loading and the menu did not appear, but re-trying it after a few seconds (the status screen had JUST populated the bandwidth over time chart) and it worked. This is an artifact of things loading faster, and UI being available before ready. The solution for this is probably blocking the systray from even appearing until everything is ready. |
@SgtPooki can confirm, I am seeing a similar behaviour while using win11 in Parallels. It seems to work as expected. ![]() @Dominiquini @yatounoneko can you please re-check and provide a better way to reproduce this error? |
I don't know what language value in your This case may be just affcet to update from 0.28.0 or before version user? For my error config, you can apply this config to try ;)
|
I was seeing the same issue on Ubuntu, and setting |
@yatounoneko thanks for the extra info! it sounds like to reproduce, setting "language" to "en-US" should trigger the error.. retrying now |
Heyo! reproduced by manually setting language to "en-US" in thanks all.. working on fix. |
i18next has some auto-magical loading of translation files that kicks off loading multiple languages (e.g. 'en', 'en-US', 'en-UK', etc.) but we don't have all of those files for IPFS-Desktop. i18next.init will fire an error if any of those files fail to load, even if one succeeds. This fix ensures that `i18n.initDone` is set on context even if we see an error. fixes #2627
The fix is out: #2650. It's a super simple fix for an odd behavior from i18next: it shows an error when it cannot find a file for a language, even if it successfully resolves to a file for that language. |
Note that this is still not released due to some failures with windows build in CI |
Describe the bug
I'm unable to right-click on the IPFS icon on the taskbar to quit the application. No menu appears. I'm only able to open the GUI with left-click!
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Menu with options should appear!
Screenshots
Additional context
The icon also don't change color when I'm connected!
The version 0.30.1 works fine!
combined.log
error.log
Thanks
The text was updated successfully, but these errors were encountered: