Changes from review

Author: aiday-mar

src/vs/editor/contrib/gotoSymbol/browser/goToCommands.ts

@@ -232,11 +232,7 @@ export abstract class SymbolNavigationAction extends EditorAction {
 	private _openInPeek(target: ICodeEditor, model: ReferencesModel, range?: Range) {
 		const controller = ReferencesController.get(target);
 		if (controller && target.hasModel()) {
-			if (range) {
-				controller.toggleWidget(range, createCancelablePromise(_ => Promise.resolve(model)), this.configuration.openInPeek);
-			} else {
-				controller.toggleWidget(target.getSelection(), createCancelablePromise(_ => Promise.resolve(model)), this.configuration.openInPeek);
-			}
+			controller.toggleWidget(range ?? target.getSelection(), createCancelablePromise(_ => Promise.resolve(model)), this.configuration.openInPeek);
 		} else {
 			model.dispose();
 		}

src/vs/editor/contrib/inlayHints/browser/inlayHintsLocations.ts

@@ -106,8 +106,7 @@ export async function goToDefinitionWithLocation(accessor: ServicesAccessor, eve
 		const canPeek = !openToSide && editor.getOption(EditorOption.definitionLinkOpensInPeek) && !isInPeek;
 
 		const action = new DefinitionAction({ openToSide, openInPeek: canPeek, muteMessage: true }, { alias: '', label: '', id: '', precondition: undefined });
-		const locationRange = location.range;
-		return action.run(accessor, editor, { model: ref.object.textEditorModel, position: Range.getStartPosition(locationRange) }, new Range(locationRange.startLineNumber, locationRange.startColumn, locationRange.endLineNumber, locationRange.endColumn));
+		return action.run(accessor, editor, { model: ref.object.textEditorModel, position: Range.getStartPosition(location.range) }, Range.lift(location.range));
 	});
 
 	ref.dispose();