-
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
[FancyZones Editor] Fluent UX and theming support #6899
Conversation
@niels9001 One question: How does the 'custom' section looks? (Asking because of #2207.) |
how bad is this and #6562 going to conflict? |
Same behavior. I believe we need to rethink the current tabs structure and how to create new layouts. I'll open a new issue to discuss that, since it's maybe not entirely in scope for this PR. |
Do we really need a new issue? We have issue #2207. |
#6562 should be merged first, since it brings functional changes that we don't want to test in a merge with UI changes. |
@niels9001 |
@niels9001 |
@enricogior Totally agree. @htcfreek I have added my thoughts and some concepts in the Editor v2 topic. I think this would solve a lot of outstanding issues like the ones you mentioned. Some open questions pending. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
can we shift this into a draft PR so we don't accidently merge it in? |
@niels9001
Which one should we do first? |
@enricogior I think I'll close this PR and start a new branch based on master. Too many changes, but should be straightforward copy-and-pasting of e.g. brushes. I can finish this in the next week or so, so maybe better to wait for 0.28 for this work item? |
I'm not sure if .NET Core would touch the UI / XAML at all? So we can work on it simultaneously? |
Sure, it won't be in 0.27 for sure, not enough time to do it and test for 0.27.
I would assume it should not. We will start on the .NET Core migration, late this week or earlier next week. We will commit to master sometime next week. |
Closing this in favor of: #8148 |
Summary of the Pull Request
This PR contains the following changes:
And overall, could be a nice basis to further ideate on solutions for some outstanding issues.
This PR should only be merged after #6562.
Layout selection screen
Dialogs
** High contrast**
PR Checklist