nosignal/nosignal-shell
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix: prevent cacheBuffer from extending viewport over height

Browse source
This commit is contained in:
2 * r + 2 * t 2026-04-09 19:10:34 +10:00
parent 3f72dde918
commit 1223b5338b
1 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
plugin/src/Caelestia/Components/lazylistview.cpp
View file

@ -507,7 +507,19 @@ QRectF LazyListView::effectiveViewport() const {
vp = QRectF(vp.x(), top, vp.width(), bottom - top);
}
return vp.adjusted(0, -m_cacheBuffer, 0, m_cacheBuffer);
vp.adjust(0, -m_cacheBuffer, 0, m_cacheBuffer);
// Trim the cache-buffered viewport to [0, layoutHeight]. No items exist outside
// those bounds, so extending past them wastes budget and can cause edge thrashing
// when a large cache buffer reaches the opposite end of the content.
if (m_layoutHeight > 0) {
const qreal top = std::max(vp.y(), 0.0);
const qreal bottom = std::min(vp.y() + vp.height(), m_layoutHeight);
if (top < bottom)
vp = QRectF(vp.x(), top, vp.width(), bottom - top);
}
return vp;
}
std::pair<int, int> LazyListView::computeVisibleRange() const {