Skip to content

Introduce DiamondDS theme foundation and component updates#177

Open
zoharma wants to merge 9 commits into
mainfrom
zma/new-dstheme
Open

Introduce DiamondDS theme foundation and component updates#177
zoharma wants to merge 9 commits into
mainfrom
zma/new-dstheme

Conversation

@zoharma
Copy link
Copy Markdown

@zoharma zoharma commented May 8, 2026

Introduced the initial Diamond Design System theme foundation, including semantic colour roles, light/dark mode support, CSS variable mappings, and integration with the MUI theming architecture.

Updated component styling, states, and overrides across inputs, buttons, checkboxes, radio buttons, overlays, borders, and theme icons.

Introduced the initial Diamond Design System theme foundation,
including semantic colour roles, light/dark mode support,
CSS variable mappings, and integration with the MUI theming
architecture.

Updated component styling, states, and overrides across inputs,
buttons, checkboxes, radio buttons, overlays, borders,
and theme icons.

Fixed issues

Prettify
zoharma added 2 commits May 8, 2026 14:05
Mini intro and foundations pages for principles, colour, and typography
@akademy akademy added being reviewed Someone is reviewing this PR labels May 11, 2026
Copy link
Copy Markdown
Member

@akademy akademy left a comment

Choose a reason for hiding this comment

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

I've just highlighted a few things for me to check tomorrow, just getting my head around what's there. @zoharma no need for any changes yet.

Comment thread .storybook/preview.tsx Outdated
Comment thread .storybook/preview.tsx Outdated
Comment thread tsconfig.json
@@ -1,11 +1,7 @@
{
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't seem to have any changes except cosmetic.

Comment thread src/index.ts
export * from "./themes/BaseTheme";
export * from "./themes/DiamondTheme";
export * from "./themes/DiamondOldTheme";
export * from "./themes/DiamondDSTheme";
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there some reason why we've removed the old theme?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I don't see that it's being used by the dropdown. We can keep it...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's not available in Storybook, but it can be used downstream. It's just there in case someone says "we loved the other one"! But hopefully, no one will use it!

Comment thread src/themes/ThemeProvider.tsx
Comment thread .storybook/preview.tsx Outdated
Comment thread .storybook/preview.tsx
Comment thread .storybook/ThemeSwapper.tsx Outdated
Comment thread src/themes/DiamondDSTheme.ts Outdated
Comment thread src/styles/diamondDS/diamond-ds-roles.css Outdated
Comment thread src/themes/DiamondDSTheme.ts Outdated
@zoharma zoharma force-pushed the zma/new-dstheme branch 11 times, most recently from b5b07cd to e28b014 Compare May 12, 2026 13:26
zoharma added 3 commits May 12, 2026 14:28
Revert "Update ver"

This reverts commit 035cb77.

Reapply "Update ver"

This reverts commit e28b014.
…iguration support

Following suggestion of simplification by @akademy.
- Updated theme mode handling to use MUI colour scheme classes
- Removed previous data-mode logic
- Introduced back System settings option and a way to switch to it

Revert "Refactor Storybook theme handling and introduce Diamond DS theme configuration support"

This reverts commit 3784323.

Reapply "Refactor Storybook theme handling and introduce Diamond DS theme configuration support"

This reverts commit a845238.
…eme configuration support"

This reverts commit 8b304fd.
@zoharma zoharma force-pushed the zma/new-dstheme branch from 8fc1b59 to 699625a Compare May 12, 2026 13:28
@zoharma zoharma force-pushed the zma/new-dstheme branch 3 times, most recently from f05507c to 4664afc Compare May 12, 2026 17:47
@zoharma zoharma force-pushed the zma/new-dstheme branch 2 times, most recently from de3b23e to 603db8a Compare May 12, 2026 18:07
- document theme architecture and semantic token model
- simplified repeated conventions across intents
- clarify action intent, status intent, and brand usage
- add dedicated BrandPaletteColor typing with fixed brand roles
- improve helper documentation and component override comments
- keep CSS variables as the source of truth for theme roles
- remove global Paper override
- update theme colours to use grey primitives
@zoharma zoharma force-pushed the zma/new-dstheme branch from 603db8a to fa62002 Compare May 12, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

being reviewed Someone is reviewing this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants