polish(tray/chat): replace reconnecting spinner with skeleton timeline + composer#498
Merged
shanselman merged 1 commit intoMay 22, 2026
Merged
Conversation
…e + composer Replaces the centered `Reconnecting...` spinner with a message-shaped skeleton (timeline + composer) so returning users see a placeholder that visually maps to the chat surface that will replace it. Also eliminates the brief zero-state flash between thread selection and the first chat.history response. What changed - Skeleton timeline (RenderSkeletonTimeline): 5 alternating bubbles with subtle text-line stripes, bubble radius 8 (matches composer), line radius 4, staggered shimmer (Storyboard opacity 1.0 -> 0.45, 900ms, AutoReverse, SineEase). 4px-grid layout throughout. - Skeleton composer (RenderSkeletonComposer): 56-tall rounded input placeholder + 40x40 circular send placeholder, synced shimmer. - Welcome flicker fix: 800ms debounce on welcomeSettledState plus a two-path eligibility gate (isComposeOnlyThread / hasRealThreads / timeline.HistoryLoaded) so the zero-state no longer flashes while history is in flight. - Sessions-list gate (OpenClawChatDataProvider): _sessionsListReceived ensures composeReady does not fire before we know whether real threads exist. Regression test added. Validation - ./build.ps1 - Tray tests 1176/1176 - Shared tests 1813/1813 - Manual: no flicker; skeleton breathes; bubble radius matches composer. Deferred follow-ups - HC-aware ThemeResource brushes for skeleton fills - Skip shimmer when AccessibilitySettings.HighContrast - AutomationProperties.Name + LiveSetting=Polite on skeleton container Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
kenehong
force-pushed
the
polish/chat-skeleton-timeline
branch
from
ea32e10 to
20eb9e4
Compare
shanselman
added a commit
that referenced
this pull request
May 22, 2026
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.
Summary
Replaces the centered "Reconnecting…" spinner with a message-shaped
skeleton (timeline + composer) so returning users see a placeholder
that visually maps to the chat surface that will replace it. Also
eliminates the brief zero-state flash that happened between thread
selection and the first
chat.historyresponse.What changed
OpenClawChatRoot.RenderSkeletonTimeline)RenderSkeletonComposer)OpenClawChatRoot)welcomeSettledStateisComposeOnlyThread/hasRealThreads/timeline.HistoryLoadedso the zero-stateno longer appears momentarily while history is in flight
OpenClawChatDataProvider)_sessionsListReceivedensurescomposeReadydoesn't fire beforewe know whether real threads exist (regression test added)
Validation
./build.ps1✅Deferred (follow-ups)
AccessibilitySettings.HighContrastAutomationProperties.Name="Loading conversation"+ LiveSetting=Polite