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

It should be mentioned that AppLayout as designed to be a root layout #82

Open
anasmi opened this issue Jan 10, 2020 · 1 comment
Open

Comments

@anasmi
Copy link

anasmi commented Jan 10, 2020

AppLayout is designed to be a root element. While it's mentioned in client-side documentation https://cdn.vaadin.com/vaadin-app-layout/2.0.4/#/elements/vaadin-app-layout, it's completely omitted from a server-side.

For best results, the component should be added to the root level of your application (i.e., as a direct child of <body>).

Now adding it as a child makes NavBar rendered at the top of the page, not its parent element. (which causes frustration, at the very least)

@vaadin-bot vaadin-bot transferred this issue from vaadin/vaadin-app-layout-flow Oct 6, 2020
@web-padawan
Copy link
Member

Moving this to the docs repository. Let's take this into account when creating demos for AppLayout.

@web-padawan web-padawan transferred this issue from vaadin/vaadin-app-layout Dec 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants