Skip to content

fix: use theme-aware colors for automation modal and telegram language picker#775

Merged
benjaminshafii merged 1 commit intodifferent-ai:devfrom
jcllobet:fix/dark-mode-modals
Mar 7, 2026
Merged

fix: use theme-aware colors for automation modal and telegram language picker#775
benjaminshafii merged 1 commit intodifferent-ai:devfrom
jcllobet:fix/dark-mode-modals

Conversation

@jcllobet
Copy link
Contributor

@jcllobet jcllobet commented Mar 7, 2026

Summary

  • Replace hardcoded bg-white with theme-aware Radix color variables in modals

Why

  • Modals on the Scheduled and Identities pages render with white backgrounds in dark mode, breaking the theme

Scope

  • packages/app/src/app/pages/scheduled.tsx: delete modal and create automation modal (bg-whitebg-gray-1)
  • packages/app/src/app/pages/identities.tsx: Telegram pairing code box (bg-white/80bg-sky-2/80)

Out of scope

  • Language picker modal (uses hardcoded zinc palette but I assume it is intentionally dark-styled, works in both modes)
  • text-white on solid-color buttons (also assume it is intentional white text on accent backgrounds is correct in both themes)

Testing

Ran

  • pnpm run dev with the changes and no additional issues.

Result

  • pass
  • if fail, exact files/errors:

Manual verification

  1. Open app → Scheduled page → click "New automation" → verify modal background adapts to dark mode
  2. Open app → Scheduled page → click "Delete" on a job → verify delete confirmation modal adapts to dark mode
  3. Open app → Messaging tab → Telegram identity → verify pairing code box blends with dark theme
  4. Toggle back to light mode → verify all three still look correct

Evidence

Before

  • CleanShot 2026-03-06 at 18 47 39
  • CleanShot 2026-03-06 at 18 50 32

After

  • CleanShot 2026-03-06 at 18 47 46
  • CleanShot 2026-03-06 at 18 50 39

Risk

  • Minimal: CSS class swaps only, no logic changes

Rollback

  • Revert the three class name changes

@vercel
Copy link

vercel bot commented Mar 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-software Ready Ready Preview, Comment Mar 7, 2026 3:01am

@vercel
Copy link

vercel bot commented Mar 7, 2026

@jcllobet is attempting to deploy a commit to the 0 Finance Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2026

The following comment was made by an LLM, it may be inaccurate:

@jcllobet jcllobet changed the title fix: use theme-aware colors for modals and divs that hardcoded as fix: use theme-aware colors for automation modal and telegram language picker Mar 7, 2026
@benjaminshafii benjaminshafii merged commit 276eb4a into different-ai:dev Mar 7, 2026
5 of 7 checks passed
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