Locked groups: flip the lock icon

[sbatten]

Testing #131196

The locked icon is shown unlocked currently when the current group is locked. I am always confused by whether an icon shows the current status or the status after clicking the button (i.e. the action to take). This is because we are not consistent. It seems we tend to favor action over the status but there are exceptions.

SCM: Show as Tree/List
Shows the action, not the current state

Search: Collapse/Expand All
Shows the action, not the current state

Explorer: Filter
Shows the current state, not the action

Timeline: Pinned
Shows the current state, not the action

GitLens Remotes: Show as Tree/List
Shows the current state, not the action

GitLens Search Commits: Keep Results
Shows the current state, not the action

Can we establish some guideline here? Maybe cool for the editor lock would be showing as locked until hover indicating that click would unlock the group.

[bpasero]

Good catch, I tend to just flip the icon to show the lock because we do the same for pinned tabs:

//cc @misolori for the general UX guidance