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

fix(macos/input): incorrect mouse input for non-main display #2461

Merged
merged 25 commits into from
May 2, 2024

use std::clamp instead of if statement.

4d89d73
Select commit
Loading
Failed to load commit list.
Merged

fix(macos/input): incorrect mouse input for non-main display #2461

use std::clamp instead of if statement.
4d89d73
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded May 2, 2024 in 1s

6.18% (+0.01%) compared to 7fb8c76

View this Pull Request on Codecov

6.18% (+0.01%) compared to 7fb8c76

Details

Codecov Report

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

Project coverage is 6.18%. Comparing base (7fb8c76) to head (4d89d73).

Additional details and impacted files
@@            Coverage Diff             @@
##           nightly   #2461      +/-   ##
==========================================
+ Coverage     6.17%   6.18%   +0.01%     
==========================================
  Files           86      86              
  Lines        17546   17548       +2     
  Branches      8190    8186       -4     
==========================================
+ Hits          1083    1085       +2     
+ Misses       15410   14655     -755     
- Partials      1053    1808     +755     
Files Coverage Δ
src/platform/macos/display.mm 42.15% <100.00%> (+1.15%) ⬆️
src/platform/macos/input.cpp 2.39% <0.00%> (ø)

... and 27 files with indirect coverage changes