RADY: Usability improvement for ignored series show/hide toggle

IUSCA / SQAN

Scalable Quality Assurance for Neuroimaging - (SQAN): A full-stack system solution for extracting, translating, and logging (ETL) + web portal-based quality-control verification of DICOM-formatted medical imaging data/metadata.

https://sqan.sca.iu.edu/

Other

8 stars 3 forks source link

RADY: Usability improvement for ignored series show/hide toggle #76

Closed agopu closed 5 years ago

agopu commented 5 years ago

For e.g.

[REDACTED] See for screenshot.

Let's consider doing the following

Grey out the SeriesDescription for ignored series (the rows on which the QC status background color is lightened )
Make the Show/Hide toggle a 3-way toggle with a tooltip informing the user about this so we can hide the QC status or we can hide the ignored Series (rows) altogether

FYI @aiavenak , I sent this to @youngmd

Edited by @agopu on 2019-08-01 - moved screenshots with and/or information stakeholder prefers to keep private to an issue on that project's private repo.

agopu commented 5 years ago

@youngmd I've looked but can't find a ticket update about this - perhaps we discussed it in our 1:1 (or on Slack) and we both have been confusing that with a ticket update? Let's screenshot the update and move this to QA (IIRC, we had to pull a different branch on rady-test to be able to do this?); and let's deploy to prod asap unless there's a reason not.

youngmd commented 5 years ago

Screenshots depicting the 3 states and their effects:

[REDACTED] See for screenshot.

Merging into develop branch and will merge into master along with a few other small items

Edited by @agopu on 2019-08-01 - moved screenshots with and/or information stakeholder prefers to keep private to an issue on that project's private repo.

youngmd commented 5 years ago

This has been pushed to production.

agopu commented 5 years ago

@jdwest0711 / @mparamas please be sure to confirm this via your own tests.

May need a hard refresh...

agopu commented 5 years ago

See https://github.com/IUSCA/rady-ops/issues/25 for screenshots.