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

add more lxqt files #600

Open
wants to merge 44 commits into
base: main
Choose a base branch
from
Open

add more lxqt files #600

wants to merge 44 commits into from

Conversation

Besanon
Copy link
Contributor

@Besanon Besanon commented Nov 4, 2024

No description provided.

first file for the LXQT 2.0 desktop group
starter file for LXQT Desktop
I use sddm as display manager

I cant remove the other file - only use graphical env., sorry

After startlxqt i would add  2 lines to sddm to enable the start of LXQT desktop
Enable sddm to start an lxqt desktop session
lxqt-session to be started by startlxqt. Display manager: sddm
is needed for several complaints:
DENIED  kscreen_backend_launcher open owner @{user_config_dirs}/lxqt/lxqt.conf comm=kscreen_backend requested_mask=r denied_mask=r
DENIED  kscreen_backend_launcher open /usr/share/lxqt/lxqt.conf comm=kscreen_backend requested_mask=r denied_mask=r
DENIED  kscreen_backend_launcher open owner @{user_config_dirs}/lxqt/session.conf comm=kscreen_backend requested_mask=r denied_mask=r
DENIED  kscreen_backend_launcher open /usr/share/lxqt/session.conf comm=kscreen_backend requested_mask=r denied_mask=r
add child-open
you think its too permissive to have app-launcher-user here, right?
add needed programs
turning back to layout of corresponding xfce file.

@{exec_path} mr,

/usr/share/icons/{,**} r,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is included in lxqt

profile lxqt-about @{exec_path} {
include <abstractions/base>
include <abstractions/lxqt>
include <abstractions/video>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it really needed? This abstraction is usually useful for media player, it is less common in classic UI app. Maybe you need something else here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its not needed, a left-over from an old version. I will check several other profiles in the group whether graphics or video is used

include <abstractions/lxqt>
include <abstractions/nameservice-strict>
include <abstractions/qt5-shader-cache>
include <abstractions/gvfs-open>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't use gvfs-open directly. You need someting like: @{open_path} rPx -> child-open, here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, its used in lxqt-globalkeysd. I altered it in 2 other occurences in profiles to be considered later

@roddhjav
Copy link
Owner

roddhjav commented Nov 4, 2024

Most basic UI app can open external resource to show the help page. Therefore you need something like the following in most of these profiles:

  @{open_path} rPx -> child-open-help,

Have a look at the xfce app as example.

Comment on lines 22 to 23
@{bin}/screengrab rpx,
@{bin}/lxqt-config-brightness rpx,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rPx instead of rpx (unless it is intentional)

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 this pull request may close these issues.

2 participants