Skip to content

feat(ui): apply branding tokens to chat surfaces#165

Merged
preetsuthar17 merged 2 commits intomainfrom
preet-spritz-chat-brand-tokens
Mar 27, 2026
Merged

feat(ui): apply branding tokens to chat surfaces#165
preetsuthar17 merged 2 commits intomainfrom
preet-spritz-chat-brand-tokens

Conversation

@preetsuthar17
Copy link
Copy Markdown
Member

TL;DR

Makes the Spritz /c chat surfaces consume the existing branding tokens so deployment-provided branding, including TextCortex's conversation palette, is reflected consistently in the conversation shell.

Summary

  • switch neutral ACP status pills to the configured primary token treatment
  • use brand-emphasis surfaces for active chat rows and sidebar hover states
  • tint the composer shell, slash command pills, and empty conversation state with the existing branding tokens
  • add focused tests for the neutral status pill, active sidebar row, and empty conversation state card

Overview

This PR does not add any TextCortex-specific values to Spritz.

Instead, it fixes the generic chat shell so the /c route actually respects the runtime branding config already provided by downstream deployments. That keeps TextCortex-specific branding in the deployment overlays while making the upstream Spritz chat UI honor those tokens consistently.

Testing

  • cd ui && pnpm test src/components/acp/message.test.tsx src/components/acp/sidebar.test.tsx src/pages/chat.test.tsx
  • Result: 12 passed

@preetsuthar17 preetsuthar17 merged commit 370f794 into main Mar 27, 2026
1 check failed
@preetsuthar17 preetsuthar17 deleted the preet-spritz-chat-brand-tokens branch March 27, 2026 15:38
@gitrank-connector
Copy link
Copy Markdown

👍 GitRank PR Analysis

Score: 20 points

Metric Value
Component Other (1× multiplier)
Severity P2 - Medium (20 base pts)
Final Score 20 × 1 = 20

Eligibility Checks

Check Status
Issue/Bug Fix
Fix Implementation
PR Documented
Tests
Lines Within Limit

Impact Summary

This PR fixes the Spritz chat shell to properly consume existing branding tokens for deployment-provided customization. The changes apply brand-emphasis surfaces to active chat rows, sidebar hover states, status pills, composer shell, slash command pills, and empty conversation state. The fix ensures TextCortex-specific branding from deployment overlays is reflected consistently throughout the chat UI without adding TextCortex-specific values to upstream Spritz.

Analysis Details

Component Classification: This PR affects UI styling and branding token application across chat surfaces (composer, sidebar, message pills, empty state). Since it doesn't fit neatly into a specific component category and involves cross-cutting UI changes, OTHER is the appropriate classification.

Severity Justification: This is a P2 (Medium) severity fix. It addresses a functional bug where the chat UI was not respecting runtime branding configuration tokens that should have been applied consistently. While this impacts user experience and visual consistency, it's not a critical service outage or data loss issue, and users have workarounds (the chat still functions, just without proper branding).

Eligibility Notes: Issue: True - PR describes fixing a bug where branding tokens weren't being applied. Fix Implementation: True - code changes align with applying branding tokens to all mentioned surfaces. PR Linked: True - comprehensive description with TL;DR, summary, and testing instructions. Tests: True - PR adds 64 lines of test code across 3 test files with 12 passing tests. Tests Required: True - this is a functional UI fix affecting business logic (branding application), not a trivial change, so tests are appropriately required and included.


Analyzed by GitRank 🤖

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.

1 participant