Arvados

• I noticed a horizontal scrollbar shows briefly up when the icons are animating in, it would be a little nicer if we could suppress that somehow - either by tweaking the transition properties of the icons and the container to make sure it doesn't overflow for no reason while animating or if it's easier by adding overflow:hidden to an animation-in-progress class
• When clicking on an items name to navigate to it, it also checks the item and shows the toolbar before the navigation happens - I think you could fix that by doing something like event.stopPropagation() or event.preventDefault() on the item name navigation handler - I forget which
  • This also applies to the 3 button action menu, the UUID and container UUID and modified by user uuid column copy buttons
  • The output uuid and log uuid links don't seem to have this problem when you click them, so you can probably copy the solution from there
• It would be great if you can merge main into your branch to make sure there are no issues merging with the progress bar work - in that branch / now in main I fixed the process page subprocess panel close/maximize/options/search being shifted down instead of in the top right corner of the panel

Changes at arvados-workbench2|bf6b23800e9eb0cfcd2d07eb03a868a63d7142cd branch 21128-toolbar-context-menu

• horizontal scrollbar fixed. Originally, I thought it was fine because it would indicate to the user that scrolling was possible, but to show it during every transition is unnecessary and shoddy, so it's gone until viewport width requires it.

• other points addressed, merged main into local branch and subprocess panel still works as expected. There is some occlusion that happens at <580-ish px width, but I don't think that's relevant to our users.

Great! LGTM