fix: avoid aggressive scrolling when navigating markers

When navigating to the next/prev problem using editor.action.marker.next,
the editor would always scroll the marker to the top of the viewport
(using revealPositionNearTop), even if the marker was already visible
or just slightly off-screen. This caused a "jumpy" experience and confusion.

Changed to use revealPositionInCenterIfOutsideViewport, which maintains
context by only scrolling if necessary, and centers the marker if it is
significantly far away.

Fixes #156782

Author: murataslan1

src/vs/editor/contrib/gotoError/browser/gotoErrorWidget.ts

@@ -358,7 +358,7 @@ export class MarkerNavigationWidget extends PeekViewWidget {
 		}
 		this._icon.className = `codicon ${SeverityIcon.className(MarkerSeverity.toSeverity(this._severity))}`;
 
-		this.editor.revealPositionNearTop(position, ScrollType.Smooth);
+		this.editor.revealPositionInCenterIfOutsideViewport(position, ScrollType.Smooth);
 		this.editor.focus();
 	}