-
Notifications
You must be signed in to change notification settings - Fork 20
Report issue button functionality #203
Comments
Are you saying the "Report Bug" button does not do anything at all? It's working fine for me so I suspect it's got something to do with your linux distribution not supporting the command that is called to open the url in the default browser. What I will do as a next step is bump the version of the Rust crate that's being used and if we're lucky it has been fixed upstream already. Does opening the logs folder work fine for you (Settings > Open Logs)? There we're using a different Rust crate to open the folder and I had reports that it's not working on Kubuntu. So it would be interesting to know whether that works fine on your Linux distribution. |
And I think regarding the popups not displaying properly I think I cannot do much unfortunately since this is not something the Launcher UI actively controls. It's handled either by the operating system or the webview. |
It changes the cursor (as if there was a button that you can click) but that is all.
It looks like it is trying. (see video) It appears to loose focus on the launcher as the bottom menu bar appeared so something is happening but not successfully. 2023-10-06.09-38-26.mp4https://1drv.ms/u/s!AuXD18607r4Z11kntX2yVYKAVU4E?e=rxV4wJ Let me know if it would be better to just delete the file and try both again so that there is less noise to sort through. |
I did this anyway as the file takes too long to load. It looks like there is nothing that indicates an error but I hope it helps. |
Thanks for the logfiles! As I feared, it did not even log an error in the log file so it's likely related to this issue. And since both opening the logs and clicking the link doesn't work means that both available Rust crates ( |
I have tried with the latest version of the launcher and nothing works. There are some other holochain apps that allow me to open links so I would say there is another way to do it. I have just installed the .dmg and I can open both the logs and the issues. Also I tried to run the appimage using sudo and it did not recognise the previous installation. Any ideas as to why? Also it did not help in any way that I could tell.
I looked at the comments and there were some other tools were installed.
|
It's not really a Holochain problem but a tauri problem. Acorn for example uses Electron instead of tauri. I already try to catch clicks on links and the open them explicitly with the browser but apparently that's not working on your operating system :/ I'm not sure how to fix this if all available Rust crates seem to fail...I will have to think about it.
It works fine when I run the AppImage on my Ubuntu 22.04 so I think it's specific to your Linux distribution
That's fascinating. I didn't know about that but I also see that when I run it on my Ubuntu 22.04. Maybe sudo stores things in different file locations - I need to look into it. Also, interstingly the menu items look differently when run with sudo so it seems to use a different version of webkit
Thanks! So if you have |
By the way, it's probably not a good idea to run AppImage files with |
Yes. The idea behind it was that perhaps it was a permissions issue but it was not. Just @ me if you need me to test anything. (I will probably see any comments anyway) |
Describe the bug
I would assume that the Report Bug on GitHub button would direct you to the appropriate page however it does not do this.
Also there is a UI issue shown in the video provided where the popups are not always in the correct location.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The https://github.com/holochain/launcher/issues/new page should open
Logs
N/A
Screenshots
If applicable, add screenshots to help explain your problem.
The following shows the UI issue where the Settings and Report Bug on GitHub (https://github.com/holochain/launcher/issues/new) popup do not appear in the correct place and can be off the screen.
2023-09-28.09-57-09.mp4
Desktop (please complete the following information):
Holochain Launcher v0.11.1 (Beta 0.2)
Additional context
N/A
The text was updated successfully, but these errors were encountered: