Skip to content

Test merge 818 828 - #830

Open
Photon89 wants to merge 19 commits into
masterfrom
test-merge-818-828
Open

Test merge 818 828#830
Photon89 wants to merge 19 commits into
masterfrom
test-merge-818-828

Conversation

@Photon89

Copy link
Copy Markdown
Member

@DarthGandalf @underwaterdba

I tried to resolve the conflicts between #818 and #828. Cannot do any multimonitor testing though.

@Photon89
Photon89 marked this pull request as draft August 17, 2026 21:45
@Photon89
Photon89 marked this pull request as ready for review August 18, 2026 07:18
@Photon89

Copy link
Copy Markdown
Member Author

So, now, according to my testing, the current state supports both the redo from #828 and the xdg-portal targets from #818. I cannot test whether multimonitor capture works as expected though. A new $screen_mode variable, which has been initially introduced by AI while trying to solve the merge conflict, is supposed to tell whether we are talking to the current monitor or not (defined in line 6337). It is passed to Wayland::xdg_portal but not used there. I cannot tell if it is useful because of lacking testing on multimonitor...

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