Skip to content

[WinUI3 Gallery] [RichEditBox]: On applying Text Scaling to 200%, 'Enter search text' placeholder text is getting trimmed#1741

Merged
akanpatel2206 merged 3 commits intomainfrom
user/akanpatel2206/RichEditBoxPage_Accessibility
Feb 4, 2025
Merged

[WinUI3 Gallery] [RichEditBox]: On applying Text Scaling to 200%, 'Enter search text' placeholder text is getting trimmed#1741
akanpatel2206 merged 3 commits intomainfrom
user/akanpatel2206/RichEditBoxPage_Accessibility

Conversation

@akanpatel2206
Copy link
Contributor

@akanpatel2206 akanpatel2206 commented Feb 3, 2025

[WinUI3 Gallery] [RichEditBox]: On applying Text Scaling to 200%, 'Enter search text' placeholder text is getting trimmed.

Description

Removed the Width property from the TextBox element in the RichEditBoxPage.xaml file to prevent the placeholder text from getting trimmed when applying Text Scaling to 200
asoqs961

Motivation and Context

This is an Accessibility issue. User with low vision who rely on text scaling will be impacted and will face difficulties if the placeholder gets trimmed on application of Text Scaling to 200.

How Has This Been Tested?

Made the changes in the code and tested the scenario by applying Text Scaling to 200% and verified that the placeholder text is not getting trimmed.

Screenshots (if appropriate):

For scaling 225:
image
For scaling 100:
image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

…er search text' placeholder text is getting trimmed
@akanpatel2206 akanpatel2206 requested a review from a team February 3, 2025 09:42
@akanpatel2206
Copy link
Contributor Author

/azp run

@akanpatel2206
Copy link
Contributor Author

/azp run

@akanpatel2206 akanpatel2206 merged commit a43c68b into main Feb 4, 2025
1 check passed
@niels9001 niels9001 deleted the user/akanpatel2206/RichEditBoxPage_Accessibility branch March 12, 2025 08:33
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.

7 participants