From a33611af3132c9878da3547b3a26faaab5511bd4 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Sat, 11 Jul 2026 22:41:11 +0000
Subject: [PATCH 1/2] feat(ux): improve drilldown navigation accessibility

- Replace generic div container with semantic `<nav aria-label="Breadcrumb">`
- Add `aria-current="step"` to indicate active level
- Add `aria-live="polite"` to dynamic level counter to ensure announcements
- Add `aria-hidden="true"` to decorative navigation icons
- Add visible focus styles for keyboard navigation (`focus-visible`)
- Document learnings in `.Jules/palette.md`

Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
---
 .Jules/palette.md                             |  3 +++
 .../drilldown/drilldown-navigation.tsx        | 22 +++++++++++--------
 2 files changed, 16 insertions(+), 9 deletions(-)
 create mode 100644 .Jules/palette.md

diff --git a/.Jules/palette.md b/.Jules/palette.md
new file mode 100644
index 00000000..fbeb6c05
--- /dev/null
+++ b/.Jules/palette.md
@@ -0,0 +1,3 @@
+## 2026-07-11 - Drilldown Navigation Accessibility
+**Learning:** Drilldown components that use a sequence of breadcrumb-like buttons inside a generic `<div>` are not recognized as navigation by screen readers, and dynamic level counters are often missed.
+**Action:** Always wrap breadcrumb-style sequences in `<nav aria-label="Breadcrumb">`, mark the current level with `aria-current="step"`, and use `aria-live="polite"` on dynamic level text indicators to ensure screen readers announce changes as users navigate deeper.
diff --git a/src/components/drilldown/drilldown-navigation.tsx b/src/components/drilldown/drilldown-navigation.tsx
index 9489f7f8..912e3048 100644
--- a/src/components/drilldown/drilldown-navigation.tsx
+++ b/src/components/drilldown/drilldown-navigation.tsx
@@ -56,7 +56,10 @@ export function DrilldownNavigation({
   return (
     <div className={clsx('flex flex-col', className)}>
       {/* Navigation Breadcrumb */}
-      <div className="flex items-center gap-2 mb-4 text-sm text-neutral-400">
+      <nav
+        aria-label="Breadcrumb"
+        className="flex items-center gap-2 mb-4 text-sm text-neutral-400"
+      >
         {levels.map((level, idx) => (
           <div key={level.id} className="flex items-center gap-2">
             <button
@@ -64,17 +67,18 @@ export function DrilldownNavigation({
                 setCurrentLevel(idx)
                 onLevelChange?.(level.id)
               }}
+              aria-current={idx === currentLevel ? 'step' : undefined}
               className={clsx(
-                'hover:text-neutral-200 transition-colors',
+                'rounded hover:text-neutral-200 transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-neutral-400',
                 idx === currentLevel && 'text-neutral-200 font-medium'
               )}
             >
               {level.title}
             </button>
-            {idx < levels.length - 1 && <ChevronRight className="size-4" />}
+            {idx < levels.length - 1 && <ChevronRight className="size-4" aria-hidden="true" />}
           </div>
         ))}
-      </div>
+      </nav>
 
       {/* Current Level Content */}
       <div className="flex-1">{current.content}</div>
@@ -85,18 +89,18 @@ export function DrilldownNavigation({
           onClick={handleBack}
           disabled={currentLevel === 0}
           className={clsx(
-            'flex items-center gap-2 px-4 py-2 rounded-md text-sm font-medium transition-opacity',
+            'flex items-center gap-2 px-4 py-2 rounded-md text-sm font-medium transition-opacity focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-neutral-400',
             'hover:opacity-80',
             currentLevel === 0
               ? 'opacity-50 cursor-not-allowed text-neutral-500'
               : 'text-neutral-300'
           )}
         >
-          <ChevronLeft className="size-4" />
+          <ChevronLeft className="size-4" aria-hidden="true" />
           Back
         </button>
 
-        <div className="text-xs text-neutral-500">
+        <div className="text-xs text-neutral-500" aria-live="polite">
           Level {currentLevel + 1} of {levels.length}
         </div>
 
@@ -104,7 +108,7 @@ export function DrilldownNavigation({
           onClick={handleNext}
           disabled={currentLevel === levels.length - 1}
           className={clsx(
-            'flex items-center gap-2 px-4 py-2 rounded-md text-sm font-medium transition-opacity',
+            'flex items-center gap-2 px-4 py-2 rounded-md text-sm font-medium transition-opacity focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-neutral-400',
             'hover:opacity-80',
             currentLevel === levels.length - 1
               ? 'opacity-50 cursor-not-allowed text-neutral-500'
@@ -112,7 +116,7 @@ export function DrilldownNavigation({
           )}
         >
           Next
-          <ChevronRight className="size-4" />
+          <ChevronRight className="size-4" aria-hidden="true" />
         </button>
       </div>
     </div>

From be75354dc40b3b799ea58d0c9e0fa70e849d1591 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Sat, 11 Jul 2026 22:55:04 +0000
Subject: [PATCH 2/2] feat(ux): improve drilldown navigation accessibility

- Replace generic div container with semantic `<nav aria-label="Breadcrumb">`
- Add `aria-current="step"` to indicate active level
- Add `aria-live="polite"` to dynamic level counter to ensure announcements
- Add `aria-hidden="true"` to decorative navigation icons
- Add visible focus styles for keyboard navigation (`focus-visible`)
- Document learnings in `.Jules/palette.md`

Co-authored-by: aarjava <218419324+aarjava@users.noreply.github.com>
---
 .Jules/palette.md                                 | 1 +
 src/components/drilldown/drilldown-navigation.tsx | 5 +----
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/.Jules/palette.md b/.Jules/palette.md
index fbeb6c05..3756a698 100644
--- a/.Jules/palette.md
+++ b/.Jules/palette.md
@@ -1,3 +1,4 @@
 ## 2026-07-11 - Drilldown Navigation Accessibility
+
 **Learning:** Drilldown components that use a sequence of breadcrumb-like buttons inside a generic `<div>` are not recognized as navigation by screen readers, and dynamic level counters are often missed.
 **Action:** Always wrap breadcrumb-style sequences in `<nav aria-label="Breadcrumb">`, mark the current level with `aria-current="step"`, and use `aria-live="polite"` on dynamic level text indicators to ensure screen readers announce changes as users navigate deeper.
diff --git a/src/components/drilldown/drilldown-navigation.tsx b/src/components/drilldown/drilldown-navigation.tsx
index 912e3048..acbc348f 100644
--- a/src/components/drilldown/drilldown-navigation.tsx
+++ b/src/components/drilldown/drilldown-navigation.tsx
@@ -56,10 +56,7 @@ export function DrilldownNavigation({
   return (
     <div className={clsx('flex flex-col', className)}>
       {/* Navigation Breadcrumb */}
-      <nav
-        aria-label="Breadcrumb"
-        className="flex items-center gap-2 mb-4 text-sm text-neutral-400"
-      >
+      <nav aria-label="Breadcrumb" className="flex items-center gap-2 mb-4 text-sm text-neutral-400">
         {levels.map((level, idx) => (
           <div key={level.id} className="flex items-center gap-2">
             <button