revert: bump up the target rate to 120fps (#6868)#6985
Merged
MitjaBezensek merged 1 commit intomainfrom Oct 22, 2025
Merged
Conversation
This reverts commit 2e5f9f2.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
4 Skipped Deployments
|
MitjaBezensek
temporarily deployed
to
e2e-dotcom
GitHub Actions
Inactive
MitjaBezensek
temporarily deployed
to
bemo-canary
GitHub Actions
Inactive
MitjaBezensek
temporarily deployed
to
deploy-staging
GitHub Actions
Inactive
Contributor
|
API Changes Check Passed Great! The PR description now includes the required "### API changes" section. This helps reviewers and SDK users understand the impact of your changes. |
MitjaBezensek
added
the
dotcom-hotfix-please
huppy-bot bot
pushed a commit
that referenced
this pull request
Oct 22, 2025
This reverts commit 2e5f9f2. ### API changes - Revert the 120 fps changes. ### Change type - [x] `bugfix` <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Standardizes throttling to 60fps (removing custom/120fps logic), updates TLSyncClient scheduling, simplifies FPS debug display, and adjusts the utils API. > > - **Utils**: > - Reverts `fpsThrottle` to fixed 60fps; removes custom FPS support and related state. > - Updates docs/comments and `throttleToNextFrame` to reference 60fps. > - API change: `fpsThrottle(fn)` no longer accepts `getTargetFps`. > - **Sync Core** (`packages/sync-core/src/lib/TLSyncClient.ts`): > - Removes dynamic sync FPS logic (`SOLO_MODE_FPS`, `COLLABORATIVE_MODE_FPS`, `getSyncFps`). > - `flushPendingPushRequests` and `scheduleRebase` now use `fpsThrottle()` without FPS getter. > - **UI** (`packages/tldraw/src/lib/ui/components/DefaultDebugPanel.tsx`): > - Simplifies FPS output to `FPS <current>` (removes max FPS display). > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit c9995c0. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
3 tasks
huppy-bot bot
added a commit
that referenced
this pull request
Oct 22, 2025
This is an automated hotfix for dotcom deployment. Original PR: #6985 Original Author: @MitjaBezensek
MitjaBezensek
added a commit
that referenced
this pull request
Oct 24, 2025
mimecuvalo
added a commit
that referenced
this pull request
Dec 17, 2025
steveruizok
changed the title
NathanFlurry
pushed a commit
to rivet-dev/tldraw
that referenced
this pull request
Jan 18, 2026
This reverts commit 2e5f9f2. ### API changes - Revert the 120 fps changes. ### Change type - [x] `bugfix` <!-- CURSOR_SUMMARY --> --- > [!NOTE] > Standardizes throttling to 60fps (removing custom/120fps logic), updates TLSyncClient scheduling, simplifies FPS debug display, and adjusts the utils API. > > - **Utils**: > - Reverts `fpsThrottle` to fixed 60fps; removes custom FPS support and related state. > - Updates docs/comments and `throttleToNextFrame` to reference 60fps. > - API change: `fpsThrottle(fn)` no longer accepts `getTargetFps`. > - **Sync Core** (`packages/sync-core/src/lib/TLSyncClient.ts`): > - Removes dynamic sync FPS logic (`SOLO_MODE_FPS`, `COLLABORATIVE_MODE_FPS`, `getSyncFps`). > - `flushPendingPushRequests` and `scheduleRebase` now use `fpsThrottle()` without FPS getter. > - **UI** (`packages/tldraw/src/lib/ui/components/DefaultDebugPanel.tsx`): > - Simplifies FPS output to `FPS <current>` (removes max FPS display). > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit c9995c0. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY -->
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This reverts commit 2e5f9f2.
API changes
Change type
bugfixTest plan
Release notes
Note
Standardizes throttling to 60fps (removing custom/120fps logic), updates TLSyncClient scheduling, simplifies FPS debug display, and adjusts the utils API.
fpsThrottleto fixed 60fps; removes custom FPS support and related state.throttleToNextFrameto reference 60fps.fpsThrottle(fn)no longer acceptsgetTargetFps.packages/sync-core/src/lib/TLSyncClient.ts):SOLO_MODE_FPS,COLLABORATIVE_MODE_FPS,getSyncFps).flushPendingPushRequestsandscheduleRebasenow usefpsThrottle()without FPS getter.packages/tldraw/src/lib/ui/components/DefaultDebugPanel.tsx):FPS <current>(removes max FPS display).Written by Cursor Bugbot for commit c9995c0. This will update automatically on new commits. Configure here.