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: Close button minimizes the app to tray when tray icon enabled #318

Open
marouamghar opened this issue Nov 28, 2024 · 11 comments
Open
Assignees
Labels
help wanted Extra attention is needed

Comments

@marouamghar
Copy link

When having tray icon enabled, I would like to have an option to close to tray. That way, I can still have the tray and closing the ui would just minimize it to tray, giving me more space on the apps bar.

@f-koehler
Copy link
Owner

f-koehler commented Nov 29, 2024

Thanks for the report @marouamghar!

May I ask what desktop you are using and how you installed KTailctl? On my machine with the KDE Plasma desktop the close button behaves exactly as your desired behavior, so I need some info to reproduce this...

@f-koehler f-koehler self-assigned this Nov 29, 2024
@f-koehler f-koehler added the question Further information is requested label Nov 29, 2024
@marouamghar
Copy link
Author

It's fedora, gnome. I could have sworn it was like that, but I started reusing that distro last week, and got confused by the app behaving this way.

@f-koehler
Copy link
Owner

@marouamghar, ok thanks for the info. Maybe there is some special behavior required on GNOME. I will look into it.

@f-koehler f-koehler added bug Something isn't working and removed question Further information is requested labels Dec 3, 2024
@f-koehler f-koehler moved this to Todo in KTailctl Dec 3, 2024
@f-koehler f-koehler added this to the v0.18.2 milestone Dec 3, 2024
@f-koehler
Copy link
Owner

@marouamghar I think GNOME does not have a tray area by default, especially on Fedora where it is pretty much vanilla. Which extension do you use to provide a tray area?

@marouamghar
Copy link
Author

Tray Icons: Reloaded, if i am not mistaken. I also have Dash to Panel, which could have an impact too... maybe?

@f-koehler
Copy link
Owner

Tray Icons: Reloaded, if i am not mistaken. I also have Dash to Panel, which could have an impact too... maybe?

Thanks, this might help me to reproduce your situation more accurately.

@f-koehler f-koehler modified the milestones: v0.18.2, v0.19.0 Dec 9, 2024
@f-koehler
Copy link
Owner

@marouamghar, I just tried to reproduce this on Fedora 41 with Gnome 47. Unfortunately, I did not see any tray icons with the Tray Icons Reloaded extensions. I tried KTailctl, Telegram, and Nextcloud Client with both Dash to dock enabled and disabled. Could it be that this extension is broken for Gnome 46+? (saw this issue in their repo: martinpl/tray-icons-reloaded#143)

@f-koehler f-koehler added help wanted Extra attention is needed and removed bug Something isn't working labels Jan 14, 2025
@f-koehler f-koehler removed this from the v0.19.0 milestone Jan 14, 2025
@f-koehler f-koehler removed the status in KTailctl Jan 14, 2025
@f-koehler f-koehler removed this from KTailctl Jan 14, 2025
@marouamghar
Copy link
Author

I am still on Fedora 40, gnome 46. I didn't upgrade yet because last year I had a similar issue with an extension taking its time to work... I should have precised that earlier.

@f-koehler
Copy link
Owner

No worries, let me try with Fedora 40 then. But in the issue I referenced above, problems with Gnome 46 are mentioned as well.
On the few occasions I have worked with Gnome I always used this extension instead: https://github.com/ubuntu/gnome-shell-extension-appindicator

@loukamb
Copy link

loukamb commented Jan 22, 2025

Using the AppIndicator/KStatusNotifierItem extension, closing the application despite having the tray icon enabled will entirely shutdown KTailctl on Gnome 47. I always have to relaunch the application afterwards to have it persist in the tray. This is not happening with other applications making use of the tray (e.g. megasync)

@marouamghar
Copy link
Author

@f-koehler you know what, I also have that appindicator installed ! I have multiple ones, so I missed it. Precisely, it's AppIndicator and KStatusNotifierItem Support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants