Error navigation has one empty hop when switching between cells

[isidorn]

Refs: #93742

1. Have a notebook file, with multiple cells and each cell having an error
2. Navigate between errors via F8. Notice how after navigated to a new cell the error is not shown, it takes an additional F8 to get the inline error widget

[rebornix]

My guess is F8 will reveal the second cell which triggers it be rendered and then it forgets to open the error peek view after being opened. I'm wondering why it works fine with F8 to another file.

[rebornix]

vscode/src/vs/editor/contrib/gotoError/gotoError.ts

Lines 398 to 406 in cd0d648

	return editorService.openCodeEditor({
	resource: newMarker.resource,
	options: { pinned: false, revealIfOpened: true, selectionRevealType: TextEditorSelectionRevealType.NearTop, selection: newMarker }
	}, editor).then(editor => {
	if (!editor) {
	return undefined;
	}
	return editor.getAction(this.id).run();
	});

It's waiting for the editor to be open with editorService. @jrieken not sure if composite editor is hooked up with it too.

[jrieken]

hm, it should be hooked up but I am unsure if it returns an editor or the composite editor...