Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It's great that this module supports the core toolbar and admin_menu out of the box, but we should consider backporting what the D8 version of the module does to support use of the https://drupal.org/project/navbar module when used in Drupal 7.
Comment | File | Size | Author |
---|---|---|---|
#7 | Selection_007.png | 7 KB | Dave Reid |
#7 | Selection_005.png | 7.09 KB | Dave Reid |
#4 | 2287983-environment-indicator-navbar-support.patch | 5.9 KB | Dave Reid |
#3 | Selection_001.png | 14.03 KB | Dave Reid |
#3 | 2287983-environment-indicator-navbar-support.patch | 5.89 KB | Dave Reid |
Comments
Comment #1
Dave ReidComment #2
Dave ReidHere's a diff against a fairly recent 7.x-2.x-dev build, so I hope it still applies to the branch. This seems to get me what I need. I couldn't use the same pattern that Drupal.behaviors.environment_indicatorAdminMenu does by defining Drupal.admin.behaviors['environment_indicator'] because the navbar.js file that contains Drupal.behaviors.navbar runs before the environment indicator behaviors on my site.
Comment #3
Dave ReidRevised version floats the environment indicator off to the left side. Screenshot attached of it in use with a fairly narrow viewport and the Ember theme:
Comment #4
Dave ReidNeeded one little merge conflict fixed to truly apply to the latest 7.x-2.x branch.
Comment #6
e0ipsoComment #7
Dave ReidAwesome! Thanks for helping bring it home and getting it in. It does look like we could have a follow-up for the styling of the 'Available environments'. This is what I'm seeing compared to the other navbar 'sub-bars': it's really thin and doesn't appear to have the same styling.
Comment #8
e0ipsoI see what's going on… I'll do a follow up.
Comment #10
Dave ReidOk great. I thought maybe it might be worth a discussion about either removing the tab bar altogether if there aren't any other environments, or maybe a link to the settings page to create new environments, but as per the original request this is resolved. Thanks so much @e0ipso!