Skip to content

Comments

feat: Design tokens implementations for the dark theme#47

Merged
Faraz32123 merged 1 commit intodark-theme-tokensfrom
rahat-dark-theme-tokens
Feb 4, 2026
Merged

feat: Design tokens implementations for the dark theme#47
Faraz32123 merged 1 commit intodark-theme-tokensfrom
rahat-dark-theme-tokens

Conversation

@arbirali
Copy link

No description provided.

@arbirali arbirali marked this pull request as draft January 27, 2026 15:04
@arbirali arbirali self-assigned this Jan 28, 2026
@arbirali arbirali changed the title feat: Work in progress feat: Design tokens implementations for the dark theme Jan 28, 2026
@arbirali arbirali force-pushed the rahat-dark-theme-tokens branch from a9fa6d7 to f18ef58 Compare January 28, 2026 13:09
@arbirali arbirali force-pushed the rahat-dark-theme-tokens branch from f18ef58 to 2fedde9 Compare February 2, 2026 13:11
@Faraz32123 Faraz32123 marked this pull request as ready for review February 3, 2026 09:56
Copy link

@ahmed-arb ahmed-arb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’m no styling expert, but this looks good to me.

I do have a concern about the dist folder. If we’ve decided to consume this brand in Indigo directly from GitHub (instead of npm), then we shouldn’t rely on developers to manually update dist in PRs.

We should either:

  1. set up a workflow to generate and commit dist automatically on every merge to ulmo, or
  2. remove dist from PRs entirely and rely solely on the workflow to generate it in the ulmo branch.

We can look into this separately.

@Faraz32123 Faraz32123 merged commit 21e8959 into dark-theme-tokens Feb 4, 2026
1 check passed
Faraz32123 pushed a commit that referenced this pull request Feb 4, 2026
* fix: add dark theme tokens

* feat: Design tokens implementations for the dark theme (#47)

---------

Co-authored-by: Rahat Ali <rahat.ali@arbisoft.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.

3 participants