Arvados

Branch 11365-collection-lock-button implements:

• Add a "lock" button for collection#show page with confirmation dialog

• On collection page load, (since the collection is in locked state on page load), disable the Upload tab, the pencil icons on files (rename option) and trash icon on files

• On unlock, they are enabled and accessible

• When the collection is locked, the drop down button "Selection... "-> "Remove selected files" is disabled even when one or more files are selected using the checkbox(es). Only when the collection is "unlocked" this selection option is enabled (if one or more files are selected)
  • One caveat here is that if the user clicks on the checkboxes and clicks on Selection... and then goes and unlocks the collection, this button event will not be fired and will need the user to toggle one or more checkboxes to re-fire the event
  • Another caveat is with the Upload tab. If a user clicks on lock and unlocks the collection, transitions into the Upload tab and then goes and locks the collection again, the user can still work in the Upload tab (since he was already here). It doesn't seem like putting in effort to prevent this edge case is worth the while

Test run @ https://ci.curoverse.com/job/developer-run-tests/230/

Some cosmetic questions:

• Seems to me that the lock/unlock icon is too small in comparison to the buttons on its side. It may be difficult to spot by the users, if we don’t have any bigger icon, maybe we can place it on the left side so it’s more visible?
• Can the lock icon have a tooltip explaining its purpose? Or maybe the disabled buttons to have an alternate tooltip to explain why they’re disabled?
• Typo on the user warning: ‘portalble'
• apps/workbench/test/integration/collections_test.rb:377 - Typo on the test name
• Can the unlock code on the tests be moved somewhere to be reused as a func?

Updates @ 022d82146b1f004cec158d9801fecd8a17637e47 LGTM, thanks!