Arvados

Changes at arvados-workbench2|967a861f541ec6e8ef26cd794e7f7890ae65bef7 branch 19079-dual-mode-nav-links
Tests developer-tests-workbench2: #764

• Adds anchor element around rendered names with nav action to enable middle and right click on object names
• onClick is disabled to allow the nav function to handle left click routing
• Right click isn't interfered with so areas using context menu still receive the event and override the browser context menu

Some comments:

• Upon first experimentation, I believe using the browser's right-click menu to allow the user open a search result in a new tab/window would bring issues when the cluster config is set in a way that makes Workbench2 save its credentials on session storage (as it's the case of ce8i5, for example)
• What I think Peter was originally aiming is to have something equivalent to the "Open in new tab" context menu item that we already have on project listings (projects & collections, it seems that processes are also needing it), that already works fine with wb2 session storage usage.
• I personally think using one version of context menu is a lot better for consistency reasons. Using a react-based context menu also allows us to test it with Cypress, that I believe won't be possible otherwise?

If in doubt, let's double-check with Peter.

Yes, I agree with Lucas.

Changes at arvados-workbench2|6aa9f73fef64bbca1e3b71eb526946d69d014246 branch 19079-search-results-open-newtab
Tests developer-tests-workbench2: #826

• Added a search context menu with a few basic options (advanced, details, new tab, and copy to clipboard), since the full featured context menus for each resource kind don't work out of the box without tweaking each action
• Added cypress tests for clipboard and new tab

• The new cypress test is being called with .only()
• I've tried it on ce8i5 and seems to work great, except for one edge case: When having search results from other clusters (like: tordo), the simple click will take the browser to the remote cluster's wb2, but the "Open in new tab" and "Copy to clipboard" will use the local wb2 URL.

Changes at arvados-workbench2|d6e180f251c934a6c9b504eae5bbe50215240e19
Tests developer-tests-workbench2: #829

• For the federated case for copy to clipboard, I switched to getNavUrl which I added a flag to omit the token (to behave similarly to getClipboardUrl, which I renamed to getCollectionItemClipboardUrl since that function is exclusively for keep-web URLs, and the token sanitization there doesn't work on the getNavUrl value when it does add a token anyway)
• Also added mock remote cluster & cluster config using cy.intercept

This LGTM, thanks!

Merged in arvados-workbench2|2157030354bef3870b2703804ec18aab48cce27d