Skip to content

Made the app theme configurable per organization - #311

Open
tifpham021 wants to merge 5 commits into
mainfrom
tiffany/300-configurable-app-theme-per-org
Open

tifpham021 wants to merge 5 commits into
mainfrom
tiffany/300-configurable-app-theme-per-org

Conversation

@tifpham021

@tifpham021 tifpham021 commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Closes #300

Description

  • Adds configurable themes for each organization
  • Configures Visionaries with its purple, cream, fonts, and rounded controls while keeping the default ServiceStart theme unchanged

Changes Made

  • Loads the organization theme on the server to prevent a flash of the default theme
  • Adds validated color, font, and corner-style configuration
  • Applies theme tokens to navigation, buttons, inputs, dashboards, settings, media, and notifications
  • Adds Visionaries theme values and automated tests

Video Showing Updated App Theme for Visionaries Org

Uploading Screen Recording 2026-10-08 at 10.35.41 PM.mov…

How to Test

  • Run pnpm dev
  • Visit Visionaries and log in
  • Navigate between the dashboard, members, events, inbox, and settings pages; confirm the purple accents, cream background, and updated fonts appear consistently
  • Open Customize Dashboard and select Newsletter. Confirm the enabled Save Layout button is purple and pill-shaped
  • Hard-refresh a Visionaries page and confirm the default orange theme does not flash
  • Visit localhost and log in with the same credentials. Confirm it still uses the original orange and white ServiceStart theme

Checklist:

  • I have mentioned a ticket above
  • My changes meet the acceptance criteria of said ticket
  • I have self-reviewed my changes
  • I have requested a review from the appropriate team member(s)
  • I have fixed all merge conflicts
  • I have ensured CI checks pass

@netlify

netlify Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for servicestart ready!

Name Link
🔨 Latest commit b7aabee
🔍 Latest deploy log https://app.netlify.com/projects/servicestart/deploys/6ac883ce08cadc000836c4fe
😎 Deploy Preview https://deploy-preview-311--servicestart.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@greptile-apps

greptile-apps Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Disclaimer: This comment was AI-generated and is not necessarily completely accurate. Please take code comments with a grain of salt.

RetriggerConfidence Score: 5/5

[Medium impact] The PR appears safe to merge, with a non-blocking readability issue in the dark-themed dashboard picker.

Findings

  1. P2 Dark themes hide widget labels ▶

Summary

Adds server-loaded organization colors, fonts, and control corners. Visionaries receives its purple-and-cream theme while unconfigured organizations retain the existing defaults.

  • The previous unnumbered findings about white-card headings, default control corners, and dropdown corners are addressed.
  • Frontend improvement needed: keep dashboard picker labels readable on dark backgrounds.
  • Review score: 90/100. Tests and lint were not run.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Request host] --> B[Find organization]
  B --> C[Load saved theme]
  C --> D[Validate values and choose defaults]
  D --> E[Render HTML with theme variables]
  C --> F[Validate saved corner style]
  F --> G[Set corner attribute when valid]
  E --> H[Pages and shared controls]
  G --> H
Loading

Reviews (2) · Last reviewed commit: "fix themed card text and control corners" · Reviewed by Greptile

Comment thread components/dashboard/widgets/EventsWidget.tsx
Comment thread styles/globals.css Outdated
Comment thread components/theme/ThemedDropdown.tsx
@tifpham021

Copy link
Copy Markdown
Collaborator Author

@greptileai

Comment thread lib/theme.ts

This branch was successfully deployed

1 active deployment
Preview — b7aabeeb Deployed Oct 9, 2026 by tifpham021 via migrate-db #791
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.

[Visionaries] Make the app theme configurable per organization

1 participant