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

Meerdere thema's uit 1 json #728

Open
AlineNap opened this issue Oct 2, 2024 · 3 comments · May be fixed by #755
Open

Meerdere thema's uit 1 json #728

AlineNap opened this issue Oct 2, 2024 · 3 comments · May be fixed by #755
Assignees

Comments

@AlineNap
Copy link
Contributor

AlineNap commented Oct 2, 2024

Omschrijving

  1. Graag de token sets die vallen onder de map overwrite negeren bij maken van een thema zodat wij als designers ermee kunnen werken en experimenteren.
  2. Dat het mogelijk is om van thema te wisselen in Storybook. Interessante tokenset hiervoor:

Verschillende thema's zijn beschikbaar in deze PR in de map overwrites/*.

Mogelijke oplossing

Bij LUX is dit werkend, daar hebben we echter de token sets in losse json files, dit is een betaalde feature van TS. Daarnaast hebben Robbert en Jaap-Hein aangegeven lux andere 'build' opzet heeft dan Rijkshuisstijl. Maar wie weet kan er inspiratie en kennis gehaald worden uit LUX project.

Screenshot TS uitleg

Image

@AlineNap
Copy link
Contributor Author

AlineNap commented Oct 4, 2024

Ik heb een extra fork gemaakt van de pr overwrites, ik heb daar stukje themes toegevoegd. Dat is een iets uit de betaalde versie van TS. Is waarschijnlijk handig om te weten hoe dat eruit ziet (in 1 json). Lees ook zeker documentatie of stel je vragen in de slack van TS, ze reageren vaak binnen een dag.

De themas die ik erin heb gezet:

Rijkshuisstijl:

  • wetgevend
  • uitvoerend - violet
  • uitvoerend - violet - oud (oude rijkshuisstijl)
  • uitvoerend - mintgroen - andere fontweight - focus

Deze varianten zitten ook in de thema maker.

Bij LUX is het meer ‘multi-dimensional’ opgezet (met name voor variables, maar wellicht voor RHC ook handig) a la:

mode:

  • dark
  • light

product:

  • Logius
  • MijnOverheid
  • DigiD

viewport:

  • XS
  • md
  • xl

Hier de PR

@Rozerinay
Copy link
Contributor

Wat is hier de status van @remypar5?

@Rozerinay Rozerinay moved this from In Progress to On hold in Community Sprint - Rijkshuisstijl componenten Dec 13, 2024
@remypar5
Copy link

Wat is hier de status van @remypar5?

@Rozerinay Er staat een PR (#755) voor open. Er zitten nog wat issues in. We hebben nog niet uitgezocht waar dat probleem ligt. Kan aan de design tokens liggen, maar ook aan de code.

Van @AlineNap begreep ik in ieder geval dat die setup voor de tokens alleen was om mee te spelen, niet om per se naar productie te brengen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

3 participants