When I use route: on a parent menu item as suggested in https://www.drupal.org/node/1543750 the menu works fine (prevents the link but shows the dropdown children items) on desktop. However, on mobile, the menu does not show the + sign to access the children items.

I am using AT 8.x-1.0-rc2+104-dev

Here is the link to a copy of the site: http://prdev2017.prestonrosedale.com You will have to enter "shield" in both fields in the popup. Only the "About" and "Services" parent links are currently using the route: option. The others are using the Menu Firstchild module (https://www.drupal.org/project/menu_firstchild).

Thanks.

Comments

BastouBach created an issue. See original summary.

  • Jeff Burnz committed 3e3095b on 8.x-1.x
    Issue #2869496: route:<nolink> prevents dropdown + sign on mobile
    

  • Jeff Burnz committed 0adf020 on 8.x-2.x
    Issue #2869496: route:<nolink> prevents dropdown + sign on mobile
    

  • Jeff Burnz committed 0adf020 on 8.x-3.x
    Issue #2869496: route:<nolink> prevents dropdown + sign on mobile
    

Jeff Burnz’s picture

OK, thats the JavsScript fix, however theres a much bigger CSS fix that needs doing.

Jeff Burnz’s picture

Version: 8.x-1.0 » 8.x-2.x-dev
Status: Active » Needs work

  • Jeff Burnz committed eab7fd4 on 8.x-2.x
    Issue #2869496 by Jeff Burnz: route:<nolink> prevents dropdown + sign on...

  • Jeff Burnz committed eab7fd4 on 8.x-3.x
    Issue #2869496 by Jeff Burnz: route:<nolink> prevents dropdown + sign on...
Jeff Burnz’s picture

Status: Needs work » Fixed

I have pushed the necessary JS and template updates to Adaptivetheme core.

For your existing theme you still need to update CSS/SCSS files - see the child issue #2906378: Convert menu item link selectors to .menu__link for the commits.

NOTE: all this pertains to 8.x-2.x and the upcoming 8.x-3.x releases, be careful with 8.x-1.x, it should work OK, you'll have to do some work.

Lastly my apologies for not initially supporting this, honestly this is a easter egg feature of Drupal and I never even knew you could do this in D8, lol, so thanks for the heads up.

Status: Fixed » Closed (fixed)

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