Polish select all component

[cdmo]

Like make the label text and placing consistent in holds and checkouts and make it... good.

See #83 (comment)

[cdmo]

Hi @ruthtillman would you mind letting me know your thoughts on options for refactoring the display/behavior of select all:

option 1

One option is to remove the first column called "Select" and merge it with "Item" and then have a checkbox next to Item in the table header that can select all items. This would require an aria-label to ensure that it would make sense to unsighted users.

See https://material-ui.com/components/tables/#sorting-amp-selecting

option 2

Another option is to do what is done in elibrary, essentially an empty row that is just for the "All" checkbox.

option 3

Another option might be to make companion "all" buttons. Sort of like below, but for every hold and renew button.

I'm not sure which I like best to tell the truth. But I think it's the companion "all" buttons OR the merging of select and item columns.

option 4: leave it as is

Maybe a final option is to leave the select alls looking like they do now and say it's good enough.

[ruthtillman]

I also like Options 1 and 3. I think that users probably do expect a "select all" option, so perhaps Option 1? Would the checkbox clearly be for all items for sighted users as well? I'm thinking that sometimes without appropriate textual labeling such boxes aren't entirely clear.

[cdmo]

It is always tricky to walk the line between "simplifying the interface" and "providing enough context" for best usability IMO. I'll go with option 1 and try to get some options for how that will look. Thanks