ImagingDataCommons / IDC-WebApp

Web Application front end for IDC (CORE REPO)
Apache License 2.0
6 stars 2 forks source link

Windows OS - selection of cases/studies/series is not obvious #522

Closed ulrikew closed 3 years ago

ulrikew commented 3 years ago

It is difficult to select multiple items in a Window environment. Holding the Shift key highlights other rows or highlights the header. A selection can only be done one item at the time. image

s-paquette commented 3 years ago

Shift and Control select are reversed; preventDefault() needed to stop the normal drag behavior

G-White-ISB commented 3 years ago

Proposed behavior is that holding CNTRL or COMMAND(MAC) toggles a row without changing the selection of other rows. Without holding CNTRL, selecting one row will deselect all other rows. Holding the shift key will allow multiple sequential rows to be selected at one time.

madelyngreyes commented 3 years ago

Issue 4.13 in Tin's Test Report is addressed in this ticket.

fedorov commented 3 years ago

Issue 4.13 in Tin's Test Report is addressed in this ticket.

To be precise, specific issue identified by Tin is the following:

Improvement: Awkward selection of Cases for a selected Collection. If I wanted to select every other case under a Collection, I would have to hold down the Shift key. In TCIA portal for each Case I would click on the Cart Icon. I think this is also a 508 violation having to hold the shift key to perform random selection. Also, it would be tedious for a Collection having over a thousand cases. This same behavior is also exhibited for Studies and Series box.

I would strongly prefer not too have selection using alt-keys, and instead have checkboxes, buttons etc. I can't comment on 508, but I agree with Tin's suggestion overall.

madelyngreyes commented 3 years ago

I shift+selected two studies, the total count for the series should be 20 entries but, only 15 entries appear. I need to re-select the first option to get the total count of 20 entries to appear in the selected series panel.

Screen Shot 2021-02-19 at 9 58 21 AM

fedorov commented 3 years ago

Issue 4.13 in Tin's Test Report is addressed in this ticket.

@madelyngreyes the issue above is not addressed in this issue. Can you please create a new issue for not using an alt-key for selection:

Improvement: Awkward selection of Cases for a selected Collection. If I wanted to select every other case under a Collection, I would have to hold down the Shift key. In TCIA portal for each Case I would click on the Cart Icon. I think this is also a 508 violation having to hold the shift key to perform random selection. Also, it would be tedious for a Collection having over a thousand cases. This same behavior is also exhibited for Studies and Series box.

s-paquette commented 3 years ago

There doesn't seem to be any violation: https://www.w3.org/TR/WCAG20/#keyboard-operation-keyboard-operable

There's an explicit exception for timing based on movement. That said, we can definitely add in a checkbox.

s-paquette commented 3 years ago

@fedorov @ulrikew @madelyngreyes This is now available on test.

fedorov commented 3 years ago

I am not a 508 compliance expert. @ulrikew is there any chance Tin can be included in this discussion?

s-paquette commented 3 years ago

@ulrikew @fedorov It'd be preferable for any 508 concerns to be addressed in their own ticket, as that wasn't the specified issue in this ticket. (Trying to avoid scope creep in specific task items.) We have a 508 ticket in #555, which would be where we should focus that discussion.

ulrikew commented 3 years ago

Has the decision been made to have checkboxes as mentioned in #555?

s-paquette commented 3 years ago

@ulrikew Yes, that'll be done in hopefully Sprint 19 (depends on our workload otherwise).

madelyngreyes commented 3 years ago

Number of entries selected in the series panel matches the number of series' selected in the selected studies panel in the screenshot below,

Screen Shot 2021-03-01 at 9 13 53 AM
G-White-ISB commented 3 years ago

@ulrikew and @fedorov, are clicking the checkboxes going to be the only way to select a row or can one still highlight the row itself?

fedorov commented 3 years ago

I am fine either way.

fedorov commented 3 years ago

Am I the only one finding it not particularly intuitive or usable that the only way to un-select individual rows is to do control-click for those already selected?

G-White-ISB commented 3 years ago

?? That's not how its supposed to work, nor how its working in my experience (Mac, Safari or Chrome). If you click a row that's already selected, the corresponding rows in the following table will be removed. You should not need to do control click, just click.

fedorov commented 3 years ago

Ok, I really don't know how it is supposed to work. My (apparently, incorrect) expectation was that the behavior would be the same as when selecting multiple files on desktop - click+ctrl adds a new file to selection, but click without control resets the selection to just the single file clicked.

I tested it just now on the production tier, and apparently all tables multi-select without the need to use any alt key. In clicking around in the video below, I did not use any alt keys. I do not know if this is expected behavior or a bug. Either way is fine with me - I will let the @ulrikew as the original reporter comment on usability of this solution.

2021-03-02_10-33-40

G-White-ISB commented 3 years ago

@fedorov I am seeing the same behavior you recorded above. In my last comment I expect my browser may have used a cached version of some javascript sources. We are close to implementing selection with checkboxes only, which is the same as in https://portal.gdc.cancer.gov/, and the same as in https://public.cancerimagingarchive.net/nbia-search/ (if you replace the shopping cart with a checkbox)

fedorov commented 3 years ago

@G-White-ISB what effect, if any, will this selection have on the definition of the cohort?

G-White-ISB commented 3 years ago

@fedorov - no effect. The cohort is defined by the filter definition and the data version number, which are not changed by what you select to view in these tables.The table selections allow one to view information pertaining to the cohort as it is already defined.

madelyngreyes commented 3 years ago

Can this ticket be closed by chance?

G-White-ISB commented 3 years ago

Superseded by #555, so lets close