docs(plan): PageHeights.range end-exclusive matches its tests
This commit is contained in:
@@ -1885,8 +1885,8 @@ export class PageHeights {
|
|||||||
range(scrollTop: number, viewport: number, overscan: number): [number, number] {
|
range(scrollTop: number, viewport: number, overscan: number): [number, number] {
|
||||||
if (this.n === 0) return [0, 0];
|
if (this.n === 0) return [0, 0];
|
||||||
const a = Math.max(0, this.pageAt(scrollTop) - overscan);
|
const a = Math.max(0, this.pageAt(scrollTop) - overscan);
|
||||||
const b = Math.min(this.n - 1, this.pageAt(scrollTop + viewport) + overscan);
|
const b = Math.min(this.n, this.pageAt(scrollTop + viewport) + overscan); // [start, end)
|
||||||
return [a, b + 1];
|
return [a, b];
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
```
|
```
|
||||||
|
|||||||
Reference in New Issue
Block a user