PowerToys icon indicating copy to clipboard operation
PowerToys copied to clipboard

Made PT Launcher per-monitor DPI aware

Open drawbyperpetual opened this issue 1 year ago • 2 comments

Summary of the Pull Request

Makes PowerToys Launcher per-monitor DPI aware.

PR Checklist

  • [X] Closes: #8949
  • [X] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
  • [ ] Tests: Added/updated and all pass
  • [ ] Localization: All end user facing strings can be localized
  • [ ] Dev docs: Added/updated
  • [ ] New binaries: Added on the required places
  • [ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Makes PowerToys Launcher per-monitor DPI aware by setting this in application manifest.

Validation Steps Performed

When DPI-scaling is different (e.g. 100% vs 175%) between primary and secondary monitor, verified that:

  • Before fix, Launcher is sharp on primary and fuzzy on secondary.
  • After fix, Launcher is sharp on primary and secondary.

drawbyperpetual avatar May 16 '24 12:05 drawbyperpetual

I feel like we should make this into a wpf props

crutkas avatar May 16 '24 19:05 crutkas

I feel like we should make this into a wpf props

This is in the app.manifest , not a props file, so I'm not sure that would apply here 🤔

jaimecbernardo avatar May 21 '24 09:05 jaimecbernardo