refine global checkpoints UI#3457
Merged
Kvadratni merged 9 commits intoblock:feature/global-checkpointsfrom Jul 16, 2025
Merged
refine global checkpoints UI#3457Kvadratni merged 9 commits intoblock:feature/global-checkpointsfrom
Kvadratni merged 9 commits intoblock:feature/global-checkpointsfrom
Conversation
- Remove redundant ConditionalSidecarProvider from App.tsx that was causing duplicate providers - Keep SidecarProvider scoped to BaseChat.tsx where it's actually needed - Clean up unused imports and debug content - Simplify architecture by having sidecar functionality only in chat components - Each chat instance now has its own isolated sidecar state
- Add right-to-left slide animation matching sidebar behavior - Use fixed positioning with translate transforms for smooth performance - Add main content margin adjustment when sidecar is visible - Include visual enhancements with border and shadow - Maintain consistent 300ms ease-out timing with sidebar
- Replace GitBranch icons with FileDiff icons in both sidecar diff viewer and GooseMessage component - Improve semantic meaning and visual clarity for diff-related functionality - Update icon imports and usage throughout the application - Fix linting issues including TypeScript unused parameter warnings - Ensure proper type safety with window object extensions
…nent
- Replace custom tooltip div with proper Tooltip component from ui/Tooltip
- Match the tooltip style and positioning used in sidecar view mode buttons
- Use consistent side='bottom' and sideOffset={8} positioning
- Improve tooltip accessibility and visual consistency across the application
…ibility in Sidecar
…onents for improved readability
…ut for diff content
Kvadratni
approved these changes
Jul 16, 2025
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 PR cleans up the text diff sidecar UI to match the new look and feel:
goose-text-diff-sidecar-ui-refresh.mov
Currently lacks window width management and sidecar entrance animations.