Skip to content

Use semantic theme tokens in OpenDocsButton#7

Open
piyush-3020 wants to merge 1 commit into
gnick18:mainfrom
piyush-3020:fix-open-docs-button-theme
Open

Use semantic theme tokens in OpenDocsButton#7
piyush-3020 wants to merge 1 commit into
gnick18:mainfrom
piyush-3020:fix-open-docs-button-theme

Conversation

@piyush-3020

Copy link
Copy Markdown

Summary

Replaced hardcoded theme colors in OpenDocsButton with semantic theme tokens.

Changes:

  • bg-slate-900/85 → bg-surface-raised
  • hover:bg-slate-900 → hover:bg-surface-sunken
  • text-white → text-foreground
  • border-white/10 → border-border

Fixes #6

@vercel

vercel Bot commented Jun 7, 2026

Copy link
Copy Markdown

@piyush-3020 is attempting to deploy a commit to the Grant Nickles' projects Team on Vercel.

A member of the Team first needs to authorize it.

gnick18 added a commit that referenced this pull request Jun 8, 2026
…est mocks)

- Remove dead vi.mock TelegramHeaderButton/TelegramPopup/TelegramEncryptedRecoveryPrompt
  from AppShell.tourGate, AppShell.featurePicksTabs, AppShell.piHomeTab tests
- WelcomePage: remove Telegram-capture bento tile (tag="Telegram capture"); leave
  a comment that a phone-companion showcase can replace it later
- wiki/features/notifications: replace "Telegram inbox" language with "photo inbox"
  throughout intro, pill description, modal section, toast section, and footer
- wiki/security: remove Telegram file-CDN proxy bullet, both Telegram token-recovery
  Callouts (IndexedDB cache + encrypted backup), "Bot tokens" warning Callout,
  Telegram entries in file layout, DevTools IndexedDB step updated to two databases;
  audit note updated; "two proxies" → "one proxy" throughout
- wiki/getting-started/welcome-wizard: remove Q5 Telegram phrasing (replaced with
  "phone companion"), remove settings-tour-telegram step, remove Telegram conditional
  walkthrough from Phase 2b list, remove telegram===yes tab-visibility rule
- wiki/start-here: remove Telegram from callout #1 and callout #7, replace
  "telegram" search example with "calendar"
- wiki/nav: update Notifications blurb; remove Telegram Bot nav entry (page deleted)
- wiki search test fixture: replace Telegram Bot entry with Calendar Feeds

Intentionally left (correct to keep):
  - telegramMediaGroupId backward-compat sidecar reads in PhotosInboxTab (backward-compat)
  - image-folder.ts legacy sidecar field definitions (backward-compat)
  - onboarding step-machine tests asserting "telegram" step is GONE (correct guards)
  - artifacts.ts encodeTelegramImageId / decodeTelegramImageLocation (cleanup engine uses these)
  - wiki/features/settings: FLAG for wiki-voice pass (deep Telegram settings that no
    longer exist in the UI; needs a full rewrite of that section, not a line edit)
  - Code comments in attach-image, store, poll, etc. referencing Telegram as historical context

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

Convert remaining hardcoded colors to semantic theme tokens

1 participant