fix(web): keep messages clear of composer banners - #7792
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — ... You can add or adjust custom eligibility rules. Learn more. |
## What's Changed * fix(web): keep messages clear of composer banners by @Bil0000 in pingdotgg/t3code#7792 **Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260821.1151...v0.0.34-nightly.20260821.1153 Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260821.1153
What Changed
Why
Composer banners add reserved space after the timeline can already be at the live edge. The list updated its inset but could keep the old scroll offset, which left the last message hidden behind notices such as Server update available.
This fix adjusts the live timeline when that overlay grows. It leaves manual history reading and sent-message anchoring unchanged.
Checks
Checklist
Note
Keep
MessagesTimelinescrolled to end when composer overlay height growskeepTimelineEndVisibleAfterOverlayGrowthto timelineScrollAnchoring.ts, which callstimeline.scrollToEnd({ animated: false })whenfollowingEndis true and overlay height increased; otherwise it is a no-op.useLayoutEffectthat tracks the previous overlay height and only fires when live following is active and no anchor message is set.Macroscope summarized 4eb835a.