Previously we would fire a wrong active-editor-change-event when more than one group is opened and the user clicks into an inactive editor of the inactive group because we would first activate the group and then open the editor.

Fixes #88059

Fixes #88064

fixes #87922

tslint to eslint migration

Fix #87939