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

fix: Update Flow-Hilla layout documentation #3666

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

Conversation

caalador
Copy link
Contributor

@caalador caalador commented Sep 13, 2024

Update the documentation on Flow server
layout usage with hilla views.
Add since tag to respective parts.

Part of vaadin/hilla#2709
Closes #3662

Update the documentation on Flow server
layout usage with hilla views.
Add since tag to respective parts.

Part of vaadin/hilla#2709
@github-actions github-actions bot added the Language unchecked English language check has not been done label Sep 13, 2024
@mshabarov mshabarov self-requested a review September 16, 2024 11:37
articles/flow/integrations/hilla.adoc Outdated Show resolved Hide resolved
articles/flow/integrations/hilla.adoc Show resolved Hide resolved
articles/flow/integrations/hilla.adoc Show resolved Hide resolved
@@ -222,9 +222,11 @@ export default function Layout() {
----

[[hilla-server-side-layout]]
[role="since:com.vaadin:vaadin@V24.5"]
== Flow Server Side Layout for Hilla Views
Copy link
Contributor

Choose a reason for hiding this comment

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

By the way, this chapter should give a link to https://vaadin.com/docs/latest/flow/advanced/menu-registry#creating-client-routes-menu-on-server to make it clear for readers how to create a dynamic menus. For hilla layouts it is mentioned, so should be the same for flow layout.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added at the end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language unchecked English language check has not been done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vaadin Flow 24.5.x-alpha release related documentation already shown on https://vaadin.com/docs/
2 participants