When you "move local tasks into menu", they're just appended to the menu via jQuery; and because they retain the class secondary, they oftentimes inherit styling from the theme that makes them unreadable. This is a common problem with Zen-based themes and, more recently, with Admin module enabled. I think the simplest solution is to remove the secondary class after appending. Screenshots and patch below. Thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | admin_menu-645526.patch | 936 bytes | traviscarden |
| #2 | admin_menu-645526.patch | 1.03 KB | traviscarden |
| admin_menu.patch | 890 bytes | traviscarden | |
| After: Theme styles don't apply | 3.21 KB | traviscarden | |
| Before: Theme styles override display | 3.63 KB | traviscarden |
Comments
Comment #1
traviscarden commented[Edit: Removed obsolete note on compatibility]
Comment #2
traviscarden commentedNo love, huh? What if I roll it for 3.x-dev? :)
Comment #3
traviscarden commentedAnybody?
Comment #4
klonosI'd love to help but I use a custom made theme and don't seem to be having any issues. Is this only zen-based theme specific or what?
Comment #5
traviscarden commentedThank you, klonos. It's not specific to Zen-based themes... Any theme that styles secondary tabs may experience the issue described. Zen happens to experience it badly because it gives secondary links a light background, so when Admin Menu moves them up top and changes the text color to white you can't read the text any more. The conflict could look like anything, depending on what your theme does with
.secondary. Does that clear it up a little? :)Comment #6
klonosYep, I kinda already knew by taking a look at the before-after screenshots in the issue description. Still... this doesn't apply to me, so I can't help :(
Comment #8
traviscarden commentedThis one may be better.
Comment #10
sunThanks for reporting, reviewing, and testing! Committed to all branches.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.