-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PowerRename] Resolve UI focus issue #14605
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than what I commented looks good..
@jaimecbernardo This want would be nice to have as part of 0.51 :). |
@niels9001 , sounds good. Feel free to merge it in once checks are done ;) |
Thank you for the contribution! |
Looks like this just made it in. @jaimecbernardo can we cherry pick this into stable for v0.51? |
@dedavis6797 Yes, that's the idea. Just fixing those pipelines first and then I'll sync stable. |
Summary of the Pull Request
This PR removes a vertical ScrollViewer that caused bad focus behavior. Removed custom AutoSuggestBox style. The i button is now next to the textbox to resolve an issue where the clear button was not correctly aligned.
It also introduces a regression: the placeholder text is not vertically aligned. This will be fixed in the next WinUI update however: microsoft/microsoft-ui-xaml#5968. I believe this is fine for now, and allows us to not have a complex custom style.
Quality Checklist
Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.