Skip to content

Unify active section highlight and refine Advanced divider - #33

Merged
Lyptik merged 3 commits into
mainfrom
fix/sidebar-active-highlight-advanced-divider
Oct 2, 2026
Merged

Lyptik merged 3 commits into
mainfrom
fix/sidebar-active-highlight-advanced-divider

Conversation

@Lyptik

@Lyptik Lyptik commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up polish based on the latest mobile rendering.

Section selection

  • make every open top-level section use the same explicit selected/highlight background
  • remove reliance on Safari's transient tap/focus painting
  • keep a keyboard-visible focus state
  • preserve the subtle hover state on desktop

Advanced divider

  • indent Advanced slightly to the right
  • change the divider composition to:
    • short line
    • chevron + Advanced
    • long line
    • optional compact value summary
  • keep the whole row vertically aligned on Safari/mobile

Audit cleanup

While auditing the current sidebar code:

  • removed obsolete pre-accordion .panel styling
  • removed unused .text-button styling
  • removed unused .sticky-heading styling
  • removed an obsolete Advanced hint rule
  • found and fixed a real geometry conflict: the Debug sticky-header rule was still overriding shared summary padding, which could make a collapsed Debug row differ in height from the other collapsed sections
  • checked the responsive breakpoint: CSS and JS use the same phone/touch-device condition
  • checked the current Pages workflow and latest main deployment; both are healthy
  • no unused CSS classes remain in the current HTML/JS surface after this cleanup

CSS-only UI change. No tracking, connection, rendering, persistence, or data-processing behavior is changed.

@Lyptik
Lyptik merged commit 6e59738 into main Oct 2, 2026
1 check passed
@Lyptik
Lyptik deleted the fix/sidebar-active-highlight-advanced-divider branch October 2, 2026 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant