Skip to content

Add a manufacturer landing page to the probe viewer#64

Merged
alejoe91 merged 7 commits into
SpikeInterface:mainfrom
h-mayorquin:improvements
Jun 26, 2026
Merged

Add a manufacturer landing page to the probe viewer#64
alejoe91 merged 7 commits into
SpikeInterface:mainfrom
h-mayorquin:improvements

Conversation

@h-mayorquin

Copy link
Copy Markdown
Collaborator

The viewer opened straight onto a hard-coded default probe, giving no overview of what the catalog holds. This adds a landing page at the root: one card per manufacturer (logo and probe count). Selecting a card enters the existing sidebar-and-viewer on that manufacturer's first probe, and a Home control in the sidebar returns to the landing. Browsing within a manufacturer is unchanged; this only changes what you see before picking one.

I also made a few other viewer improvements on this branch. Probes now open fitted to their contacts (a "Full Contacts View", used as the default) rather than to the whole probe outline, which matters for probes whose electrodes occupy a small band of a long shank. The maximum zoom is now computed per probe so the smallest contact can roughly fill the viewport, instead of a single global constant that meant very different things for a 3 mm probe versus a 90 mm one. And the shared-link URL now carries the view toggles (contact IDs, scale bar, overview), so a shared view reproduces what the sender was looking at. These are independent of the landing page, so I can split them into their own pull request if you prefer.

On the logos: the six cards use the manufacturers' own marks to identify their products in the catalog (nominative use). A brand-colored wordmark fallback remains in the code, so any logo can be dropped without affecting the layout.

@alejoe91 alejoe91 marked this pull request as ready for review June 26, 2026 08:41
@alejoe91 alejoe91 merged commit dca31ae into SpikeInterface:main Jun 26, 2026
1 check 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.

2 participants