Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

COM-85: Docs, changelog & types for new theming method #1400

Conversation

jamesricky
Copy link
Contributor

No description provided.

@jamesricky jamesricky self-assigned this Nov 15, 2023
@jamesricky jamesricky force-pushed the refactor-admin-component-theming-docs-and-changelog branch 3 times, most recently from 789c595 to f26bf62 Compare November 16, 2023 06:18
@jamesricky jamesricky marked this pull request as ready for review November 16, 2023 06:19
@jamesricky jamesricky changed the title Docs, changelog & types for new theming method COM-85: Docs, changelog & types for new theming method Nov 16, 2023
@jamesricky jamesricky force-pushed the refactor-admin-component-theming-docs-and-changelog branch from f26bf62 to 5370ce7 Compare November 16, 2023 08:35
@thomasdax98
Copy link
Member

thomasdax98 commented Nov 16, 2023

@johnnyomair Do you know why there is no Netlify Deploy Preview for the docs for this PR? (like here #1240 (comment))

nvm I guess that only works if you target main

Copy link
Member

@thomasdax98 thomasdax98 left a comment

Choose a reason for hiding this comment

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

Very good guide 👍 I think everyone should understand that, meaning we can split up the component rewrites

jamesricky and others added 3 commits November 22, 2023 11:54
Co-authored-by: Johannes Obermair <48853629+johnnyomair@users.noreply.github.com>
This is to have a single object that contains all values that influence
a component's styling. Similar to how MUI does it internally.
johnnyomair
johnnyomair previously approved these changes Nov 22, 2023
@jamesricky jamesricky merged commit 92eae2b into feature/refactor-admin-component-theming Nov 23, 2023
@jamesricky jamesricky deleted the refactor-admin-component-theming-docs-and-changelog branch November 23, 2023 09:28
jamesricky added a commit that referenced this pull request Jan 9, 2024
Co-authored-by: Thomas Dax <thomas.dax@vivid-planet.com>
Co-authored-by: Johannes Obermair <48853629+johnnyomair@users.noreply.github.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