Created attachment 160613 [details] [review]
[linearView] Animate scroll bar size changes

When adding/removing workspaces in linear view, both workspaces and
scrollbar movement are animated, but the size of the scrollbar handle
changes abruptly. It is more consistent to animate the size change
as well.

Review of attachment 160613 [details] [review]:

This looks fine, and does indeed make sense.

(Side note, we probably should animate hiding/showing the scrollbar too).

Comment on attachment 160613 [details] [review]
[linearView] Animate scroll bar size changes

Attachment 160613 [details] pushed as 88211ed - [linearView] Animate scroll bar size changes

(In reply to comment #2)
> (Side note, we probably should animate hiding/showing the scrollbar too).

Good point - leaving bug open for now then...

Created attachment 160614 [details] [review]
[workspacesView] Animate showing/hiding of controls

Some of the workspace view controls are hidden when the number of
workspaces is one (view toggle button, scroll bar in single view).
Use a fade effect instead of showing/hiding the control abruptly.

Is that the more or less what you had in mind?

(In reply to comment #4)
> Created an attachment (id=160614) [details] [review]
> [workspacesView] Animate showing/hiding of controls
> 
> Some of the workspace view controls are hidden when the number of
> workspaces is one (view toggle button, scroll bar in single view).
> Use a fade effect instead of showing/hiding the control abruptly.
> 
> Is that the more or less what you had in mind?

Yes, even though it isn't as clearly visible as the scrollbar size change, the abrupt change was still odd.

Review of attachment 160614 [details] [review]:

Looks good.

Attachment 160614 [details] pushed as 2ca1fe3 - [workspacesView] Animate showing/hiding of controls