Skip to content

BF: Multihead fixes for X11 #5520

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

Merged
merged 1 commit into from
May 12, 2023
Merged

Conversation

mdcutone
Copy link
Member

Fixes #5407 using the provided workaround for multihead Linux systems running X11.

Needs additional testing on systems which use such configurations. Stuff works normally on my Linux machine after applying these changes.

@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #5520 (73db220) into dev (ad6aa5b) will decrease coverage by 6.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #5520      +/-   ##
==========================================
- Coverage   47.08%   40.25%   -6.84%     
==========================================
  Files          65       65              
  Lines        9749     9749              
  Branches     1257     1257              
==========================================
- Hits         4590     3924     -666     
- Misses       4905     5627     +722     
+ Partials      254      198      -56     

see 15 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@peircej peircej merged commit 3a9f66c into psychopy:dev May 12, 2023
@geigermartin
Copy link

This fix unfortunately isn’t working for us.
The workaround proposed in #5407 also broke with the fix.
While initially only using Spyder, I did some further testing in VSCode, and the issue is consistent across both.

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