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

feat: CommandBarFlyout #10411

Draft
wants to merge 42 commits into
base: master
Choose a base branch
from
Draft

Conversation

MartinZikmund
Copy link
Member

@MartinZikmund MartinZikmund commented Nov 11, 2022

GitHub Issue (If applicable): closes #12165

PR Type

What kind of change does this PR introduce?

  • Feature

What is the current behavior?

Not supported

What is the new behavior?

Supported

TODO

  • Workaround missing SystemBackdrop support

PR Checklist

Please check if your PR fulfills the following requirements:

@MartinZikmund MartinZikmund self-assigned this Nov 11, 2022
@gitpod-io
Copy link

gitpod-io bot commented Nov 11, 2022

@MartinZikmund MartinZikmund force-pushed the dev/mazi/winui-commandbarflyout branch 3 times, most recently from 3a35ada to 86d1e73 Compare November 16, 2022 09:21
@github-actions github-actions bot added the area/automation Categorizes an issue or PR as relevant to project automation label Nov 17, 2022
@MartinZikmund MartinZikmund changed the title [WIP] feat: CommandBarFlyout feat: CommandBarFlyout Nov 17, 2022
@MartinZikmund MartinZikmund marked this pull request as ready for review November 17, 2022 13:52
@MartinZikmund MartinZikmund force-pushed the dev/mazi/winui-commandbarflyout branch 2 times, most recently from dda14f8 to 0dcbb8d Compare November 17, 2022 14:23
Copy link
Member

@jeromelaban jeromelaban left a comment

Choose a reason for hiding this comment

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

Looking good, though still quite a few build failures :)

Comment on lines +296 to +301
// TODO:MZ: Move to Unloaded
//CommandBarFlyout.~public CommandBarFlyout()
//{
// m_primaryCommands.VectorChanged(m_primaryCommandsVectorChangedToken);
// m_secondaryCommands.VectorChanged(m_secondaryCommandsVectorChangedToken);

// UnhookAllCommandBarElementDependencyPropertyChanges();
//}
Copy link
Member

Choose a reason for hiding this comment

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

Was it moved?

@jeromelaban jeromelaban marked this pull request as draft January 16, 2023 18:34
auto-merge was automatically disabled January 16, 2023 18:34

Pull request was converted to draft

@MartinZikmund MartinZikmund force-pushed the dev/mazi/winui-commandbarflyout branch 2 times, most recently from 322fcd8 to 1144b9c Compare May 10, 2023 08:24
@MartinZikmund MartinZikmund force-pushed the dev/mazi/winui-commandbarflyout branch from 1144b9c to f3c0e82 Compare March 31, 2024 20:14
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

1 similar comment
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

@MartinZikmund MartinZikmund force-pushed the dev/mazi/winui-commandbarflyout branch from bcd56f5 to f79633b Compare April 15, 2024 11:30
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

2 similar comments
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

@MartinZikmund MartinZikmund force-pushed the dev/mazi/winui-commandbarflyout branch from 233cf11 to 038c701 Compare April 15, 2024 15:25
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-10411/index.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/automation Categorizes an issue or PR as relevant to project automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for CommandBarFlyout
4 participants