-
Notifications
You must be signed in to change notification settings - Fork 1
Use the Drupal defined menu for the primary navigation #99
Conversation
e311be5 to
43c8d81
Compare
|
@andybroomfield I'll pull this into my local tomorrow and take a look - im a bit worried the work involved to get it looking right might take me away from the croydon migration. |
|
@cjstevens78 To demo this you need to create a menu under structure -> menu and add its block to the primary and/or secondary menu region. Shouldn't effect your Croydon verison if your overridding the header region template, though you may find it useful to use the technique in your own theme in the future. |
13ea419 to
0aec0fe
Compare
- Adjust the menu--main.html.twig markup to match bootstrap
- Change the region name to services_menu - Change the menu link to services - Only show if the region has items - Add services menu alone to the services mega row region - Fix the colours in case plain text blocks added.
- Apply the menu colums logic to the mega row menus - Use bootstrap classes on Drupal menu templates - Nudge the padding and margin in the megarow menu and the responsive menu
27eaee1 to
54bbe58
Compare
|
@cjstevens78 Had to update this branch to avoid merge conflicts, can this be reviewed soonish before more fixes as its getting a awkward to deal with merge conflicts when rebasing from master. |
@andybroomfield apologies ! approved |
|
Thanks @cjstevens78 :-) |


Fix #94.
This uses the Drupal primary menu for the menu
Also restyles the dropdows to appoximate the main menu style.