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

Cria pré-definições de tipografia #19

Merged
merged 9 commits into from
Aug 21, 2023
Merged

Conversation

gabrielguilhoto
Copy link
Contributor

@gabrielguilhoto gabrielguilhoto commented Aug 18, 2023

Contexto

Tínhamos criado os tokens de tipografia isolados (font family, font size, font weight etc.), mas queremos também uma forma de aplicar esses tokens em conjuntos como definidos no Figma. Mais detalhes em https://geekie.atlassian.net/browse/EOF-66.

Mudanças

Defini objetos com as pré-definições de tipografia, me baseando no que está documentado no Figma. Mais detalhes sobre a escolha desse formato no Notion.

Separei as pré-definições em 3 arquivos (basic, ludic e reading) de acordo com o tipo de fonte, agrupados numa pasta presets, e documentei como usar no readme.

Ainda está pendente:

  • documentar essas pré-definições no Storybook
  • resolver warnings de TypeScript que acontecem ao usar os objetos numa stylesheet
  • permitir que as pré-definições sejam usadas num contexto sem polyfills do RN (pela diferença de formato do line height)

Como testar

No SGLearner:

  • adicionei a lib do design system como uma dependência (colocando a URL do Git no package.json e rodando yarn
  • importei uma pré-definição numa tela que use polyfills do RN (peguei a ProfileEditor) e usei ela na definição de um estilo
  • vi que o estilo esperado aparecia na web

DSA_FONT_WEIGHT_NORMAL,
DSA_LETTER_SPACING_NORMAL,
DSA_LETTER_SPACING_WIDE,
DSA_LINE_HEIGHT_L,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Usei os nomes de tokens novos, que ainda não estão no codebase, então os erros de lint são esperados. Mas vou fazer o merge só depois dos nomes novos entrarem na master.

@gabrielguilhoto gabrielguilhoto marked this pull request as ready for review August 18, 2023 17:53
@leocpadua7
Copy link
Contributor

Boa!! Testei todos os preset do grupo basic aqui. Pareceu tudo certo
Captura de Tela 2023-08-21 às 16 33 21
Captura de Tela 2023-08-21 às 16 56 27

@gabrielguilhoto gabrielguilhoto merged commit 1370d92 into master Aug 21, 2023
1 check passed
@gabrielguilhoto gabrielguilhoto deleted the predefs-tokens branch August 21, 2023 21:06
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