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

Style dictionary update including adding colors #353

Closed
wants to merge 49 commits into from

Conversation

lukasoppermann
Copy link
Contributor

@lukasoppermann lukasoppermann commented Sep 19, 2022

As mentioned to @rezrah in the loom, I am trying to get our config back to the SD basics and at the same time extract small functions.

Additionally I want the json files to be the only influence on the token output so that a user can just edit json files and does not have to expect any side effects form the script.

Focus: easy authoring & json that conform to w3c design token spec

Todos

Cleanup

Could also be tackled after merging.

  • Cleanup build file and how different builds are started
  • Is there a better way to import color json files (currently it may be confusing for new people) at least storing names in short variables could improve readability
  • Organize token folder?
    • base color tokens should files be prefixed with extension to show that they extend dark.json or light.json?
    • functional should exception files be placed in a subfolder?
    • should we split between dark and light mode into subfolders?

Questions

@changeset-bot
Copy link

changeset-bot bot commented Sep 19, 2022

⚠️ No Changeset found

Latest commit: 58c5979

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Contributor

github-actions bot commented Sep 19, 2022

Variables changed
No variables changed

@github-actions
Copy link
Contributor

github-actions bot commented Sep 19, 2022

🟢 No design token changes found

@lukasoppermann lukasoppermann temporarily deployed to github-pages September 19, 2022 09:37 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 19, 2022 10:54 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 19, 2022 11:10 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 19, 2022 11:38 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 19, 2022 12:20 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 20, 2022 08:53 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 20, 2022 09:08 Inactive
@lukasoppermann
Copy link
Contributor Author

Hey @langermank & @rezrah, I am currently thinking about the prefix.

When do we want to have the primer prefix and for which outputs?

You are export with and without here: https://primer.style/primitives/typography and for sizes.
But only base has no prefix.

It also seems we don't have prefixes here: https://primer.style/react/theming or here https://unpkg.com/@primer/css@20.4.5/dist/primer.css

So should we not add prefixes?

Lastly I am not sure adding prefixes to js objects makes sense as you can prefix them when importing and with prefix you would get something like theme.primer.color.canvas.default.

import theme from '@primer/ts/light.ts'

@lukasoppermann lukasoppermann temporarily deployed to github-pages September 20, 2022 14:45 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 20, 2022 16:23 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 20, 2022 19:15 Inactive
@lukasoppermann lukasoppermann temporarily deployed to github-pages September 21, 2022 09:44 Inactive
@lukasoppermann lukasoppermann force-pushed the style-dictionary-colors branch from d71d730 to 6855dec Compare October 4, 2022 12:52
@lukasoppermann lukasoppermann force-pushed the style-dictionary-colors branch from 6855dec to 8b9b14e Compare October 4, 2022 19:59
@lukasoppermann lukasoppermann force-pushed the style-dictionary-colors branch from 8b9b14e to cde1b71 Compare October 5, 2022 10:37
@lukasoppermann lukasoppermann changed the title [Colors] Integrating colors into SD and update SD workflow Style dictionary update including adding colors Oct 10, 2022
@lukasoppermann
Copy link
Contributor Author

Closing as everything is not merged via multiple other PRs.

@lukasoppermann lukasoppermann deleted the style-dictionary-colors branch April 20, 2023 09:53
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.

2 participants