github/atom
1
1
Fork 0
mirror of https://github.com/atom/atom.git synced 2026-01-23 22:08:08 -05:00
Code Issues Packages Projects Releases Wiki Activity

Don't populate more screen rows than necessary

Browse Source 
See 4eecf8d1a6.
This commit is contained in:
Antonio Scandurra
2017-05-10 10:48:16 +02:00
parent ebf2aaa688
commit ab20fc3b19
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/text-editor-component.js
View File

@@ -2673,11 +2673,10 @@ class TextEditorComponent {
// Ensure the spatial index is populated with rows that are currently
// visible so we *at least* get the longest row in the visible range.
populateVisibleRowRange () {
const lastPossibleVisibleRow = this.rowForPixelPosition(this.getScrollBottom())
const maxPossibleVisibleRows = lastPossibleVisibleRow - this.getFirstVisibleRow()
const maxPossibleVisibleTileCount = Math.ceil(maxPossibleVisibleRows / this.getRowsPerTile()) + 1
const lastPossibleRenderedRow = this.getRenderedStartRow() + maxPossibleVisibleTileCount * this.getRowsPerTile()
this.props.model.displayLayer.populateSpatialIndexIfNeeded(Infinity, lastPossibleRenderedRow)
const editorHeightInTiles = this.getScrollContainerHeight() / this.getLineHeight()
const visibleTileCount = Math.ceil(editorHeightInTiles) + 1
const lastRenderedRow = this.getRenderedStartRow() + (visibleTileCount * this.getRowsPerTile())
this.props.model.displayLayer.populateSpatialIndexIfNeeded(Infinity, lastRenderedRow)
}
populateVisibleTiles () {