Skip to content

fix: use real links for dashboard navigation - #994

Open
ppeerttu wants to merge 3 commits into
mainfrom
fix/dashboard-real-links
Open

fix: use real links for dashboard navigation#994
ppeerttu wants to merge 3 commits into
mainfrom
fix/dashboard-real-links

Conversation

@ppeerttu

@ppeerttu ppeerttu commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

Sidebar items and rows in the bot, calendar, and calendar event lists navigate via hx-get on elements without an href. That means they can't be opened in a new tab, middle-clicked, or copied as links.

This switches them to real anchors with hx-boost. Normal clicks still swap only the content pane as before; modified clicks are left to the browser. Row clicks now ignore clicks on links inside the row so a click on the new anchor doesn't fire twice.

Templates only, no server changes.

Screen recording

Demonstrating the nav links and bot listing support properly e.g. opening on new tab.

Screen.Recording.2026-09-07.at.10.28.54.mov

@ppeerttu ppeerttu changed the title Use real links for dashboard navigation fix: use real links for dashboard navigation Sep 7, 2026
@ppeerttu
ppeerttu marked this pull request as ready for review September 7, 2026 13:12
@ppeerttu
ppeerttu requested a review from a team as a code owner September 7, 2026 13:12
@ppeerttu
ppeerttu requested a review from noah-duncan September 7, 2026 13:13
Sidebar nav items and list rows (bots, calendars, calendar events)
used hx-get on elements without an href, so they could not be opened
in a new tab or copied as links. Replace them with real anchors and
hx-boost, which keeps the existing partial content swap for normal
clicks while leaving modified clicks to the browser.
The inline click handler marked a nav item active on any click,
including cmd/ctrl-click, which opens a new tab and leaves the current
page unchanged. Update the active item from the current URL on load
and after htmx pushes or restores history.
Bootstrap tab buttons in page content also use .nav-link, so the
unscoped selector cleared their active state on load and history
restore. Bump version to 1.72.1.
@ppeerttu
ppeerttu force-pushed the fix/dashboard-real-links branch from fd3190a to 53183c8 Compare September 10, 2026 12:19
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