Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add (optional) description field to view #50

Open
constantinpape opened this issue May 31, 2021 · 12 comments
Open

Add (optional) description field to view #50

constantinpape opened this issue May 31, 2021 · 12 comments

Comments

@constantinpape
Copy link
Contributor

We should add an (optional) field description to view in order to support more detailed information on views, e.g. for figure legends.
(Suggested by @tischi in a google doc on MoBIE project ideas.)
This is very easy on the spec side, however the question is how to display this in Fiji.
I think a good option would be to have it like a tool tip when hovering over the view menu item, but I am not sure if that's feasible in the fiji UI.

@tischi
Copy link
Contributor

tischi commented May 31, 2021

In addition to the too tip (which should be possible in Java Swing), I think it would be cool if the view legend was visible within the BDV window. In BDV one can have text overlays. I think having an additional window would create too much clutter...

@KateMoreva
Copy link

AbstractSourceDisplay now has a String sourcesDescription that could be filed (from the metadata or maybe we should think about an extra window for that also).

Implemented in: mobie/mobie-viewer-fiji#416

@constantinpape
Copy link
Contributor Author

AbstractSourceDisplay now has a String sourcesDescription that could be filed

Ok, I think filling the sourcesDescription (shouldn't it be called sourceDescription?) from the description field in the metadata is a good idea. I will create an example for that and post it here once I have it.

@constantinpape
Copy link
Contributor Author

@KateMoreva I have added a dataset with description for all the sources. You can find it both on the group share or github:

@tischi
Copy link
Contributor

tischi commented Sep 6, 2021

@KateMoreva @constantinpape

I think this issue could be another one with a relatively high priority to work on. What do you think? Personally I feel this could be even more important than the custom grid view.

For instance, imagine people browsing the bookmarks of the platybrowser and we could provide a link to the published figure. I think that'd be quite awesome and useful.

In fact, I found out that one can link to the figures of the Cell publication:

https://www.cell.com/cell/fulltext/S0092-8674(21)00876-X#fig1

We should also think about adding instructions for each source of how to download the full raw data.

@KateMoreva
Copy link

@KateMoreva @constantinpape

I think this issue could be another one with a relatively high priority to work on. What do you think? Personally I feel this could be even more important than the custom grid view.

For instance, imagine people browsing the bookmarks of the platybrowser and we could provide a link to the published figure. I think that'd be quite awesome and useful.

In fact, I found out that one can link to the figures of the Cell publication:

https://www.cell.com/cell/fulltext/S0092-8674(21)00876-X#fig1

We should also think about adding instructions for each source of how to download the full raw data.

Work over this issue is in progress, I think by the end of the week it will be fully functional.

@constantinpape
Copy link
Contributor Author

Just as a side-note, I think the description we are adding here is for sources instead of views.
But I think we should just go ahead with this as is and it should be fairly straightforward to implement the same mechanism for views once we have it for sources.

@KateMoreva
Copy link

Just as a side-note, I think the description we are adding here is for sources instead of views.
But I think we should just go ahead with this as is and it should be fairly straightforward to implement the same mechanism for views once we have it for sources.

@constantinpape, have we got a project with a views description?

@constantinpape
Copy link
Contributor Author

@constantinpape, have we got a project with a views description?

Not yet, but I will create one.

@constantinpape
Copy link
Contributor Author

@KateMoreva I added descriptions for all the views in the same dataset as before:

@KateMoreva
Copy link

KateMoreva commented Sep 10, 2021

@KateMoreva I added descriptions for all the views in the same dataset as before:

@constantinpape
Thank you! Just a small question, at the moment for the views we want to have a simple tooltip with the description or labels over images like in https://www.cell.com/cell/fulltext/S0092-8674(21)00876-X#fig1?

@constantinpape
Copy link
Contributor Author

Just a small question, at the moment for the views we want to have a simple tooltip with the description

Let's start with tooltips for now and then have a look at it together next week to discuss if we want some more advanced functionality.

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

No branches or pull requests

3 participants