Skip to content

Comments

Scale the toast window for DPI#38198

Merged
zadjii-msft merged 3 commits intomainfrom
dev/migrie/b/toast-scaling
Apr 3, 2025
Merged

Scale the toast window for DPI#38198
zadjii-msft merged 3 commits intomainfrom
dev/migrie/b/toast-scaling

Conversation

@zadjii-msft
Copy link
Member

@zadjii-msft zadjii-msft commented Mar 27, 2025

Ah of course, AppWindow.Resize doesn't use DIPs. Why would it? It's not like literally everything else in XAML does.

Related to half of #38332

Ah of course, AppWindow.Resize doesn't use DIPs. Why would it? It's not
like literally everything else in XAML does.
@zadjii-msft zadjii-msft added the Product-Command Palette Refers to the Command Palette utility label Mar 27, 2025
@vanzue
Copy link
Contributor

vanzue commented Mar 28, 2025

First time switch still have some problem in my testing, looks like winui3 AppWindow remembers its original window scale, we may need to get up-to-date info from monitor itself.
image

Here is a proposed pr for it:
#38211

@vanzue vanzue self-requested a review March 28, 2025 08:35
vanzue and others added 2 commits March 28, 2025 04:17
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? -->
## Summary of the Pull Request

<!-- Please review the items on the PR checklist before submitting-->
## PR Checklist

- [ ] **Closes:** #xxx
- [ ] **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
   - [ ] [JSON for signing](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ESRPSigning_core.json) for new binaries
   - [ ] [WXS for installer](https://github.com/microsoft/PowerToys/blob/main/installer/PowerToysSetup/Product.wxs) for new binaries and localization folder
   - [ ] [YML for CI pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ci/templates/build-powertoys-steps.yml) for new test projects
   - [ ] [YML for signed pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/release.yml)
- [ ] **Documentation updated:** If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/windows-uwp/tree/docs/hub/powertoys) and link it here: #xxx

<!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here -->
## Detailed Description of the Pull Request / Additional comments

<!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well -->
## Validation Steps Performed

https://github.com/user-attachments/assets/62653d5f-e1ec-4547-b15b-c86bad610383
@vanzue
Copy link
Contributor

vanzue commented Mar 31, 2025

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@zadjii-msft zadjii-msft merged commit d48286a into main Apr 3, 2025
15 checks passed
colin-tso pushed a commit to colin-tso/PowerToys that referenced this pull request Apr 8, 2025
Ah of course, AppWindow.Resize doesn't use DIPs. Why would it? It's not like literally everything else in XAML does.

Related to half of zadjii-msft#508
@crutkas crutkas deleted the dev/migrie/b/toast-scaling branch April 10, 2025 05:24
sadirano pushed a commit to sadirano/PowerToys that referenced this pull request Jun 10, 2025
Ah of course, AppWindow.Resize doesn't use DIPs. Why would it? It's not like literally everything else in XAML does.

Related to half of zadjii-msft#508
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Product-Command Palette Refers to the Command Palette utility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants