When we enabled the Admin Toolbar, the menu are expanded by default, like this:

On my "Toolbar Anti-flicker", I dropped a simple line to fix above issue:
http://cgit.drupalcode.org/toolbar_anti_flicker/tree/css/toolbar-anti-fl...

I'm so exciting the Core issue #2542050: Toolbar implementation creates super annoying re-rendering. is about to commit to fixing the flicker problem. Now I'm interested if maintainers of Admin Toolbar want to do a fix on Admin Toolbar itself.

(It may be very simple as above just one line or super complex of few hours work of testing. Honestly, I have no interests to upload a patch without module maintainers approve because there's an effective way to get the same job done.)

Thanks :)

CommentFileSizeAuthor
c20170225_220547.png61.48 KBdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet created an issue. See original summary.

droplet’s picture

droplet CreditAttribution: droplet commented
Issue tags: +toolbar-flicker
eme’s picture

eme CreditAttribution: eme at emerya commented

Hello !

I'm open to integrate this fix into Admin Toolbar of course. Just one question : if it is going to be integrated into core is it still needed for Admin Toolbar, and if yes should we just wait for it first or not ?

droplet’s picture

droplet CreditAttribution: droplet commented

Thanks @eme!

I'm trying to get rid of this problem in CORE now:
https://www.drupal.org/node/2542050#comment-11959736

If the CORE doesn't accept my workaround, we able to patch AdminToolbar with few lines:

/** Hidden dropdown menu on loading **/
.toolbar-horizontal ul li.menu-item--expanded ul {
  display: none;
}
eme’s picture

eme CreditAttribution: eme at emerya commented

Is this display:none just enough to get it right ? Core patch seems quite more complicated...

eme’s picture

eme CreditAttribution: eme at emerya commented
Status: Active » Closed (won't fix)

It has been integrated to 8.4.x.