feat(client): site bonus — click site bar to open Moorhen + ribbons in Site tab - #40
Merged
Merged
Conversation
…n Site tab
1) Click a bar on the 'events per site' chart to deep-link into the Moorhen
viewer focused on that site. SummaryCharts gains onSiteClick (clicked bar
index -> site_num; pointer cursor on hover); ProjectDashboard navigates to
/projects/:id/inspect?site=N; InspectPage reads the param and the drawer
consumes it once datasets+Coot are ready — switching to the Site tab and
loading that site's first event.
2) In the Site tab the protein renders as ribbons (CRs) + the bound ligand as
sticks ('ligands') for spatial context while scanning sites, instead of
bonds (CBs). Chosen via an axisRef at model-load time; other tabs keep CBs.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two site-focused bonus features (they compose: clicking a site bar lands you in Site mode → ribbons).
1) Click a site bar → open Moorhen at that site
SummaryCharts: the "events per site" bar gains anonClick→onSiteClick(site_num)(clicked bar index →sites[index]), pointer cursor on hover.ProjectDashboard: wires it tonavigate('/projects/:id/inspect?site=N').InspectPage: reads?site=and passesinitialSiteto the drawer.InspectDrawer: a consume-once effect waits for datasets + Coot, then switches to the Site tab and loads that site's first event.2) Ribbons in the Site tab
At model load, when the tab is Site, the protein draws as
"CRs"(ribbons) +"ligands"(bound ligand as sticks); other tabs keep"CBs". Chosen via anaxisRef.Notes
Verification
tsc clean. Verified in-app ("excellent").
🤖 Generated with Claude Code