Some users maybe want to assign specific menu to specific user role. Right now module use only 'management' system menu.
For example, if you are 'administrator' then 'management' menu is suitable for you, but what if your role is 'content manager' and you want to have your own menu in navigation toolbar.

So, I think, it would be great to have this settings.

CommentFileSizeAuthor
#2 navbar-ability-to-assign-menu-for-user-role-1927830-2.patch2.65 KBandrey_zb
#1 navbar-ability-to-assign-menu-for-user-role-1927830-1.patch3.94 KBandrey_zb

Comments

andrey_zb’s picture

andrey_zb commented
Status: Active » Needs review
StatusFileSize
new navbar-ability-to-assign-menu-for-user-role-1927830-1.patch3.94 KB

So I have a solution for this point.
Added settings menu item and respective functionality.

andrey_zb’s picture

andrey_zb commented
StatusFileSize
new navbar-ability-to-assign-menu-for-user-role-1927830-2.patch2.65 KB

Please excuse me, this is my first public patch.
So, patch with few improvements is in attach

jessebeach’s picture

jessebeach commented
Status: Needs review » Postponed

andrey_zb, thank you for the patch! Would you mind working on the D8 version first, then we can backport the solution to D7 after?

#1939884: Make the administration menu in the toolbar plugabble: don't hardcode to the 'admin' menu

Postponing on a solution in the D8 Toolbar module.

andrey_zb’s picture

andrey_zb commented
Issue summary: View changes
Status: Postponed » Closed (won't fix)

Patch was designed for Navbar 7.x-1.1, and will not work for the latest module version

nithinkolekar’s picture

nithinkolekar commented
Status: Closed (won't fix) » Active

IMO lot of users/contributors are still not moved to d8 and very familiar with d7. So it would be nice to keep this open for d7 until D8 RC get released.
I think admin menu source could be ported to support navbar.

eshta’s picture

eshta commented
Status: Active » Closed (won't fix)

Navbar is just a port of the D8 toolbar. As such, all of the issues have to first be addressed in D8. That's why it was suggested to continue work in that area first. Once it is added to D8 then we can look at back-porting it here.