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.

Sign up for GitHub

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

Jump to bottom

"Power" button doesn't work in new menu under KDE #8326

Open
marmarek opened this issue Jul 6, 2023 · 13 comments · Fixed by QubesOS/qubes-desktop-linux-menu#36
Open

"Power" button doesn't work in new menu under KDE #8326

marmarek opened this issue Jul 6, 2023 · 13 comments · Fixed by QubesOS/qubes-desktop-linux-menu#36
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: app menu The primary user-facing GUI application menu in Qubes OS C: desktop-linux-kde Support for the K Desktop Environment (KDE) needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. r4.2-host-stable r4.2-vm-bookworm-stable r4.2-vm-bullseye-stable r4.2-vm-centos-stream8-stable r4.2-vm-fc36-cur-test r4.2-vm-fc37-stable r4.2-vm-fc38-stable T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@marmarek
Copy link
Member

marmarek commented Jul 6, 2023

How to file a helpful issue

Qubes OS release

R4.2

Brief summary

As title says

Related to #8159

Steps to reproduce

  1. Start qubes-app-menu under KDE
  2. Click "power" icon

Expected behavior

KDE native logout screen shows up

Actual behavior

Received error while trying to log out
GDbus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name is not activatable
@marmarek marmarek added T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. C: app menu The primary user-facing GUI application menu in Qubes OS labels Jul 6, 2023
@marmarek marmarek added this to the Release 4.2 milestone Jul 6, 2023
@marmarek
Copy link
Member Author

marmarek commented Jul 6, 2023

The KDE logout can be open by calling org.kde.LogoutPrompt.promptLogout() dbus method on /LogoutPrompt object and org.kde.LogoutPrompt destination.
dbus-send command: dbus-send --type=method_call --session --dest=org.kde.LogoutPrompt /LogoutPrompt org.kde.LogoutPrompt.promptLogout
But since the menu already interacts with dbus, it's probably better to call the method from python directly.

@marmarek marmarek added the diagnosed Technical diagnosis has been performed (see issue comments). label Jul 6, 2023
@andrewdavidwong andrewdavidwong added the C: desktop-linux-kde Support for the K Desktop Environment (KDE) label Jul 6, 2023
@marmarta marmarta mentioned this issue Jul 6, 2023
Merged
Closed
@qubesos-bot
Copy link

Automated announcement from builder-github

The package desktop-linux-menu has been pushed to the r4.2 testing repository for the Debian template.
To test this update, first enable the testing repository in /etc/apt/sources.list.d/qubes-*.list by uncommenting the line containing bullseye-testing (or appropriate equivalent for your template version), then use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package desktop-linux-menu has been pushed to the r4.2 testing repository for the Debian template.
To test this update, first enable the testing repository in /etc/apt/sources.list.d/qubes-*.list by uncommenting the line containing bookworm-testing (or appropriate equivalent for your template version), then use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package desktop-linux-menu has been pushed to the r4.2 testing repository for the CentOS centos-stream8 template.
To test this update, please install it with the following command:

sudo yum update --enablerepo=qubes-vm-r4.2-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component desktop-linux-menu (including package desktop-linux-menu) has been pushed to the r4.2 testing repository for the Fedora template.
To test this update, please install it with the following command:

sudo dnf update --enablerepo=qubes-vm-r4.2-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component desktop-linux-menu (including package desktop-linux-menu) has been pushed to the r4.2 testing repository for the Fedora template.
To test this update, please install it with the following command:

sudo dnf update --enablerepo=qubes-vm-r4.2-current-testing

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component desktop-linux-menu (including package desktop-linux-menu) has been pushed to the r4.2 testing repository for the Fedora template.
To test this update, please install it with the following command:

sudo dnf update --enablerepo=qubes-vm-r4.2-current-testing

Changes included in this update

@andrewdavidwong andrewdavidwong added the pr submitted A pull request has been submitted for this issue. label Jul 7, 2023
@marmarek marmarek reopened this Jul 12, 2023
@marmarek
Copy link
Member Author

The "Power" button does the right thing when the menu detects it's running on KDE. The problem is, it doesn't always detect it.
Specifically, it looks for XDG_CURRENT_DESKTOP variable to have KDE, but for some reason, it still have XFCE (not empty, really XFCE) after logging in to KDE.
Restarting qubes-app-menu user service fixes the issue, until next login when it's back with XFCE. Xfce was not started on this system at that time.

@DemiMarie DemiMarie self-assigned this Jul 13, 2023
@andrewdavidwong andrewdavidwong added needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. and removed diagnosed Technical diagnosis has been performed (see issue comments). labels Jul 13, 2023
@andrewdavidwong andrewdavidwong added the affects-4.2 This issue affects Qubes OS 4.2. label Aug 8, 2023
@tasket tasket mentioned this issue Aug 8, 2023
Open
@andrewdavidwong andrewdavidwong removed this from the Release 4.2 milestone Aug 13, 2023
@qubesos-bot
Copy link

Automated announcement from builder-github

The package desktop-linux-menu has been pushed to the r4.2 stable repository for the Debian template.
To install this update, please use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package desktop-linux-menu has been pushed to the r4.2 stable repository for the CentOS centos-stream8 template.
To install this update, please use the standard update command:

sudo yum update

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The package desktop-linux-menu has been pushed to the r4.2 stable repository for the Debian template.
To install this update, please use the standard update command:

sudo apt-get update && sudo apt-get dist-upgrade

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component desktop-linux-menu (including package desktop-linux-menu) has been pushed to the r4.2 stable repository for the Fedora template.
To install this update, please use the standard update command:

sudo dnf update

Changes included in this update

@qubesos-bot
Copy link

Automated announcement from builder-github

The component desktop-linux-menu (including package desktop-linux-menu) has been pushed to the r4.2 stable repository for the Fedora template.
To install this update, please use the standard update command:

sudo dnf update

Changes included in this update

@DemiMarie DemiMarie removed their assignment Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: app menu The primary user-facing GUI application menu in Qubes OS C: desktop-linux-kde Support for the K Desktop Environment (KDE) needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. r4.2-host-stable r4.2-vm-bookworm-stable r4.2-vm-bullseye-stable r4.2-vm-centos-stream8-stable r4.2-vm-fc36-cur-test r4.2-vm-fc37-stable r4.2-vm-fc38-stable T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Run KDE logout dialog in KDE marmarta/qubes-desktop-linux-menu
4 participants
@marmarek @DemiMarie @andrewdavidwong @qubesos-bot