Lock icon inconsistency between output and editor groups

[bpasero]

We recently changed the output lock icon to show a lock when the output is not locked (@mjbvz @sandy081). However we have other locations in the UI that use the lock icon, for example when you open a second editor group, you can lock it from the "..." menu:

Here the icon indicates the current state, where as in output channel it indicates the state after clicking.

This is a UX inconsistency.

[bpasero]

Somewhat similar we have an icon that shows for pinned editors, where the icon indicates the current state:

[gjsjohnmurray]

https://code.visualstudio.com/api/ux-guidelines/editor-actions could usefully say what the convention is, though that page is framed as specific to editor actions. I didn't find one for actions in general.