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

Allow to show an empty sidebar #1261

Closed
wants to merge 3 commits into from

Conversation

raimund-schluessler
Copy link
Contributor

@raimund-schluessler raimund-schluessler commented Aug 6, 2020

Allows to show a blank sidebar without header and tab navigation bar. Useful for showing a hint when loading data from the server (please ignore the unfinished sidebar implementation of the Tasks app in the GIF).

appsidebar

Start of implementing #1090.

Signed-off-by: Raimund Schlüßler <[email protected]>
@raimund-schluessler raimund-schluessler added 3. to review Waiting for reviews feature: app-sidebar Related to the app-sidebar component labels Aug 6, 2020
@raimund-schluessler raimund-schluessler marked this pull request as ready for review August 6, 2020 20:01
Signed-off-by: Raimund Schlüßler <[email protected]>
Signed-off-by: Raimund Schlüßler <[email protected]>
@raimund-schluessler
Copy link
Contributor Author

While working on #1262 I came to the conclusion that I will rework the AppSidebar component to use a render function (the this.$children array behaves inconsistent between browser and jest test environment, so it's probably better to not rely on it). Also, I realized that the close button and the navtabs (in case of more than one tab) need to be visible for an empty/blank sidebar as well. I hence close this PR and implement the blank option with the render function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews feature: app-sidebar Related to the app-sidebar component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants