ome / omero-web

Django-based OMERO.web client
https://www.openmicroscopy.org/omero
16 stars 29 forks source link

Archive status #555

Closed chris-allan closed 2 months ago

chris-allan commented 4 months ago

Adds archive status to the tree metadata, visually in the tree and to image data JSON.

image

jburel commented 4 months ago

This PR leads to some test failures see https://merge-ci.openmicroscopy.org/jenkins/job/OMERO-test-integration/64/

will-moore commented 4 months ago

Tested on merge ci with $ omero obj update Image:1673 archived=True And this displays correctly in webclient as shown above.

My only question is whether users need some clue as to what the red dot means. I guess this might not be necessary if all users have been trained accordingly, (which may well happen in places that are using archiving, since users should know to expect different behaviour from those images?), but it seems that there could be some who are confused by the red dot. Possibly a tooltip on the red-dot icon (might be a bit painful digging into how to do this in jstree) or in the right panel core_metadata.html add a row Archived: true (only if True - don't need to add Archived: false for all other images) or something else?

chris-allan commented 4 months ago

My only question is whether users need some clue as to what the red dot means. I guess this might not be necessary if all users have been trained accordingly, (which may well happen in places that are using archiving, since users should know to expect different behaviour from those images?), but it seems that there could be some who are confused by the red dot. Possibly a tooltip on the red-dot icon (might be a bit painful digging into how to do this in jstree) or in the right panel core_metadata.html add a row Archived: true (only if True - don't need to add Archived: false for all other images) or something else?

Likely. I only used image_locked16.png because it was available. I expect the red-dot is not particularly informative and this deserves a dedicated icon and a tooltip. Can also add something to core_metadata.html as a demonstration of what's possible.

chris-allan commented 4 months ago

Added right-hand panel visual:

image

chris-allan commented 4 months ago

Added a tooltip to the tree (visuals are from Windows and will look slightly different depending on operating system):

image

chris-allan commented 4 months ago

@jburel: ome/openmicroscopy#6389 should resolve the test failures.

lunson commented 4 months ago

Options for iconography icon_archived

and the image files for each of the options: icons_archive.zip

will-moore commented 4 months ago

A couple of other test failures from the last few days which seem to come from this PR: https://merge-ci.openmicroscopy.org/jenkins/job/OMERO-test-integration/81/testReport/

chris-allan commented 4 months ago

Thanks, @will-moore. Hopefully 97e479b717e34179eceea2507d3e69a04b2fec1e will get that test failure and also fix the display of the archive status when the tag tree is being displayed. Need to test that a little more thoroughly myself.

chris-allan commented 4 months ago

Display of archive status in the tag tree:

image

will-moore commented 3 months ago

Tests passing today 👍 https://merge-ci.openmicroscopy.org/jenkins/job/OMERO-test-integration/82/testReport/

jburel commented 2 months ago

Discussed today with @sbesson and @knabar, option A from @lunson has been selected.

knabar commented 2 months ago

New icon looks good:

Screenshot 2024-07-29 at 10 47 18
imagesc-bot commented 2 months ago

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

https://forum.image.sc/t/releases-of-omero-server-5-6-12-and-omero-web-5-27-0/99795/1