Skip to content

[Image Resizer] Clarify preset size descriptions - #49694

Merged
moooyo merged 2 commits into
microsoft:mainfrom
niels9001:fix-image-resizer-text
Aug 5, 2026
Merged

[Image Resizer] Clarify preset size descriptions#49694
moooyo merged 2 commits into
microsoft:mainfrom
niels9001:fix-image-resizer-text

Conversation

@niels9001

@niels9001 Niels Laute (niels9001) commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary of the Pull Request

Updates Image Resizer preset descriptions in both the resize dialog and the Settings page from sentence fragments such as Fits within 1920 × 1080 pixels to CRUTKAS's proposed Fit - 1920 × 1080 pixels format. The UI reuses existing localized mode labels and removes the obsolete third-person resources.

Screenshots

Resize dialog

Image Resizer dialog showing the new Fit - Width × Height unit format

Settings page

Image Resizer Settings page showing the new Fit - Width × Height unit format

PR Checklist

Detailed Description of the Pull Request / Additional comments

Preset details now use the infinitive resize mode followed by a neutral dash and the dimensions on both Image Resizer surfaces. This avoids requiring translators to make a sentence fragment agree grammatically with the dimensions.

Accessible Settings descriptions use the same wording, and the obsolete third-person mode resources are removed from both resource sets.

Validation Steps Performed

  • Built ImageResizerUI.csproj for x64 Debug
  • Built ImageResizer.UnitTests.csproj for x64 Debug
  • Ran all 149 Image Resizer unit tests successfully
  • Built Settings.UI.csproj for x64 Debug
  • Ran the PR-built resize dialog and Settings page and verified the new text in both surfaces

Niels Laute (niels9001) and others added 2 commits August 5, 2026 10:55
Use localized resize mode labels followed by a dash instead of sentence fragments that are difficult to translate.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: f058c3d9-38d8-4a35-8c5e-69100b2b673c
Use the same action-oriented resize mode labels and dash separator on the Settings preset cards, including accessible descriptions.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f058c3d9-38d8-4a35-8c5e-69100b2b673c

@moooyo moooyo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@moooyo
moooyo enabled auto-merge (squash) August 5, 2026 10:31
@moooyo
moooyo merged commit 16baa2d into microsoft:main Aug 5, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adjust Image Resizer string to remove translation clarity issue

2 participants