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

feat(theme): Adicionado melhoria no tema para diferentes acessibilidades #2202

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

bruno-severino
Copy link
Contributor

Adicionado possibilidade de passar mais de uma opção de tema de acessibilidade como exemplo: ‘AA’ e ‘AAA’.

DTHFUI-9037


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
Atuamente não é possivel passar diferentes tipos de acessibilidade do mesmo tema

Qual o novo comportamento?
COm nova implementação agora pode passar varios temas com diferentes tipos de acessibilidade

Simulação
segue um exemplo de uso:

theme = {
  name: 'custom',
  type: [
    {
      light: {},
      dark: {},
      accessibility: PoThemeAccessibilityEnum.AA
    },
    {
      light: {},
      dark: {},
      accessibility: PoThemeAccessibilityEnum.AAA
    }
  ],
  active: {
    type: PoThemeTypeEnum.light,
    accessibility: PoThemeAccessibilityEnum.AAA
  }
};

@bruno-severino bruno-severino force-pushed the theme/DTHFUI-9037 branch 2 times, most recently from b3a04b2 to cb81a71 Compare August 28, 2024 17:46
@bruno-severino bruno-severino force-pushed the theme/DTHFUI-9037 branch 4 times, most recently from ba05e38 to 1c22174 Compare November 1, 2024 12:21
@bruno-severino bruno-severino force-pushed the theme/DTHFUI-9037 branch 3 times, most recently from 6a9ac96 to e62abb1 Compare November 19, 2024 13:59
@bruno-severino bruno-severino force-pushed the theme/DTHFUI-9037 branch 4 times, most recently from 907f36d to d5d521e Compare November 28, 2024 17:32
jcorrea97
jcorrea97 previously approved these changes Dec 9, 2024
@bruno-severino bruno-severino dismissed jcorrea97’s stale review December 9, 2024 12:49

The merge-base changed after approval.

CSimoesJr
CSimoesJr previously approved these changes Dec 9, 2024
adicionado possibilidade de passar mais de uma opção de tema de acessibilidade
como exemplo: ‘AA’ e ‘AAA’.

fixes DTHFUI-9037
@jcorrea97 jcorrea97 merged commit 58425a1 into master Dec 9, 2024
8 checks passed
@jcorrea97 jcorrea97 deleted the theme/DTHFUI-9037 branch December 9, 2024 13: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.

4 participants