Skip to content

NavigationView pane toggle button implementation #527

Merged
pomianowski merged 16 commits intolepoco:developmentfrom
IvanDmitriev1:Navigation-expander
Feb 11, 2023
Merged

NavigationView pane toggle button implementation #527
pomianowski merged 16 commits intolepoco:developmentfrom
IvanDmitriev1:Navigation-expander

Conversation

@IvanDmitriev1
Copy link
Copy Markdown
Contributor

@IvanDmitriev1 IvanDmitriev1 commented Feb 8, 2023

Implemented pane toggle button.
TitleBar now can be placed in logical order.
Added setting default values if the TitleBar property is set in NavigationView.

Pull request type

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

@IvanDmitriev1 IvanDmitriev1 marked this pull request as ready for review February 9, 2023 12:40
@IvanDmitriev1 IvanDmitriev1 changed the title NavigationView pane toggle button NavigationView pane toggle button implementation Feb 9, 2023
@pomianowski
Copy link
Copy Markdown
Member

Hi, great job. The animation looks very elegant 😄

However, there is a new bug with statically defined menu items. Take a look at the NavigationViewPage page
image

@IvanDmitriev1
Copy link
Copy Markdown
Contributor Author

IvanDmitriev1 commented Feb 10, 2023

I didn't touch it, yesterday it worked. :(
I think I realized what I changed.

@pomianowski
Copy link
Copy Markdown
Member

Pane width breaks for fluent nav
image

@pomianowski
Copy link
Copy Markdown
Member

And AutoSuggestBox margin breaks spacings for the top one
image

@IvanDmitriev1
Copy link
Copy Markdown
Contributor Author

Should there be such icons ?
image

@pomianowski
Copy link
Copy Markdown
Member

@pomianowski
Copy link
Copy Markdown
Member

Thanks for the suggestions and contributions. Looks fine to me at this stage

@pomianowski pomianowski merged commit 90c52aa into lepoco:development Feb 11, 2023
@IvanDmitriev1 IvanDmitriev1 deleted the Navigation-expander branch February 12, 2023 07:48
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants