-
Notifications
You must be signed in to change notification settings - Fork 39
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
187323603 drag over webview bug #1530
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good -- I'm impressed that it works as well as it does. One suggestion to add a check for whether the pointer is captured rather than blindly capturing it multiple times.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1530 +/- ##
==========================================
- Coverage 85.10% 85.09% -0.01%
==========================================
Files 563 563
Lines 28149 28156 +7
Branches 7728 7730 +2
==========================================
+ Hits 23955 23959 +4
- Misses 3888 4041 +153
+ Partials 306 156 -150
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
codap-v3 Run #4520
Run Properties:
|
Project |
codap-v3
|
Branch Review |
main
|
Run status |
Passed #4520
|
Run duration | 08m 53s |
Commit |
ab40b73ce4: 187323603 drag over webview bug (#1530)
|
Committer | Evangeline Ireland |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
30
|
Skipped |
0
|
Passing |
227
|
View all changes introduced in this branch ↗︎ |
* Adds a pointer capture on component drag * remove commented out code * PR fix
Fixes bug where dragging components over a webview makes the component lose the pointer focus. This captures the pointer while dragging a component