Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2026-09-25 17:27:26 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix overlay area for minimap slider

Browse Source 
Since minimap always render from a complete line, we have to recalculate sliderTop with align offset.
This commit is contained in:
xndcn
2020-10-13 00:04:55 +08:00
parent 57dd72c09e
commit d3a3f39dbf
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/vs/editor/browser/viewParts/minimap/minimap.ts
+3 -3
View File
@@ -226,8 +226,7 @@ class MinimapLayout {
* Compute a desired `scrollPosition` such that the slider moves by `delta`.
*/
public getDesiredScrollTopFromDelta(delta: number): number {
const desiredSliderPosition = this.sliderTop + delta;
return Math.round(desiredSliderPosition / this._computedSliderRatio);
return Math.round(this.scrollTop + delta / this._computedSliderRatio);
}
public getDesiredScrollTopFromTouchLocation(pageY: number): number {
@@ -332,8 +331,9 @@ class MinimapLayout {
}
const endLineNumber = Math.min(lineCount, startLineNumber + minimapLinesFitting - 1);
const sliderTopAligned = (scrollTop / lineHeight - startLineNumber + 1) * minimapLineHeight / pixelRatio;
return new MinimapLayout(scrollTop, scrollHeight, true, computedSliderRatio, sliderTop, sliderHeight, startLineNumber, endLineNumber);
return new MinimapLayout(scrollTop, scrollHeight, true, computedSliderRatio, sliderTopAligned, sliderHeight, startLineNumber, endLineNumber);
}
}
}