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

Atualiza as propriedades do componente Tooltip para serem compatíveis com a nova versão (Tooltip v2) #1787

Merged
merged 1 commit into from
Aug 31, 2024

Conversation

aprendendofelipe
Copy link
Collaborator

Hoje utilizamos a v1 do Tooltip do Primer, que já está obsoleta e pode ser removida no futuro. Então esse PR já adequa as propriedades de maneira compatível com a v2 para cumprir com uma nova regra de lint do plugin eslint-plugin-primer-react. Para o nosso caso, a v1 funciona perfeitamente com as mesmas propriedades da v2.

Ainda não podemos simplesmente mudar para a v2, pois ela não é compatível com a maneira que usamos o Tooltip na barra de ocultar comentários (e talvez em outros locais... é preciso testar e adequar cada um).

Mudanças realizadas

  • Usa a propriedade text ao invés de aria-label. No HTML gerado continua como aria-label.
  • Deixa de usar a propriedade noDelay, o que não gera mudança visual significativa.
  • Deixa de usar a propriedade wrap, que não tinha efeito prático onde estava sendo usada.
  • Atualiza as versões do @primer/octicons-react, @primer/react e eslint-plugin-primer-react, sendo esse último o responsável pelas adequações acima de forma automática.

Checklist:

  • As modificações não geram novos logs de erro ou aviso (warning).

Obs.: Infelizmente permanecem os alertas antigos do Primer (primer/react#4836)

Copy link

vercel bot commented Aug 30, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 30, 2024 0:03am

@Rafatcb Rafatcb added front Envolve modificações no frontend dependências Atualização de dependências labels Aug 30, 2024
@aprendendofelipe aprendendofelipe merged commit 3d5fde5 into main Aug 31, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the primer/react branch August 31, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependências Atualização de dependências front Envolve modificações no frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants