Skip to content

Commit a5a7bf7

Browse files
committed
fix(docs): remove dead breadcrumb docs segment filter
1 parent 5496439 commit a5a7bf7

2 files changed

Lines changed: 21 additions & 2 deletions

File tree

docs/app/reflex_docs/templates/docpage/docpage.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -607,11 +607,11 @@ def breadcrumb(path: str, nav_sidebar: rx.Component, doc_content: str | None = N
607607
docs_sidebar_drawer,
608608
)
609609

610-
# Split the path into segments, removing 'docs' and capitalizing each segment
610+
# Split the path into segments and capitalize each segment.
611611
segments = [
612612
segment.capitalize()
613613
for segment in path.split("/")
614-
if segment and segment != "docs"
614+
if segment
615615
]
616616

617617
# Initialize an empty list to store the breadcrumbs and their separators

docs/app/tests/test_breadcrumb.py

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
"""Tests for docs breadcrumb rendering."""
2+
3+
import sys
4+
from pathlib import Path
5+
6+
import reflex as rx
7+
8+
sys.path.append(str(Path(__file__).resolve().parent.parent))
9+
10+
11+
def test_breadcrumb_keeps_docs_segment_when_present_in_path():
12+
"""Breadcrumb segment mapping should not special-case the docs segment."""
13+
from reflex_docs.templates.docpage.docpage import breadcrumb
14+
15+
rendered = str(breadcrumb("/docs/ai/integrations/", rx.box()))
16+
17+
assert "Docs" in rendered
18+
assert "Ai" in rendered
19+
assert "Integrations" in rendered

0 commit comments

Comments
 (0)