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

docs: clarifying DSP backwards compatibility #1439

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rajamatt
Copy link
Contributor

GitHub Issue: closes #1438

PR Type

What kind of change does this PR introduce?

  • Documentation content changes

Description

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

Internal Issue (If applicable):

@rajamatt rajamatt self-assigned this Jun 27, 2024
@rajamatt rajamatt force-pushed the dev/mara/docs-dsp-clarification branch from aca4f68 to 8706b39 Compare June 27, 2024 18:53
Comment on lines +73 to +74
> Although the **Material Theme Builder Tool** doesn't export **Material Tokens (DSP)** .zip packages anymore, Uno.Dsp.Tasks still supports them for backwards compatibility. To use a DSP .zip package just save it as `ColorPaletteOverride.zip` in the `Styles` folder of your application project.
> Make sure to delete the `material-theme.json` file from `Styles` folder before building your application, to avoid conflicts.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Discussion with @agneszitte:

"I think we should add a section to the same page on Uno.Themes
Where we detail what to do in a scenario where we have an old Uno project and want to use Material Theme Builder with the .json file
But I think we have to wait till we know if singleproject is required to add a section like that"

Copy link
Contributor

Choose a reason for hiding this comment

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

fyi @carldebilly we would need to sync back on this doc update when you are back so we can close the PR with the appropriate last changes please

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.

[Docs] Clarify Material Theme Builder output
3 participants