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

iViewer UI #75

Open
will-moore opened this issue Apr 25, 2017 · 16 comments
Open

iViewer UI #75

will-moore opened this issue Apr 25, 2017 · 16 comments

Comments

@will-moore
Copy link
Member

Some proposed cleanup & future feature prototyping for iViewer.
Generally aiming to reduce 'ink' on the page to give a cleaner look.

cc @waxenegger @jburel

screen shot 2017-04-25 at 16 25 23

screen shot 2017-04-25 at 16 31 02

@jburel
Copy link
Member

jburel commented Apr 25, 2017

scale bar location has been raised as a problem so we should try to keep it in the "bottom-left corner" of the image not the viewport

@jburel
Copy link
Member

jburel commented Apr 26, 2017

To avoid confusion, User settings with the list of thumbnails will still be displayed in "settings"

@jburel
Copy link
Member

jburel commented Apr 26, 2017

Discussed today: look into making the scalebar draggable

@waxenegger
Copy link
Member

Should the dataset image side panel on the left still be resizable/collapsible?

@waxenegger
Copy link
Member

waxenegger commented May 3, 2017

@jburel and @will-moore

Is the only way to turn off rois the checkbox on the header of the table....

This is going to be a bit of a technical nightmare, the reason being that instead of being able to turn off the vector layer altogether in ol3, I keep it on but make all shapes "invisible" (by tricking ol3 with filtering). Essentially all the workflow and code paths will be still active. Originally the visibility was more a means to turn off individual shapes, for hiding the rois altogether it's not a great solution but will be one that's inevitable if we still show the table and allow for some actions. There are various events in place to guarantee that after some action such as selection or any modification the state of the shapes are sync with the embedded ol3 viewer. Now what that means is that I cannot allow for certain actions such as delete or modify or copy and paste or attachment locks without having the vector layer on. shapes or features as their counterpart in ol3 are inextricably linked to the vector layer and that makes good sense because without that layer there is no means of working with them.

So if we plan on showing the table while every shape in it is not visible to the user allowing for some actions on invisible rois, the only one that really makes sense to me being delete, not sure of copy & pasting and modifying invisible shapes....then I'm going to have to disable some actions at a minimum.

Another thing is that the columns of the roi table are actually resizable and the checkbox can be pushed over.

@will-moore
Copy link
Member Author

As discussed, unchecking the Show in table header will simply hide every shape in the viewer but you can still interact with them in the table, delete them etc. If a user draws a new shape it will be visible and the header Show will become checked (same as if user manually checks Show on one or more shapes).

@jburel
Copy link
Member

jburel commented May 14, 2017

That is not what is currently implemented.
The Show in the header is only ticked if all checkboxes are checked

@will-moore
Copy link
Member Author

Some ideas on how to show more measurements and/or annotations on Shapes:

Small changes: move Show column to left, re-order toolbars so that those showing/editing selected shapes are closer to the shapes themselves. Also adds a 'wrench' to popup a checklist of which columns to show.
screen shot 2017-06-23 at 14 21 07

Choose to show Length and Area for shapes.
NB: we don't show Comments, so the Comments toolbar is not shown.

screen shot 2017-06-23 at 14 21 07

Now we show Tags column. This shows a summary of tags for each shape and also adds the Tags toolbar where we can see full tag list and Add / Remove tags. Using bootstrap's "badges" to style tags. Can show more info on a tool-tip for each tag (at least in the toolbar - maybe in the table too?). TODO: add a "remove" icon for each tag in the toolbar.

screen shot 2017-06-23 at 14 21 27

cc @jburel @waxenegger

@jburel
Copy link
Member

jburel commented Jun 23, 2017

For any measurements we will need units and maybe ability to switch between for example pixels and microns

@pwalczysko
Copy link
Member

Export intensities in the ROI to somewhere - is that planned to be in a new tab ?

@jburel
Copy link
Member

jburel commented Jun 26, 2017

Idea is to export measurement as csv
I don't think displaying the intensity as it is in insight is that useful

@pwalczysko
Copy link
Member

@jburel : Agreed. I was just thinking about where the export button would live.

@jburel
Copy link
Member

jburel commented Jun 26, 2017

We will also need a way to trigger the analysis of the selected shapes
In that case, do we clean the results already displayed for the shapes not in the "active" selection?

@will-moore
Copy link
Member Author

Probably add an "Export/Download" button beside the Save/Undo/Redo buttons at the top.
But I'm not sure how we'd display pixel intensities differently from shape dimensions. I think we would want to display them in the shapes table in the same way (not just download them) and this wouldn't be too hard if the server could give us intensities for specified shapes. It gets tricky when we have a large number of shapes or they cover a large area. Maybe we can use cut-offs to say "If more than 50 shapes or more than 1 000 000 pixels in total area then we're not going to show you intensities and you have to download them"?

@pwalczysko
Copy link
Member

@will-moore : Intensities ? You mean to show average intensity per ROI ? You would not have a space for showing even all intensities from relatively small ROIs ?

@jburel
Copy link
Member

jburel commented Jun 26, 2017

Export might be more suitable since we were thinking of having a Download button on the info tab

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

4 participants