-
Notifications
You must be signed in to change notification settings - Fork 125
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: (core) Add shellbar side navigation example and directive. #1604
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 1f77020 |
2235276
to
c8f1bc6
Compare
@JKMarkowski can you move |
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.
@JKMarkowski can you update the product switch examples? Also the hamburger menu doesn't look like per specification
Now, the 3rd levels are removed completely. |
Hi @InnaAtanasova, the button is with white background only on hover, shouldn't it be like that ? |
It should look like the Product Switch button |
305b094
to
60e2488
Compare
Hi @InnaAtanasova, it fully follows product-switch control button now. |
Please provide a link to the associated issue.
fixes: #347
Please provide a brief summary of this pull request.
There is added side navigation directive on shellbar and 1 example of usage.
Also I removed information about side navigation up to 10 levels usage.
Please check whether the PR fulfills the following requirements
https://github.com/SAP/fundamental-ngx/blob/master/CONTRIBUTING.md
Documentation checklist:
README.md