Skip to content

Add tooltip for ro-crate#64

Merged
will-moore merged 2 commits intoome:mainfrom
lubianat:ro_crate_note
Nov 17, 2025
Merged

Add tooltip for ro-crate#64
will-moore merged 2 commits intoome:mainfrom
lubianat:ro_crate_note

Conversation

@lubianat
Copy link
Copy Markdown
Contributor

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 17, 2025

Deploy Preview for ome-ngff-validator ready!

Name Link
🔨 Latest commit b395a09
🔍 Latest deploy log https://app.netlify.com/projects/ome-ngff-validator/deploys/691b2a2ba045370008877e31
😎 Deploy Preview https://deploy-preview-64--ome-ngff-validator.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@imagesc-bot
Copy link
Copy Markdown

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/help-with-ro-crate-metadata-for-validating-ome-zarr-v0-5/117684/6

@will-moore
Copy link
Copy Markdown
Member

The tooltip looks great, thanks.

I wonder if you could also include the change I made in #48 of moving the <h1>RO-Crate metadata</h1> title under the {#if jsonData} block so that it's not shown if there's no RO-Crate. I think the tooltip will be fine alongside the "No RO-Crate metadata.json message. Hope that's not too painful to change?

@lubianat
Copy link
Copy Markdown
Contributor Author

Something like that?

@will-moore
Copy link
Copy Markdown
Member

Looks nice:

Screenshot 2025-11-17 at 14 16 28

And with Ro-Crate:

Screenshot 2025-11-17 at 14 16 18

Copy link
Copy Markdown
Member

@will-moore will-moore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@will-moore will-moore merged commit 01ca6c8 into ome:main Nov 17, 2025
5 checks 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.

3 participants