Comments

eshta’s picture

Status: Active » Closed (won't fix)

This is a cool idea, but navbar is intended to remain a simple backport of the D8 core toolbar. As such, any major functional changes need to be introduced and implemented in Drupal 8 first.

olragon’s picture

I will implement in Drupal 8 when I have free time. Thank you for your suggestion.
Update patch for Drupal 7, include missing icons.

thehong’s picture

StatusFileSize
new12.64 KB

Just correct the patch name. Credit to olragon.

dotmanblue’s picture

Great Improvement! I had been using administration menu and was missing the dropdown functionality it offered.
Thanks for the patch.

fugazi’s picture

This is wonderful like it very much, will now uninstall my admin menu and use this one. Thanks, a great feature

marcoka’s picture

for drupal 8 there is a module that does this already. maybe the code helps
https://www.drupal.org/project/admin_toolbar

the dropdown would the bar make perfect.

rwilson0429’s picture

This is a great usability improvement. The D8 admin_toolbar is great but, the D7 admin_toolbar does not have the responsive features as does the D8 version.

This patch pretty much gives us the greatness of the D8 admin_toolbar in the D7 navbar. There are some tweaks that would be good (i.e z-index when the navbar has two rows; and maybe, for child-menus to appear on-hover instead of on-click).

Hope this gets committed.

izmeez’s picture

1+ for child menus to appear on hover

Andrew Gorokhovets’s picture

@ eshta, how we see in Drupal 8 it's already done with admin toolbar module. So, maybe, it's time to apply patch?
I want to uninstall my admin menu and use this one, but only with drop down functionality for horizontal display.
Thanks

Andrew Gorokhovets’s picture

Status: Closed (won't fix) » Needs review

The last submitted patch, dropdown-for-navbar-horizontal-display.patch, failed testing.

The last submitted patch, 2: dropdown-for-navbar-horizontal-display.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 3: dropdown_for_navbar-2481207-3.patch, failed testing.

bwaindwain’s picture

StatusFileSize
new2.47 KB

Here's another CSS only attempt that uses code from by bccrayton at https://www.drupal.org/node/2390047

izmeez’s picture

@bwaindwain Thanks for the patch. It works as expected providing dropdown and child menus on hover.
Yes, yes, yes, I know it has to be accepted as a feature for drupal 8+ and then filter down.
It's great to have it working now.

rwilson0429’s picture

the patch in #14 works beautifully. Thanks for the patch bwaindwain.

Without this patch, I could not see myself using the navbar module because it didn't provide a fast access to administration pages (too many clicks to get to specific admin pages). This patch solves that problem for me.

There is one problem that I think is caused by this patch. The 'Content' menu is missing the link to add content (node/add).
Only local images are allowed.

This is a very significant improvement in this modules usability and appeal. I Hope this gets committed soon. Thanks again for the patches.

bwaindwain’s picture

Re "missing node/add menu item": The navbar only reflects what is in the management menu. If you want to change the menu, go to /admin/structure/menu/manage/management.

I have made my own custom module that adds a bunch of useful admin links to the management menu.

izmeez’s picture

@bwaindwain would you mind sharing an example of the custom module you created to add links?

bwaindwain’s picture

StatusFileSize
new1.88 KB

Here ya go. Enjoy! :)

izmeez’s picture

@bwaindwain Thanks. That's a great example of extending the drupal system.

donapis’s picture

@bwaindwain Great work for the patch and management module. I'm going to use this for all my projects. Thanks again.

bwaindwain’s picture

StatusFileSize
new3.53 KB

here's a patch that adds right-to-left support

I don't like the excessive use of !important, but it works.

bccrayton’s picture

donapis’s picture

Status: Needs work » Reviewed & tested by the community

Patch look great. RTBC

delacosta456’s picture

hi
i would like to confirm that the patch is working for me.

However Can we have the dropdown of any item menu have an "overflow" css so that it when a list of dropdown is long the scrollbar appear.

thanks

joseph.olstad’s picture

I totally want this functionality.

Which patch out of these do we need, it looks like the patches with the svg files would likely be needed.

I'll try to test this out myself soon.

I think it's time to add some new maintainers to this project.

johan den hollander’s picture

Succesfully tried the #22 patch.

izmeez’s picture

We have been using patch in #22 for some time now.

joseph.olstad’s picture

someone should take ownership of this project and put this patch in for the benefit of everyone! This patch is pure gold!

delacosta456’s picture

hi
was'nt able to apply the patch for the current navbar version Panopoly distro 1.76

Thanks

izmeez’s picture

@delacosta456 can you provide details of how you tried to apply the patch and what response you got. Panopoly is a large distro so there may be other factors involved.

delacosta456’s picture

StatusFileSize
new218.96 KB

hi @izmeez
Thanks for replying
I am apply patch through Phpstrom and having chunks in navbar css file from line 95 to end

I attached a screenshot for you

Thanks

izmeez’s picture

Just tried the patch on a fresh download of navbar without any problems. You might have to manually edit the two files and add the lines in the patch to the end of the files. In the screenshot can't see far enough down your file (the end) to where the lines are added. Possibly, the leading lines are not the same:

--- a/css/navbar.menu.css
+++ b/css/navbar.menu.css
@@ -92,3 +92,82 @@
   top: 0;
   z-index: 1;
 }
+
+
+/**
+ * Styling for Custom Navbar Dropdown
+ * adapted from Bryant Crayton bccrayton's work at https://www.drupal.org/node/2390047
+ */
+
more
delacosta456’s picture

StatusFileSize
new1.17 KB

hi @izmeez

ok thanks ... but i would to suggest uploading you the css files in the navbar module bundled with Panopoly so that you could analyze better.

Please look at the zipped attachement.

izmeez’s picture

@delacosta456 Sorry, I don't have time to delve into your issue with panopoly. I was only confirming that the patch is still valid. The patch simply adds lines to the end of both the existing files in navbar module.

delacosta456’s picture

hi @izmeez

ok Thanks i applied it manually and everything seems to be working perfectly

Thanks very much