Skip to content
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

[ColorPicker] Converting main window to FluentWindow #31805

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

donlaci
Copy link
Collaborator

@donlaci donlaci commented Mar 8, 2024

Summary of the Pull Request

Converting main window to FluentWindow to have cornered window

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

tested locally

@donlaci donlaci requested a review from niels9001 March 8, 2024 15:43
@donlaci
Copy link
Collaborator Author

donlaci commented Mar 11, 2024

Hi @niels9001 ,
I would like to ask you to take a look at this modification. Thanks a lot!

Copy link
Contributor

@niels9001 niels9001 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Collaborator

@jaimecbernardo jaimecbernardo left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for the fix!

@jaimecbernardo jaimecbernardo merged commit 62d278b into microsoft:main Mar 14, 2024
9 checks passed
jaimecbernardo added a commit that referenced this pull request Apr 3, 2024
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.

None yet

3 participants