Skip to content

Code Quality: Migrated GetCursorPos to CsWin32 #15174

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 4 commits into from
Apr 14, 2024

Conversation

gumbarros
Copy link
Contributor

Resolved / Related Issues

Validation
How did you test these changes?

  • Did you build the app and test your changes?
  • Did you check for accessibility? You can use Accessibility Insights for this.
  • Did you remove any strings from the en-us resource file?
    • Did you search the solution to see if the string is still being used?
  • Did you implement any design changes to an existing feature?
    • Was this change approved?
  • Are there any other steps that were used to validate these changes?

@yaira2 yaira2 added the ready to merge Pull requests that are approved and ready to merge label Apr 14, 2024
@gumbarros
Copy link
Contributor Author

Sorry, I am on the phone and probably messed the license at NativeMethods

@yaira2 yaira2 merged commit a05a5e5 into files-community:main Apr 14, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants