Skip to content

feat(ui): show dirty state indicator in pages command palette - #2689

Closed
JC-Chung wants to merge 1 commit into
sourcegit-scm:developfrom
JC-Chung:feat/pages-palette-dirty-state-indicator
Closed

JC-Chung wants to merge 1 commit into
sourcegit-scm:developfrom
JC-Chung:feat/pages-palette-dirty-state-indicator

Conversation

@JC-Chung

Copy link
Copy Markdown
Contributor

What

Add a DirtyState indicator dot and tooltip to each open tab (LauncherPage) in the Pages Command Palette (Ctrl+P / ⌘+P), matching the presentation already used in LauncherTabBar.

Changes

  • LauncherPagesCommandPalette.axaml:
    • Add xmlns:m="using:SourceGit.Models" namespace
    • Add an Ellipse in the vm:LauncherPage DataTemplate that reflects DirtyState via DirtyStateConverters.ToBrush
    • Add ToolTip.Tip showing the repository path (Node.Id) and DirtyState description (DirtyStateConverters.ToDesc); the welcome tab still shows its original text

Why

Lets users see at a glance whether a tab has uncommitted changes or is out of sync with upstream, without having to switch to it first.

@JC-Chung
JC-Chung force-pushed the feat/pages-palette-dirty-state-indicator branch from ee6e7a6 to 5ef73e4 Compare September 10, 2026 04:18
@love-linger

Copy link
Copy Markdown
Collaborator

I've pushed my implementation for this feature.

image

@JC-Chung JC-Chung closed this Sep 10, 2026
@JC-Chung
JC-Chung deleted the feat/pages-palette-dirty-state-indicator branch September 10, 2026 07:50
@JC-Chung

Copy link
Copy Markdown
Contributor Author

I've pushed my implementation for this feature.

This ahead/behind formatting logic already exists in Branch and RepositoryStatus, and now a third copy here — worth extracting into a common helper?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants