Skip to content

Note for support removed for vcf-nav and vcf-nav-item in upgrading guide #4428

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

Open
wants to merge 3 commits into
base: latest
Choose a base branch
from

Conversation

mukherjeesudebi
Copy link
Contributor

@mukherjeesudebi mukherjeesudebi commented Jul 11, 2025

Some customers use the old vaadin 23 starter projects AppNav and AppNavItem which relied on vcf-nav and vcf-nav-item for side navigation. But vcf-nav and vcf-nav-item are no longer supported in vaadin 24 and customers get error when upgrading. This note provides a guide that the new SideNav and SideNavItem has to be used instead.

Copy link

github-actions bot commented Jul 11, 2025

AI Language Review

The main changes between the two versions of the Upgrading Guide documentation involve the addition of a new section titled "Side Navigation Replaces AppNav and AppNavItem". The new section suggests replacing AppNav and AppNavItem components with "Side Navigation" due to the discontinuation of support for the Vaadin Component Factory components they relied on.

Suggestions for improvement:

  1. Clarification: Provide more context or detail about where to find more information on "Side Navigation". This could come in the form of a link to relevant documentation or a brief description of where to find these new components.

  2. Section Order: Consider if the placement of the new section at the end of the document is optimal. Depending on its importance, it may benefit from being highlighted earlier in the document.

  3. Formatting: The link to "Side Navigation" documentation is technically correct, but clarifying whether it's a clickable link or a code reference in the context would improve usability.

Overall, the new section could be more effective with added information on transitioning to "Side Navigation," ensuring a smoother migration path for users.

mukherjeesudebi and others added 2 commits July 11, 2025 15:23
Co-authored-by: Jouni Koivuviita <jouni@vaadin.com>
Co-authored-by: Jouni Koivuviita <jouni@vaadin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants