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

[BUG] Some or every QT apps will not open #487

Closed
pinarruiz opened this issue Jul 23, 2024 · 10 comments · Fixed by #490
Closed

[BUG] Some or every QT apps will not open #487

pinarruiz opened this issue Jul 23, 2024 · 10 comments · Fixed by #490

Comments

@pinarruiz
Copy link

Context

Hi,

Until now i have been using bluskript/stylix from #142 as non of the QT apps had the stylix theme applied and this fork worked.

I saw that recently the 6bbae4f commit should fix the issue, but instead i am getting this error in what seems every QT app that I open:

QSystemTrayIcon::setVisible: No Icon set

Particularly Nextcloud and VLC, I dont know/think I have any others, but shows the error on both.

My stylix config:

{ pkgs, ... }: {
  stylix = {
    enable = true;
    polarity = "dark";
    image = pkgs.fetchurl {
      url = "https://...";
      sha256 = "...";
    };
  };
}

Versions

stylix: 0ef7003
nixpkgs: stable-24-05 (63d37cc)
Home Manager: release-24.05 (e1391fb)

Additional Context

$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.41, NixOS, 24.05 (Uakari), 24.05.20240721.63d37cc`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - nixpkgs: `/nix/store/lzhqf5y9dcgfkg5f7f96jjc814pgrsqy-source`

Nextcloud client in Home Manager:

{ lib, ... }: {
  services.nextcloud-client = {
    enable = lib.mkDefault true;
    startInBackground = lib.mkDefault true;
  };
}

VLC was added as a system package:

{ pkgs, ... }: {
  environment.systemPackages = with pkgs; [
    ...
    vlc
    ...
  ];
}

Related: #367

If any extra context is required please let me know.

Thanks for the help in advance 😄.

@trueNAHO
Copy link
Collaborator

@Jackaed, do you maybe know how to resolve this?

@Jackaed
Copy link
Contributor

Jackaed commented Jul 24, 2024

No idea while I'm away from my computer, I think I tested VLC on my system so I'm surprised it isn't working. I'll have a look.

@Jackaed
Copy link
Contributor

Jackaed commented Jul 24, 2024

I can't reproduce this on unstable. Could you try a "more normal" Qt app (just for my information), e.g. dolphin or pcmanfm-qt?

@danth
Copy link
Owner

danth commented Jul 24, 2024

Does VLC use a system tray icon? (That's the ones that appear in the taskbar usually to show that an app is running in the background.) I know Nextcloud does and that seems to be what the error is hinting towards. Perhaps we have accidentally overwritten their icons somehow by installing the Breeze icons package?

@pinarruiz
Copy link
Author

I can't reproduce this on unstable. Could you try a "more normal" Qt app (just for my information), e.g. dolphin or pcmanfm-qt?

Hi, @Jackaed

Will try later in the day, as I cannot do it now, and will give some feedback, the vlc version i am using is the one in stable-24-05 (63d37cc), but i have my config ready for allowing some unstable packages, I will let you know how that goes.

@pinarruiz
Copy link
Author

Does VLC use a system tray icon? (That's the ones that appear in the taskbar usually to show that an app is running in the background.) I know Nextcloud does and that seems to be what the error is hinting towards. Perhaps we have accidentally overwritten their icons somehow by installing the Breeze icons package?

Hi, @danth

I don't know if VLC has a tray icon, i cant remember ever seeing one, that said, #142 seems to work, and i guess maybe the theme is the culprit as @Jackaed is suggesting (#487 (comment))

trueNAHO added a commit to trueNAHO/stylix that referenced this issue Jul 25, 2024
Revert commit 6bbae4f ("kde: apply Qt theme on non-KDE systems and
add Qt6 support (danth#367)") due to unexpected issues.

Follow-up work is tracked in danth#480.

Closes: danth#480
Closes: danth#485
Closes: danth#487
trueNAHO added a commit to trueNAHO/stylix that referenced this issue Jul 25, 2024
Revert commit 6bbae4f ("kde: apply Qt theme on non-KDE systems and
add Qt6 support (danth#367)") due to unexpected issues.

Follow-up work is tracked in danth#489.

Closes: danth#480
Closes: danth#485
Closes: danth#487
@pinarruiz
Copy link
Author

Hi @Jackaed

Just checked with unstable vlc (versions seem to be the same) and i got the same result. The app appears not to launch and it gives the same error message:

QSystemTrayIcon::setVisible: No Icon set

@pinarruiz
Copy link
Author

I am not sure #490 should close this issue, it will render it useless thats true, but disabling a feature that's not working should not be a reason to close an issue at least in my opinion, what do you think @trueNAHO ?

@trueNAHO
Copy link
Collaborator

I am not sure #490 should close this issue, it will render it useless thats true, but disabling a feature that's not working should not be a reason to close an issue at least in my opinion, what do you think @trueNAHO ?

Closing issues by reverting the commits introducing them is somewhat controversial.

IMHO, issues related to #490 should be closed to distinguish existing and non-existing problems. However, the issues closed by #490 will serve as references and considerations for #489.

@pinarruiz
Copy link
Author

Makes sense to me, adding it to the related section at #489 looks good, so all good from my part closing the issue with #490 as its being referenced on the mentioned tracking issue.

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

Successfully merging a pull request may close this issue.

4 participants