Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
As described in #2716773: [Meta] Convert 7.x-6.x Twig components to be Drupal 8-compatible, we need to convert the navbar component from Zen 7.x-6.x to be compatible with Drupal 8.
Proposed resolution
The parent issue describes the general solution we need to use when converting a component.
Solutions specific to the navbar component should be added here as we figure them out.
Remaining tasks
(reviews needed, tests to be written or run, documentation to be written, etc.)
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | interdiff-2725007-6-7.txt | 1.51 KB | finnsky |
| #7 | convert_navbar-2725007-7.patch | 1.94 KB | finnsky |
| #6 | zen-convert_navbar_component_PD8-2725007.patch | 1.31 KB | Jerenus |
| #4 | 2725007-navbar-component.patch | 4.42 KB | sarahjean |
Comments
Comment #2
sarahjean CreditAttribution: sarahjean at Acquia commentedComment #3
sarahjean CreditAttribution: sarahjean at Acquia commentedComment #4
sarahjean CreditAttribution: sarahjean at Acquia commentedThis patch is a mess, just adds menu.html.twig (but adds them to all the menus right now, where we just want to add to the primary menu). What we really need to do for navbar is going to need to live in a more specific template file. The integration with navbar.twig is not done at all in this patch.
Comment #5
sarahjean CreditAttribution: sarahjean at Acquia commentedComment #6
Jerenus CreditAttribution: Jerenus as a volunteer commentedStart a new patch to catch up the "Date: Thu Jun 2 13:50:02 2016 +0800" dev version.
Comment #7
finnsky CreditAttribution: finnsky at Skilld commentedHi! I updated Jerenus patch with json to display navbar in styleguide.
Comment #8
andriyun CreditAttribution: andriyun at Skilld, Drupal Ukraine Community commented