Skip to content

0.9.0 - #61

Merged
thomasgallagher merged 7 commits into
mainfrom
release
Apr 26, 2026
Merged

0.9.0#61
thomasgallagher merged 7 commits into
mainfrom
release

Conversation

@thomasgallagher

Copy link
Copy Markdown
Collaborator

No description provided.

@claude

claude Bot commented Apr 26, 2026

Copy link
Copy Markdown

Review

This PR bundles several independent improvements: a l_ui_title_bar helper (combining breadcrumbs, h1, and optional actions into a single call), a shared scroll_lock.js utility with reference counting (replacing duplicated scroll-lock logic in the navigation and panel controllers), color-scheme declarations for native browser UI elements, a l-ui-hr horizontal rule class, and cursor/pointer additions for checkbox and radio inputs.

No issues found.

Notes

  • scroll_lock.js reference counting is correct. Module-level lockCount and savedScrollY survive Turbo navigations, but both controllers call unlockScroll() in disconnect(), which is invoked on navigation - no stale lock state can persist.
  • l_ui_title_bar helper - content_tag(:h1, title, ...) HTML-escapes the title, and safe_join handles the actions array correctly. breadcrumb.fetch(:label) fails loudly on a missing key, which is the right behaviour.
  • Both actions: and a block silently prefers the block, which is documented as intended.

Ship

@thomasgallagher
thomasgallagher merged commit 5827eff into main Apr 26, 2026
3 checks passed
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