magdev/wc-bootstrap
1
Fork 0
Code Issues Pull Requests Actions Releases 12 Activity

Make category tree items span full parent width (v0.1.9)
All checks were successful
Create Release Package / PHP Lint (push) Successful in 46s
Details
Create Release Package / PHPUnit Tests (push) Successful in 51s
Details
Create Release Package / Build Release (push) Successful in 1m3s
Details

Browse Source 
Move <a> elements to be direct children of the list-group so they
naturally fill the full offcanvas/sidebar width. Chevron toggle moved
inside the link with click interception to prevent navigation.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Marco Grätsch
2026-03-29 16:02:13 +02:00
parent 1e7f82615b
commit f64e9cd5a2
5 changed files with 48 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

55
templates/global/category-tree-node.html.twig
View File

@@ -2,7 +2,7 @@
# Single Category Tree Node (recursive)
#
# Renders one category as a list-group-item-action link with optional
# collapsible nested list for children. Included recursively.
# collapsible nested children. Included recursively.
#
# Expected context:
# node - Category node from wc_bootstrap_get_category_tree()
@@ -15,33 +15,30 @@
{% set has_children = node.children is not empty %}
{% set is_open = node.is_active or node.is_ancestor %}
<div class="category-tree-item">
<div class="d-flex align-items-center">
<a href="{{ node.url }}"
class="list-group-item list-group-item-action border-0 py-1 flex-grow-1{% if node.is_active %} active{% endif %}"
style="padding-left: {{ 0.5 + (level - 1) * 1 }}rem;"
{% if node.is_active %}aria-current="page"{% endif %}>
{{ node.name }}
<small class="text-body-secondary ms-1">({{ node.count }})</small>
</a>
{% if has_children %}
<button class="btn btn-sm btn-link text-body-secondary p-0 px-2 category-tree-toggle"
type="button"
data-bs-toggle="collapse"
data-bs-target="#cat-{{ node.term_id }}"
aria-expanded="{{ is_open ? 'true' : 'false' }}"
aria-controls="cat-{{ node.term_id }}"
aria-label="{{ __('Toggle subcategories', 'wc-bootstrap') }}">
<i class="bi bi-chevron-down small" aria-hidden="true"></i>
</button>
{% endif %}
</div>
<a href="{{ node.url }}"
class="list-group-item list-group-item-action border-0 py-1 d-flex align-items-center{% if node.is_active %} active{% endif %}"
style="padding-left: {{ 0.75 + (level - 1) * 1 }}rem;"
{% if node.is_active %}aria-current="page"{% endif %}>
<span class="flex-grow-1">{{ node.name }}</span>
<small class="text-body-secondary ms-1">({{ node.count }})</small>
{% if has_children %}
<div class="collapse{{ is_open ? ' show' : '' }}" id="cat-{{ node.term_id }}">
{% for child in node.children %}
{% include 'global/category-tree-node.html.twig' with { node: child, level: level + 1 } only %}
{% endfor %}
</div>
<span class="category-tree-toggle ms-1"
role="button"
data-bs-toggle="collapse"
data-bs-target="#cat-{{ node.term_id }}"
aria-expanded="{{ is_open ? 'true' : 'false' }}"
aria-controls="cat-{{ node.term_id }}"
aria-label="{{ __('Toggle subcategories', 'wc-bootstrap') }}"
onclick="event.preventDefault(); event.stopPropagation();">
<i class="bi bi-chevron-down small" aria-hidden="true"></i>
</span>
{% endif %}
</div>
</a>
{% if has_children %}
<div class="collapse{{ is_open ? ' show' : '' }}" id="cat-{{ node.term_id }}">
{% for child in node.children %}
{% include 'global/category-tree-node.html.twig' with { node: child, level: level + 1 } only %}
{% endfor %}
</div>
{% endif %}