fix status index approxmiation in timeline rendering for dynamically changing viewport geometries #3543

Merged

hj merged 1 commit from iamtakingiteasy/pleroma-fe:fix-timeline-rendering-status-index-approx into develop

2026-08-10 13:50:37 +00:00

iamtakingiteasy commented

2026-08-09 09:27:22 +00:00

Member

Checklist

Adding a changelog: In the changelog.d directory, create a file named <code>.<type>.

Currently dynamically changing viewport geometry can result in estimated indices outside the statuses array boundaries due to race between const height = Math.max(document.body.offsetHeight, window.pageYOffset) and const centerOfScreen = window.pageYOffset + window.innerHeight * 0.5 in timeline.js as underlying window properties change their values between two expressions (at very least in firefox, window properties are not captured for the entirety of function execution unless bound to local variables and do reflect native updates immediately), this MR adds capping to approxIndex similar to cappedScrollIndex.

### Checklist - [x] Adding a changelog: In the `changelog.d` directory, create a file named `<code>.<type>`. Currently dynamically changing viewport geometry can result in estimated indices outside the `statuses` array boundaries due to race between `const height = Math.max(document.body.offsetHeight, window.pageYOffset)` and `const centerOfScreen = window.pageYOffset + window.innerHeight * 0.5` in timeline.js as underlying window properties change their values between two expressions (at very least in firefox, window properties are not captured for the entirety of function execution unless bound to local variables and do reflect native updates immediately), this MR adds capping to `approxIndex` similar to `cappedScrollIndex`.

iamtakingiteasy added 1 commit

2026-08-09 09:27:22 +00:00

fix status index approxmiation in timeline rendering for dynamically changing viewport geometries

ci/woodpecker/pr/build Pipeline is pending approval

Details

ci/woodpecker/pr/changelog Pipeline is pending approval

Details

ci/woodpecker/pr/lint Pipeline is pending approval

Details

ci/woodpecker/pr/test-e2e Pipeline is pending approval

Details

ci/woodpecker/pr/test Pipeline is pending approval