Bug 350920 - please revert removal of "minimize to tray"
Summary: please revert removal of "minimize to tray"
Status: RESOLVED FIXED
Alias: None
Product: krusader
Classification: Applications
Component: general (other bugs)
Version First Reported In: Git
Platform: Gentoo Packages Linux
: NOR wishlist
Target Milestone: ---
Assignee: Alex Bikadorov
URL:
Keywords:
: 371766 371774 (view as bug list)
Depends on:
Blocks:
 
Reported: 2015-08-03 14:32 UTC by Marc Schiffbauer
Modified: 2017-05-04 12:33 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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.