Simplify LegionGrid screen capture - Githubissues

dictation-toolbox / Caster

Dragonfly-Based Voice Programming and Accessibility Toolkit

Other

336 stars 122 forks source link

Simplify LegionGrid screen capture #900

Closed LexiconCode closed 3 years ago

LexiconCode commented 3 years ago

Description

Use ImageGrab to capture multi-monitor screens ImageGrab all_screens is Windows OS only

Motivation and Context

Removes the need for gdi.py which was a workaround for a lack of multi-monitor support in ImageGrab

How Has This Been Tested

Tested on primary and secondary monitor

There are issues with DPI scaling but this is unrelated to this PR.
- Tested old implementation of DPI scaling issues the results were the same.
Tested with both the detailed and rough parameters on monitors 1 and 2

Maintainer/Reviewer Checklist

[x] Basic functionality has been tested and works as claimed.
[x] New documentation is clear and complete.
[x] Code is clear and readable.