Bug 350920

Summary:	please revert removal of "minimize to tray"
Product:	[Applications] krusader	Reporter:	Marc Schiffbauer <mschiff>
Component:	general	Assignee:	Alex Bikadorov <alex.bikadorov>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	alex.bikadorov, davide, KDE, piwsko, stephena, tietze.heiko
Priority:	NOR
Version First Reported In:	Git
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Marc Schiffbauer 2015-08-03 14:32:36 UTC

SInce some time I am using git master of krusader and at some point the "minimize to system tray" feature seems to have been removed.

I just wanted to kindly ask you to add this feature again as I used to use this a lot and I am missing it ;-)

TIA

Reproducible: Always

Comment 1 Davide Gianforte 2015-08-05 10:39:04 UTC

The "minimize to tray" feature was removed with the Qt5/KF5 porting because it wasn't a trivial one. Atm, there are no plan to add it; it will be done in the future.

http://quickgit.kde.org/?p=krusader.git&a=commit&h=8a05cacaf58fba11d6d96b934ce2588a43dc3019

Comment 2 Alex Bikadorov 2016-07-19 18:17:35 UTC

I was already working on this but there is a bug in KF5 that prevents me from commiting: https://bugs.kde.org/show_bug.cgi?id=365105

As soon as it is fixed I'll upload for testing.

Comment 3 Alex Bikadorov 2016-10-27 21:07:49 UTC

*** Bug 371766 has been marked as a duplicate of this bug. ***

Comment 4 Toni Asensi Esteve 2016-10-28 08:21:05 UTC

*** Bug 371774 has been marked as a duplicate of this bug. ***

Comment 5 Heiko Tietze 2016-10-28 08:26:58 UTC

Minimize to tray works nicely for Konversation, in case you need inspiration. Thanks for the most awesome file manager!

Comment 6 Alex Bikadorov 2016-10-28 16:32:33 UTC

Konversation is probably recreating the icon when enabled/disabled, we have to do the same.

Comment 7 Alex Bikadorov 2016-10-28 17:18:50 UTC

Its in master now. Sorry, should have done this a lot earlier.

Comment 8 Stephen Anthony 2016-10-29 15:31:50 UTC

Along with minimize to tray, clicking the 'X' (close) button on the main window used to do the same thing (ie, minimize to tray).  Now it closes the app.  Is this a related bug, or should a new bug be opened??

Comment 9 Alex Bikadorov 2016-11-10 13:55:25 UTC

The feature was called "minimize to tray", so I implemented it exactly that way. Maybe change it (name and behaviour) to "close to tray"?

I would like to avoid two different settings here for almost the same thing.
If REALLY needed please open another report.

Comment 10 Stephen Anthony 2016-11-10 20:30:10 UTC

I guess it's not really needed.  You're right that the description is exactly what it does now.  But it could also be considered a regression, since the 4.x version worked with the close button too.  I'm fine with it as-is.

Comment 11 Heiko Tietze 2016-12-29 07:35:50 UTC

Feature is still missing in 2.5.0 (Plasma 5.8.5; Arch Linux).

Comment 12 Marc Schiffbauer 2016-12-29 12:12:02 UTC

Thanks for reimplementing this feature!

I'd prefer previous behavior because it was my way to work with krusader: Klick the Icon in the tray when I wanted to use it and close krusader when done with managing files (no matter if I used, alt-f4 or close button). Now I have to be careful to not use the close button but minimize, so I really liked the way it was before a bit more.

Comment 13 Alex Bikadorov 2017-05-04 12:33:31 UTC

Ok, please switch to Bug 379459.