Skip to content

Commit 09f8dff

Browse files
Fixed issues with aside menu after config structure changes
1 parent c830eea commit 09f8dff

File tree

3 files changed

+32
-28
lines changed

3 files changed

+32
-28
lines changed

composer.json

+1
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
"phpspec/phpspec": "^3.3",
2828
"symfony/asset": "^3.3",
2929
"symfony/console": "^3.3",
30+
"symfony/debug-bundle": "^3.3",
3031
"symfony/twig-bundle": "^3.3",
3132
"symfony/var-dumper": "^3.3",
3233
"symfony/web-server-bundle": "^3.3",

src/LIN3S/PatternLibraryBuilder/Resources/templates/layouts/aside.html.twig

+29-28
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
{% macro accordion_menu(children, breadcrumbs) %}
1+
{% macro accordion_menu(children, pages, breadcrumbs) %}
22
{% import _self as macros %}
33

44
{% set items = [] %}
@@ -20,39 +20,40 @@
2020

2121
{% set items = items|merge([{
2222
header: accordion_item_title,
23-
content: macros.accordion_menu(item.children, breadcrumbs),
23+
content: macros.accordion_menu(item.children, item.pages, breadcrumbs),
2424
modifiers: modifiers,
2525
class_name: item_class_name,
2626
finder_value: item_title
2727
}]) %}
28+
{% endfor %}
2829

29-
{% for item in item.pages %}
30-
{% set accordion_item_title %}
31-
{% embed '@Lin3sPatternLibraryBuilder/components/link.html.twig' with {
32-
url: '/design-system/' ~ item.slug,
33-
target: '_self'
34-
} %}
35-
{%- block link_content -%}
36-
{% include '@Lin3sPatternLibraryBuilder/components/status.html.twig' with {
37-
level: item.status|default(' '),
38-
tag: 'span'
39-
} %}
40-
{{ item.title }}
41-
{%- endblock -%}
42-
{% endembed %}
43-
{% endset %}
30+
{% for item in pages %}
31+
{% set item_title = item.title|replace({'-': ' '}) %}
32+
{% set accordion_item_title %}
33+
{% embed '@Lin3sPatternLibraryBuilder/components/link.html.twig' with {
34+
url: '/design-system/' ~ item.slug,
35+
target: '_self'
36+
} %}
37+
{%- block link_content -%}
38+
{% include '@Lin3sPatternLibraryBuilder/components/status.html.twig' with {
39+
level: item.status|default(' '),
40+
tag: 'span'
41+
} %}
42+
{{ item.title }}
43+
{%- endblock -%}
44+
{% endembed %}
45+
{% endset %}
4446

45-
{% set modifiers = modifiers|merge(['leaf']) %}
46-
{% set item_class_name = 'finder__subject' %}
47+
{% set modifiers = [] %}
48+
{% set item_class_name = 'finder__subject' %}
4749

48-
{% set items = items|merge([{
49-
header: accordion_item_title,
50-
content: null,
51-
modifiers: modifiers|merge(['leaf']) ,
52-
class_name: 'finder__subject',
53-
finder_value: item_title
54-
}]) %}
55-
{% endfor %}
50+
{% set items = items|merge([{
51+
header: accordion_item_title,
52+
content: null,
53+
modifiers: modifiers|merge(['leaf']) ,
54+
class_name: 'finder__subject',
55+
finder_value: item_title
56+
}]) %}
5657
{% endfor %}
5758

5859
{% include '@Lin3sPatternLibraryBuilder/components/accordion.html.twig' with {
@@ -75,7 +76,7 @@
7576
} %}
7677
{% block finder_subjects %}
7778
{% import '@Lin3sPatternLibraryBuilder/layouts/aside.html.twig' as aside %}
78-
{{ aside.accordion_menu(menu.children, breadcrumbs) }}
79+
{{ aside.accordion_menu(menu.children, menu.pages, breadcrumbs) }}
7980
{% endblock %}
8081
{% endembed %}
8182
</div>

tests/Application/app/AppKernel.php

+2
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
namespace Tests\LIN3S\PatternLibraryBuilder;
1515

1616
use LIN3S\PatternLibraryBuilder\Symfony\Lin3sPatternLibraryBuilderBundle;
17+
use Symfony\Bundle\DebugBundle\DebugBundle;
1718
use Symfony\Bundle\FrameworkBundle\FrameworkBundle;
1819
use Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait;
1920
use Symfony\Bundle\TwigBundle\TwigBundle;
@@ -40,6 +41,7 @@ public function registerBundles() : array
4041

4142
if ('dev' === $this->getEnvironment()) {
4243
$bundles[] = new WebServerBundle();
44+
$bundles[] = new DebugBundle();
4345
}
4446

4547
return $bundles;

0 commit comments

Comments
 (0)