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

Improve integration with PowerDevil and libnotificationmanager #4

Open
romangg opened this issue May 14, 2020 · 0 comments
Open

Improve integration with PowerDevil and libnotificationmanager #4

romangg opened this issue May 14, 2020 · 0 comments

Comments

@romangg
Copy link
Member

romangg commented May 14, 2020

Introduction

Both daemons/libraries have a dependency on libkscreen. According to the maintainer of both projects:

  • PowerDevil uses libkscreen to check when there is an external monitor connected to a laptop and then not go into standby should the laptop lid be shut.
  • libnotificationmanager switches automatically in a "not-disturb" mode when an external monitor is connected. The assumption is that in such a case the user is likely holding a presentation.

Problem

While this should still work in KWinFT's X11 session it won't in KWinFT's Wayland session since libkscreen misses a respective plugin for Wrapland which is only provided by Disman.

Solution

Option 1: Ignore

Currently we ignore these two features. But at least the PowerDevil one seems quite essential for certain use cases.

Option 2: Fork PowerDevil

As established above at least the PowerDevil functionality is quite important. For that we could opt to fork PowerDevil and change the libkscreen code to Disman. At least at the moment the diff should be rather contained. But PowerDevil is a complicated project and it would be nicer to work together with its current upstream maintainer.

Option 3: Provide integration upstream

That would be the best long-term solution of course but it seems difficult since there would need to be some kind of plugin system in both consumers.

Option 4: Provide integration in KWinFT/Wrapland

KDE's outputmanagement protocol could be supported at least in a read-only way in Wrapland. Then libkscreen would still work.

Option 5: ...

...

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

No branches or pull requests

1 participant