-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
app-sidebar-nav navItems reassignment #126
Comments
I'm having the same issue. Is there a way to reassign it and get the navbar to only show the reassignment? |
- fix(sidebar): navItems reassignment bug [#126](coreui/coreui-free-angular-admin-template#126), [#42](coreui/coreui-free-angular-admin-template#42), #12 - refactor(aside): drop `Replace` - refactor(breadcrumb): drop `Replace` new component `cui-breadcrumb` (migration needed) - refactor(footer): minor changes - refactor(header): drop `Replace` - refactor(layout): cleanup - refactor(sidebar-nav): drop `Replace`, redesign
@nealyip @emilynielson |
Hi i updated @coreui/angular to @2.4.5 . But still the issue persists. I can't reassign the navItems . The value i change in my .ts file doesn't get affected in my template file. |
hi @pranavs1791 Please try to use this code example for quick and dirty verification whether dynamic reassign of sidebar items works or not. What you need is to create some sort of sidebar-nav.service to reassign navItems array on the fly and subscribe the result in default-layout.component |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions |
For the above code, the component will prepend an item instead of replacement.
Is this behavior intended or a bugs?
For semantic meaning, reassigning variables seems to be referring to replacement instead of prepend.
Thanks
The text was updated successfully, but these errors were encountered: