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

Implement multilevel navigation #512

Merged
merged 27 commits into from
Jan 31, 2023

Conversation

IvanDmitriev1
Copy link
Collaborator

@IvanDmitriev1 IvanDmitriev1 commented Jan 25, 2023

Cleans and simplified NavigationView code.
Removed multiple enumerations over MenuItems and FooterMenuItems.

Pull request type

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the new behavior?

  • Added support for BreadcrumbBar
  • Added new method NavigateWithHierarchy
  • Added the Navigating event that supports canceling the current navigation
  • Fixes GoBack

Other information

  • GoForward currently is not implemented

@github-actions github-actions bot added the styles Topic is related to styles label Jan 25, 2023
@IvanDmitriev1
Copy link
Collaborator Author

@pomianowski how about not adding GoForward functionality ?

@IvanDmitriev1 IvanDmitriev1 marked this pull request as ready for review January 27, 2023 04:57
@pomianowski pomianowski self-assigned this Jan 29, 2023
@pomianowski pomianowski added controls Changes to the appearance or logic of custom controls. navigation Changes to navigation related controls. labels Jan 29, 2023
@pomianowski pomianowski added this to the WinUi Compatible milestone Jan 29, 2023
Copy link
Member

@pomianowski pomianowski left a comment

Choose a reason for hiding this comment

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

Thank you for the huge amount of work you put into improving the navigation. Looks much better now. Breadcrumb for sub-items and styles can still be improved, but for now it looks great.

@@ -91,6 +91,7 @@ public MainWindowViewModel(IServiceProvider serviceProvider)
{
new NavigationViewItem { Content = "BreadcrumbBar", TargetPageType = typeof(BreadcrumbBarPage) },
new NavigationViewItem { Content = "NavigationView", TargetPageType = typeof(NavigationViewPage) },
new NavigationViewItem { Content = "Multilevel navigation demo", TargetPageType = typeof(MultilevelNavigationPage) },
Copy link
Member

Choose a reason for hiding this comment

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

Kinda long title, maybe just "Multilevel navigation"

@pomianowski
Copy link
Member

There are a few bugs left to fix in Wpf.Ui.Demo

@IvanDmitriev1
Copy link
Collaborator Author

How about to review a #502 ?

@github-actions github-actions bot removed the controls Changes to the appearance or logic of custom controls. label Jan 29, 2023
@pomianowski pomianowski merged commit a10c21f into lepoco:development Jan 31, 2023
@IvanDmitriev1 IvanDmitriev1 deleted the navigation branch February 1, 2023 06:37
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
navigation Changes to navigation related controls. styles Topic is related to styles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants