Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

Firetray broke on firefox 35 on KDE Plasma 5 #157

Closed
anexation opened this issue Jan 14, 2015 · 15 comments
Closed

Firetray broke on firefox 35 on KDE Plasma 5 #157

anexation opened this issue Jan 14, 2015 · 15 comments

Comments

@anexation
Copy link

Firetray broke on firefox 35 on KDE Plasma 5

@foudfou
Copy link
Owner

foudfou commented Jan 14, 2015

Hi thanks for reporting. Do you know where I can find a virtual image (ex. VirtualBox) for testing ?

@Karmaki
Copy link

Karmaki commented Jan 15, 2015

I confirm that Firetray is broken for the last Firefox update (35.0) on Xubuntu 14.04.1.
It is working for Thunderbird (31.3.0) though.

@anexation
Copy link
Author

foudou, you can use my iso just need to update firefox, "sudo pacman -Sy firefox-kde"

https://sourceforge.net/projects/manjarotest/files/0.9.0/kf5-plasma-dev/0111-OLD/

im working on packages now for this release to be based off all stable Plasma 5 packages so marked it as old.

@jensberke
Copy link

Don't know if that's relevant for this issue, but I get the following output when starting Firefox from the shell (FireTray 0.5.4, Xubuntu 14.10, Firefox 35.0):

JavaScript error: resource://firetray/linux/FiretrayAppIndicator.jsm, line 71: TypeError: redeclaration of variable pref

@jensberke
Copy link

Some additional infos to my previous post: I don't use KDE, but XFCE. FireTray completely stopped working after updating Firefox from 34 to 35. No tray icon at all, neither default nor custom. Firefox quits when clicking the window close button.

@jensberke
Copy link

...and some more shell output when I quit Firefox:
18:55:30.419 ERROR firetray.FiretrayWindow unknown window while lookup
18:55:30.420 ERROR firetray.Window can't unregister unknown window null

@peterruhnau
Copy link

On xubuntu 14.04.1: Setting extensions.firetray.with_appindicator to false (in about:config) fixes this.

@Karmaki
Copy link

Karmaki commented Jan 16, 2015

Thanks @peterruhnau: it worked for me too.

@cyberbeat
Copy link

The code for using appindicator did not work for me on plasma 5, I looked at the code, and understood, why:

a shell command "plasma-desktop -v" is executed to get plasma version.

problem: this works only for kde-4, in plasma-5 session, there is no command "plasma-desktop" anymore, but

plasmashell -v

which results in

plasmashell 5.2.0

for me. Please adapt your script.

What also does not work (could be another bug report):

  • toggle thunderbird window via left-mouse button.
  • toggling window via middle-click works, but the window is not focussed, it only appears in the taskbar when toggle is on.

@foudfou
Copy link
Owner

foudfou commented Feb 28, 2015

@cyberbeat thx for investigating.
Do you know any reliable command that would work across all plasma versions (kwin, kstart, kdepasswd, kde-open, ...) ?
The issues you mentioned are known (#130) and might be addressed by #143.

@cyberbeat
Copy link

I think the current method is ok, plasma-desktop for kde-4 and plasmashell for kde-5. perhaps usig some dbus request also could help, would that be an option?

a few links to the topic:

https://launchpad.net/sni-qt

http://blog.martin-graesslin.com/blog/2014/06/where-are-my-systray-icons/

http://agateau.com/2011/statusnotifieritem-for-qt-applications/

@foudfou
Copy link
Owner

foudfou commented Mar 1, 2015

@cyberbeat I don't have any KDE5 environment at hand. Couldn't find how to get plasma's version with dbus. Meanwhile, can you tell me if kde-open is available in KDE5 ?

@foudfou
Copy link
Owner

foudfou commented Mar 1, 2015

@cyberbeat got it: KDE_SESSION_VERSION is sufficient (b792bb8).
Can you build and confirm ?

@cyberbeat
Copy link

for me the environment variable is set:

echo $KDE_SESSION_VERSION
5

seems to be a good and more reliable way to test the desktop-session than existance/output of programs.

I am on opensuse with plasma-5 with a mix of plasma-5 desktop and kde-4 applications.

And there are
kde-open (kde4) and kde-open5 (kde5)

@foudfou
Copy link
Owner

foudfou commented Oct 13, 2015

See also #130. I'll close this one as it seems resolved. Feel free to reopen.

@foudfou foudfou closed this as completed Oct 13, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants