Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@romangg
Copy link
Member

@romangg romangg commented Apr 11, 2024

The screen locker is only relevant on Wayland and only with KDE setup. This is a breaking change, as consumers need to update their setup code for the server.

romangg added 5 commits April 12, 2024 00:13
The description field is wrong. This unexpectedly also fixes the test-wl run
when doing a release build.
The Wayland specific base header should not be included in the generic render
library part.
This removes the hard dependency on it in X11.

BREAKING CHANGE: consumer creates screen locker for desktop platform
@codecov
Copy link

codecov bot commented Apr 12, 2024

Codecov Report

Attention: Patch coverage is 64.42308% with 37 lines in your changes are missing coverage. Please review.

Project coverage is 50.37%. Comparing base (ac77050) to head (27d0577).

Files Patch % Lines
como/desktop/kde/screen_locker.h 50.00% 10 Missing and 3 partials ⚠️
como/base/wayland/server_helpers.h 62.50% 2 Missing and 1 partial ⚠️
como/win/wayland/screen_lock.h 75.00% 2 Missing and 1 partial ⚠️
tests/integration/lib/helpers.cpp 0.00% 3 Missing ⚠️
como/desktop/kde/screen_locker.cpp 84.61% 1 Missing and 1 partial ⚠️
como/input/wayland/cursor_image.h 0.00% 0 Missing and 2 partials ⚠️
como/render/gl/scene.h 0.00% 2 Missing ⚠️
como/win/wayland/window.h 33.33% 1 Missing and 1 partial ⚠️
tests/plasma/main_wayland.cpp 0.00% 2 Missing ⚠️
como/input/wayland/keyboard_redirect.h 0.00% 0 Missing and 1 partial ⚠️
... and 4 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
- Coverage   51.02%   50.37%   -0.66%     
==========================================
  Files         938      941       +3     
  Lines       87395    87404       +9     
  Branches    45013    45460     +447     
==========================================
- Hits        44593    44026     -567     
- Misses      38238    39427    +1189     
+ Partials     4564     3951     -613     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@romangg romangg merged commit 4b46eb9 into winft:master Apr 12, 2024
@romangg romangg deleted the screen-lock branch April 12, 2024 08:54
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.

1 participant