chore: sync with upstream pingdotgg/t3code main - #40
Merged
Conversation
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com> Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com> Co-authored-by: maria <254055478+maria-rcks@users.noreply.github.com>
…gdotgg#8005) Co-authored-by: Simone <185146821+Lucenx9@users.noreply.github.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
…ingdotgg#8189) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…gg#7140) Co-authored-by: Dara Adedeji <daraaded@amazon.com> Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
pingdotgg#8227) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…#8231) Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
…tgg#8358) Co-authored-by: Dara Adedeji <daraadedeji07@gmail.com> Co-authored-by: Ahmed Besic <ahmed-besic@users.noreply.github.com> Co-authored-by: Michael Brown <michaeltbrown.mtb@gmail.com> Co-authored-by: Lars Nieuwenhuis <35393046+lnieuwenhuis@users.noreply.github.com> Co-authored-by: Guilherme Barros <gbarros1095@gmail.com> Co-authored-by: PC <pc@localhost> Co-authored-by: 1xpixi <157762409+1xpixi@users.noreply.github.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com>
…pingdotgg#7538) Co-authored-by: Theo Browne <me@t3.gg>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com> Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com>
Co-authored-by: t3-code[bot] <269035359+t3-code[bot]@users.noreply.github.com> Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com>
Merge upstream/main into the fork, keeping every active ledger divergence intact where upstream reshaped the same code. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
PR SummaryMedium Risk Overview CI and releases: Adds a Mobile: Thread feed renders markdown images (remote URIs and workspace files via signed assets) with sizing helpers and a pluggable Other: Macroscope UI-consistency rules expand (contrast tokens, environment-scoped actions in shared renderers). Desktop app version bumps to 0.0.35; desktop settings tests pick up new client settings fields. Reviewed by Cursor Bugbot for commit 601e7e6. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Important Review skippedToo many files! This PR contains 370 files, which is 270 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (370)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
github-actions
Bot
added
vouch:trusted
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
Reviewed by Cursor Bugbot for commit 601e7e6. Configure here.
| const borderColor = useThemeColor("--color-border"); | ||
| const mutedColor = useThemeColor("--color-foreground-muted"); | ||
| const placeholderColor = useThemeColor("--color-placeholder"); | ||
| const headerFadeColor = String(backgroundColor); |
There was a problem hiding this comment.
Shelf headers stay disabled after load
Medium Severity
shelfPreferencesLoaded is passed into recycled shelf headers and listed on renderItem dependencies, but it is not included in LegendList extraData. When saved expansion matches the defaults, loading prefs does not change row data, so visible Settled/Snoozed headers can remain disabled and ignore taps.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 601e7e6. Configure here.
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
20 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.
@legendapp/listpatch is where both sides had been fixing the same scroll-adjust bug, so taking one would have reintroduced the other's symptom. The patch was regenerated from upstream's version with the fork's padding-restore hardening layered on top, so the fork keeps its guarantee that a timeline never strands padding, and upstream's normalized-value capture keeps working underneath it.server.tskept the fork'smakeRoutesLayerWithshape rather than upstream's flat wiring, because the MCP toolkit dependencies the fork exposes have no home in the flat version, and upstream's new attachment upload route slots into the fork's shape without loss.PullRequestService.tstook upstream's deletion of the list and detail stale windows on its own terms, since upstream deliberately retired that behavior, and the fork's viewed-files cache is unrelated to it and stays.