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

Feature: Add "Don't show again flag" for systray warnings #4691

Closed
GIJack opened this issue Oct 23, 2020 · 10 comments · Fixed by #4874
Closed

Feature: Add "Don't show again flag" for systray warnings #4691

GIJack opened this issue Oct 23, 2020 · 10 comments · Fixed by #4874

Comments

@GIJack
Copy link

GIJack commented Oct 23, 2020

Description

For desktops, such as GNOME that don't have a systray, bisq every couple of min has a pop up warning that it can't find one. Great. Please add another flag to suppress this

Version

Steps to reproduce

Expected behaviour

Actual behaviour

Screenshots

Device or machine

Arch linux

Linux BOXNAME 5.8.16.a-1-hardened #1 SMP PREEMPT Sat, 17 Oct 2020 13:04:20 +0000 x86_64 GNU/Linux
pacman -Q bisq
bisq 1.4.2-1

Additional info

@boring-cyborg
Copy link

boring-cyborg bot commented Oct 23, 2020

Thanks for opening your first issue here!

Be sure to follow the issue template. Your issue will be reviewed by a maintainer and labeled for further action.

@ghost
Copy link

ghost commented Oct 23, 2020

What OS distro & version are you running? Could you paste here a screenshot of the popup?

@GIJack
Copy link
Author

GIJack commented Oct 25, 2020

systray_error

@ghost
Copy link

ghost commented Oct 25, 2020

Thanks for the screenshot and for informing that OS is Arch linux. Looks like a generic exception handler in Bisq popped up the error dialog. The message The system tray is not supported on the current platform is coming from Java AWT. It would also be helpful to see the exception callstack that gets written to the log file upon this error happening, so as to know which line of code is triggering it.

Probably we have to find a dev with Arch linux willing to debug this.
It does not happen on Ubuntu 18.04 (GNOME).

@ghost
Copy link

ghost commented Nov 19, 2020

I installed Arch linux and Bisq 1.4.2; it runs fine and did not produce for me the error popups you experienced. Perhaps a configuration issue. I'm open to suggestions on how to reproduce this issue on Arch; otherwise its a one-off issue that cannot be reproduced.

@GIJack
Copy link
Author

GIJack commented Nov 19, 2020

Use GNOME, or another DE that doesn't have a native systray

@ghost
Copy link

ghost commented Nov 20, 2020

Yes, using GNOME, details below. Bisq runs fine and did not produce for me the error popups you experienced.

image

@premek
Copy link

premek commented Dec 1, 2020

I have the same warning showing up every time my laptop locks the screen after inactivity.
I'm on Gnome 3.38.0

The exception in the log is as follows.

Dec-01 01:24:40.490 [AWT-EventQueue-0] ERROR b.c.s.CommonSetup: Uncaught Exception from thread AWT-EventQueue-0 
Dec-01 01:24:40.490 [AWT-EventQueue-0] ERROR b.c.s.CommonSetup: throwableMessage= The system tray is not supported on the current platform. 
Dec-01 01:24:40.490 [AWT-EventQueue-0] ERROR b.c.s.CommonSetup: throwableClass= class java.lang.UnsupportedOperationException 
Dec-01 01:24:40.490 [AWT-EventQueue-0] ERROR b.c.s.CommonSetup: Stack trace:
java.lang.UnsupportedOperationException: The system tray is not supported on the current platform.
	at java.desktop/java.awt.SystemTray.getSystemTray(SystemTray.java:188)
	at java.desktop/sun.awt.X11.XTrayIconPeer$4$1.run(XTrayIconPeer.java:222)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
	at java.desktop/java.awt.EventQueue.access$600(EventQueue.java:97)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:87)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:97)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:745)
	at java.desktop/java.awt.EventQueue$5.run(EventQueue.java:743)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:87)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:742)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Note the empty space in the top right corner where the icon should probably be, i can rightclick there but the icon is in the top left corner (not active).

Screenshot from 2020-12-01 09-56-46

@ghost
Copy link

ghost commented Dec 2, 2020

@premek Thanks for the useful info! It seems we are running the same OS but as you can see the bar at the top of the screen seems a little different. On mine there is no systray icon, nor errors. Hmm. I built my Bisq from source, is yours from the AUR? If so would you tell me steps to install Bisq from pacman/AUR, I tried but could not get it to work.

Screenshot from 2020-12-02 15-13-26

Anyway, based on the stack trace you provided I can make a PR to prevent the popup. It will be linked here shortly.

@premek
Copy link

premek commented Dec 2, 2020

I'm on debian and using the downloaded .deb file. I have some gnome extensions installed which moves things around in the top bar.
It would be nice not to have the popup every now and then but for me the icon is still displaced but that may be another issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants