Skip to content

Comments

feat: data link code snippets dialog#266

Draft
allison-truhlar wants to merge 38 commits intomainfrom
data-link-code-snippets-dialog
Draft

feat: data link code snippets dialog#266
allison-truhlar wants to merge 38 commits intomainfrom
data-link-code-snippets-dialog

Conversation

@allison-truhlar
Copy link
Collaborator

@allison-truhlar allison-truhlar commented Dec 12, 2025

Clickup ids: 86acuxatq, 86accr60r, 86aeq5001

This PR adds a data link usage dialog with copyable code snippets and instructions for opening data links in external tools. The dialog is organized into tabs by tool/language (Fiji, Java, Napari, Python) and is context-aware — showing different content for zarr, N5, and directory data types. In the dialog header, viewable no matter which tab is selected, is a copy data link box. The dialog is accessible from the zarr/N5 preview panels, the properties drawer, and the data links table.

This PR also redesigns the data tool link icons (Neuroglancer, Vol-E, Avivator, etc.) as labeled circular buttons with hover effects, replacing the previous ButtonGroup layout. It adds the "More ways to open" button, displayed as a three-dot "More..." icon in the Zarr/N5 preview panels and a button with text in the properties panel, that opens the data link usage dialog.

Finally, the PR adds two reusable dialog button components (DialogIconBtn, DialogTextBtn).

@krokicki @neomorphic @cgoina

- also fixes location of the copy icon for the code blocks
- in the DataLinkUsageDialog, the copied message was not showing due to a CSS issue. However, in addressing this, I realized that this component was recreating logic from the CopyTooltip component and reworked it to use that component
@allison-truhlar allison-truhlar marked this pull request as ready for review January 22, 2026 14:24
@allison-truhlar allison-truhlar requested review from neomorphic and removed request for neomorphic January 22, 2026 14:50
@allison-truhlar allison-truhlar marked this pull request as draft January 22, 2026 15:50
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