Skip to content

feat(ui): make the sidebar brand link to Home - #561

Merged
smunini merged 3 commits into
mainfrom
552-ui-make-the-sidebar-logo-link-to-the-home-page
Aug 18, 2026
Merged

feat(ui): make the sidebar brand link to Home#561
smunini merged 3 commits into
mainfrom
552-ui-make-the-sidebar-logo-link-to-the-home-page

Conversation

@Gordex2014

@Gordex2014 Gordex2014 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Closes #552.

Summary

Makes the sidebar brand a native link to /ui, covering the logo, app name, and version. Because this is a regular <a href>, it preserves full-page browser navigation, keyboard activation, Ctrl/Cmd-click, and operation without JavaScript.

The logo remains decorative, while the link has a localized accessible name that remains available when the sidebar rail is collapsed. The existing Home navigation item remains the only element with aria-current="page".

Adds restrained hover and focus-visible affordances plus a no-JavaScript Playwright regression covering the native href, accessible name, decorative image, keyboard activation, and navigation to Home.

Testing

  • HFS build with the UI enabled — passed
  • git diff --check — passed
  • cargo test -p helios-ui — 144 passed
  • No-JavaScript Playwright suite — 9 passed
  • WCAG 2.2 AA suite in light and dark themes — 20 passed
  • Latest GitHub Actions: Test Rust, Linting, Security Audit, UI browser suite, Test Python, Test FHIRPath, Code Coverage, Codecov patch, and Cleanup Test Containers — passed

🤖 Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
@Gordex2014
Gordex2014 requested a review from smunini August 18, 2026 04:57
@Gordex2014 Gordex2014 linked an issue Aug 18, 2026 that may be closed by this pull request
6 tasks
Gordex2014 and others added 2 commits August 18, 2026 10:13
@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Gordex2014

Copy link
Copy Markdown
Contributor Author

Branch synchronization and CI follow-up:

The PR description has been updated with the current successful test results.

@smunini
smunini merged commit 132e19d into main Aug 18, 2026
20 checks passed
@smunini
smunini deleted the 552-ui-make-the-sidebar-logo-link-to-the-home-page branch August 18, 2026 16:18
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.

UI: make the sidebar logo link to the Home page

2 participants