Skip to content
This repository has been archived by the owner on Dec 10, 2018. It is now read-only.

New tray popup implementation #124

Merged
merged 1 commit into from
Dec 21, 2015
Merged

New tray popup implementation #124

merged 1 commit into from
Dec 21, 2015

Conversation

tresf
Copy link
Contributor

@tresf tresf commented Dec 21, 2015

  • Adds a JXTrayIcon class from sunlabs so that the popup menu disappears without the need for a separate timeout.
  • Adds a new ModernTrayIcon class for Linux-only (also compatible with Windows) which fixes an incompatibility with the JXTrayIcon implementation on Ubuntu and possibly others. Unfortunately, ModernTrayIcon doesn't support Mac, so we're stuck with two separate work-arounds for now.

This needs to be ported to the 2.0 branch.

@tresf
Copy link
Contributor Author

tresf commented Dec 21, 2015

Tested on Mac - PASS
Tested on Windows - PASS
Tested on Ubuntu - PASS

@Vzor- @klabarge please test on Fedora and any other Linux flavors you have laying around.

tresf added a commit that referenced this pull request Dec 21, 2015
New tray popup implementation
@tresf tresf merged commit 36f331b into qzind:1.9 Dec 21, 2015
@tresf tresf deleted the trayfix branch December 21, 2015 21:42
@klabarge
Copy link
Member

I tested on Linux Mint Mate 64-bit and this seems to be working great.

By default, notifications are not shown.

@tresf
Copy link
Contributor Author

tresf commented Dec 22, 2015

@klabarge Thanks, Fedora 22 x64 passed as well after applying 4cbe61d.

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

Successfully merging this pull request may close these issues.

None yet

2 participants