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.

Sign up for GitHub

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

Jump to bottom

Update demo application #112

Merged
merged 4 commits into from
Mar 5, 2024
Merged

Update demo application #112

merged 4 commits into from
Mar 5, 2024

Conversation

tamasmak
Copy link
Contributor

@tamasmak tamasmak commented Mar 1, 2024 •
edited
Loading

  • Updates to use SideNav and SideNavItem compontent
  • All the other changes are code formatting

@tamasmak tamasmak self-assigned this Mar 1, 2024
@tamasmak tamasmak requested a review from heruan March 1, 2024 13:02
@heruan
Copy link
Member

heruan commented Mar 4, 2024

  • Updates @vaadin-component-factory/vcf-nav version to 1.1.3 to make it run with Flow 24.4

We better replace the component with the new SideNav as we did in O11y Kit demos.

@tamasmak
Copy link
Contributor Author

tamasmak commented Mar 4, 2024 •
edited
Loading

We better replace the component with the new SideNav as we did in O11y Kit demos.

Replaced to use SideNav.

@heruan heruan merged commit f1f44bd into main Mar 5, 2024
2 checks passed
@heruan heruan deleted the update-demo-app branch March 5, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heruan heruan heruan approved these changes

Assignees

@tamasmak tamasmak

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tamasmak @heruan