ome / design

OME Design proposals
http://ome.github.io/design/
1 stars 15 forks source link

Scoping of Well-Field Grid View - Round 1 #49

Open gusferguson opened 8 years ago

gusferguson commented 8 years ago

There have been user requests to be able to view all the Fields in a Well as a grid.

This is partly because, increasingly, non-HCS data is being placed into Plate formats to aid with analysis etc. This leads to there often being different numbers of Fields per Well. The same can happen with Plate data when unwanted images are discarded to save space.

Requirements:

Additional requirements:

Current Status

Insight:

If Plate has Wells with > 1 Field:

Web:

All Plates:

Issues:

Insight:

Note: J-M: The grid view used to work nicely, recent changes in call might have introduced a bug. Preview was not in place when the feature was introduced and was overlooked. The concept of preview might have to be revisited (not now) for wells with multiple fields

Web:

Both:

Note: J-M: The label is stored in the DB, so that needs to be taken into account.

Insight

wellgridview-current-insight-1

wellgridview-current-insight-2

Web

wellgridview-current-web-1

wellgridview-current-web-2

Mockups

Points:

No more than one Field in any Well:

? if Field # drop-down should have a label: “Select:”

An annotation in this context would be linked to the Well - A1

Insight

wellgridview-one-field

Web

wellgridview-one-field-web

When any Well has > one Field:

wellgridview-view as-deselected

wellgridview-view-as-deselected-web

View Fields As Grid selected when Well A1 selected:

An annotation in this context would be linked to the Field - Field no. 1. Multi-selection of fields and batch annotation would be possible.

wellgridview-well

Note: Thumbnail zoom control shown in web

wellgridview-well-web

View Fields As Grid selected when Well A1 selected:

wellgridview-one-field 25

wellgridview-one-field 25-web

When Run selected but no individual Well selected

Fields in Wells separated by bar with label for context.

Note: Would probably need pagination.

An annotation in this context would be linked to the Field - Field #x. Multi-selection of fields across wells and batch annotation would be possible.

wellgridview-run-grid

wellgridview-run-grid-web

Drop-down for No. of Fields per row:

Probably practical to give options up to 10 and then “All”. (Web same as insight)

wellgridview-no-dd

Icon to indicate no Image - i.e. Well has no image for that Field #

Option 1 - black BG with 50% transparent “/” across standard Image icon.

wellgridview-no-image-4

wellgridview-no-image-web-3

Option 2 - centre pane BG colour with 50% transparent “/” across standard image icon.

wellgridview-no-image-3

wellgridview-no-image-web-2

Can tweak length of / etc.

Comments and feedback welcome.

pwalczysko commented 8 years ago

I like the white icon for "no image" better.

will-moore commented 8 years ago

I don't understand why we need a "Drop-down for No. of Fields per row". We used to provide something like that for images in a Dataset in Insight, but you don't need it for wrapping many images into a view. The wrapping just happens "naturally", and the number of images per row simply depends on the size of the images and the available space, both of which the user can adjust.

will-moore commented 8 years ago

We already have the ability to annotate at the Image level: When the appropriate field is chosen and the Well selected, the Well/Image is shown in the right, but all annotations there are on the Image. What we don't support is the reduced granularity of being able to annotate the Well, which is often what users want.

jburel commented 8 years ago

As suggested on the initial doc, we should have a text field to enter the number of elements per row not a drop down selection. @will-moore: We cannot use the same strategy that the one used to display dataset i.e. size of viewport since the display has to correspond to a specific grid layout e.g. 5X4 (discussed with users) for power of 2 number, this is easy otherwise we can use the viewport to have a "default" number of columns but the user needs to be able to change it.

jburel commented 8 years ago

Currently images are annotated not well samples, (that's what we opted to do since a well sample is just an image with location) Well annotation is missing from the clients i.e. cannot annotate wells

gusferguson commented 8 years ago

@jburel

As suggested on the initial doc, we should have a text field to enter the number of elements per row not a drop down selection.

Agree we need to be able to specify. The advantage of the drop-down is the it gives the "All" option - which I think people will want if they have unpredictable/inconsistent numbers of fields in wells. As a whole drop-downs are much more user friendly - all mouse: click - select - release vs. text box requires mouse and keyboard: click - type - return

jburel commented 8 years ago

What does "all" mean?

gusferguson commented 8 years ago

No wrapping.

will-moore commented 8 years ago

Not sure about drop-downs being user-friendly. E.g. http://www.formisimo.com/blog/why-are-drop-downs-and-select-boxes-bad-for-forms/ and http://www.lukew.com/ff/entry.asp?1950 A text field that is empty would not cause any wrapping (equivalent of "All") and this should be the default.

manics commented 8 years ago

What we don't support is the reduced granularity of being able to annotate the Well, which is often what users want.

Logically this is where annotations related to the sample such as experimental parameters belongs, since the whole well is treated.

gusferguson commented 8 years ago

@will-moore - both of those articles are in the context of forms and both cite the most problems with mobile devices - neither applicable in this case. Besides they are just opinion blog posts.

jburel commented 8 years ago

@gusferguson: we need wrapping and 10 will not be enough when we have a large number of values e.g. 700 fields. This will be difficult to "predict" if we use a drop-down That's why we use(d) in insight a text box in order to set the desire number (dataset view)

will-moore commented 8 years ago

My impression from multiple places (not just opinion blog posts) is that "select" is not user friendly: This is a good one: https://youtu.be/CUkMCQR4TpY?t=12m3s.

gusferguson commented 8 years ago

Mockups of text box replacing selection box.

Insight:

wellgridview-row-num-text-box-plate

wellgridview-row-num-text-box

Web:

wellgridview-row-num-text-box-plate-web

wellgridview-row-num-text-box-web

will-moore commented 8 years ago

Here's a mock-up of browsing Plate/Run -> Well (show few thumbs as preview) -> Show All (grid) -> Select image (loads in right panel). You can adjust how much the centre pane is split top/bottom depending on how big you want to show the plate vv fields, but the plate view always stays in the same location.

plate