-
Notifications
You must be signed in to change notification settings - Fork 4
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
Updated AppLayout demos to use AppNav. #123
base: main
Are you sure you want to change the base?
Updated AppLayout demos to use AppNav. #123
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- In the AppLayoutMultipleHeadersView demo, the tabs appear to be misaligned or not functioning correctly.
- The AppLayoutStickyToolbarView seems to have no content ?!
Please carefully compare it with what has been published.
I also checked stylewise wether or not the main toolbar being compact would fit better but decided to leave it as not compact. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The stick toolbars demo is still not working, and the toolbars still have spacing beneath them.
change "DWCJ Application" to become "Application" only |
Remove the drawer’s header and footer from all demos. |
Please format the code |
Quality Gate passedIssues Measures |
in |
remove webforj logo from all demos. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@EHandtkeBasis, could you please take a look at the demos that have already been published? The new rewrite version seems to be off and doesn’t match the published ones.
No description provided.