Add support for the Navbar module bundled with Panopoly.

Comments

shadcn’s picture

shadcn commented

Blockers:

#1757466: Prefix all navbar classes to prevent theme clashes
#1964996: Support Breakpoints Module w/ Latest Navbar

mpotter’s picture

mpotter commented

What's needed to move this forward? Any alternatives to waiting for Breakpoint support in Panopoly?

Right now Open Atrium 2 is stuck with Radix Alpha 3 because of the navbar issues. Would be nice to move this along if OA2 is going to be a showcase distribution for Radix.

Seems that the Navbar module is now using the generic "nav" class in the ul which messes up the display and functionality.

shadcn’s picture

shadcn commented
Assigned: Unassigned » shadcn

I have a fix for this coming up shortly. ^^

shadcn’s picture

shadcn commented

The latest Radix (dev) now has support for the navbar module (dev).

The following patch is needed for navbar http://drupal.org/node/1955912#comment-7377438

Also, watching what comes out of #1971292: Minor fixes for Navbar module in panopoly_admin.make.

andrew_mallis’s picture

andrew_mallis commented

Please see #2049867: Panopoly and Apps icons do not appear in the navbar which also affects Radix.
I'm proposing a resolution in #2051621: decouple navbar styling from panopoly-admin.css which if you weighed in on could see its way into RC-5?

shadcn’s picture

shadcn commented
Status: Active » Fixed

Latest Panopoly + Radix now works with Navbar module. Marking this as fixed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.