Problem/Motivation

"toolbar-menu-administration" is used in gin theme on the main toolbar. Toolbar menu module is using the same css class name, which leads to a conflict in css styling, and the toolbar menu is not displayed at all.
See #3258279: [toolbar_menu] Styling conflict with toolbar menu module

Steps to reproduce

Use gin theme, and try to add a toolbar menu.

Proposed resolution

Rename the class name in toolbar_menu module. It should be easier than to try to override/fix it in the gin theme.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Pasqualle created an issue. See original summary.

pasqualle’s picture

StatusFileSize
new1.1 KB

The patch makes the toolbar menu visible in gin.
It seems to be working with flat menus only. From multilevel menus only the first level is displayed.

Geoffrey Vandamme’s picture

Status: Active » Needs work
StatusFileSize
new330.85 KB

Hi Pasqualle,

I've tested the patch for the toolbar menu module, but found an extra issue with the css from admin_toolbar module. Seems like the
first link isn't displayed because of a text-indent coming from tools.css

pasqualle’s picture

The patch does not apply to latest 2.x-dev.

preeti.chawla’s picture

StatusFileSize
new1016 bytes
new75.37 KB

@Pasqualle I have created new patch which will work in latest 2.x.dev. By this Toolbar menu will be visible in Gin theme. Please apply and test for same.

preeti.chawla’s picture

Status: Needs work » Needs review
elgandoz’s picture

Status: Needs review » Needs work
StatusFileSize
new541.41 KB

The last fix currently throws an error, given patch in #5 uses a jQuery selector (which is deprecated, I believe).

elgandoz’s picture

Version: 8.x-2.x-dev » 3.0.0
Status: Needs work » Needs review
StatusFileSize
new1 KB
new532 bytes

Patch attached with the correct selector.

giuseppe87’s picture

+1 for the #8 patch: without it the menù with Gin isn't shown, with the patch it works.

jan kellermann’s picture

Status: Needs review » Reviewed & tested by the community

Patch from comment #8 works for us. Thank you very much.

dydave made their first commit to this issue’s fork.

dydave’s picture

Title: css class name conflict with gin theme » Gin Theme compatibility issues: Fix class name conflict and empty menu links
Status: Reviewed & tested by the community » Needs work

Thanks everyone for the great help on this issue!

I've rolled in the changes from the latest patch at #8 in merge request MR!6 to get the work started on this issue.

I'm not exactly sure I understand how to reproduce the problem here, could somebody please provide more details?

For example, which type of Toolbar is used with the Gin Theme?
Any other potential modules?
Where is this class toolbar-menu-administration exactly used in the Gin theme? What causes the issue exactly?
(Any link code references pointing to the Gin repo)

Could we please get some screenshots as well?

Additionally, I'm still seeing the issue mentioned above at #3 with extra empty items, when enabling Gin, with any of the Gin Toolbar selected (see in Gin theme settings):

 
I'm seeing the same exact issue with any of the Toolbar menu items created and displayed in the Toolbar.

Could we try fixing this issue as well as part of a GIN Theme compatibility fix?

Any help, comments, testing and feedback would be greatly appreciated.
Thanks in advance!

dydave’s picture

Version: 3.0.0 » 8.x-2.x-dev
dydave’s picture

Version: 8.x-2.x-dev » 3.x-dev