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

Torna visível o Tooltip do ícone de exibir/ocultar a senha #1620

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

aprendendofelipe
Copy link
Collaborator

O problema

Na tela de login ou cadastro, deixe o cursor sobre o ícone do olho no campo de senha.

O Tooltip deveria estar visível com o texto Ocultar a senha ou Visualizar a senha.

Antes Depois
image image

Não é algo específico do TabNews, mas um problema conhecido da versão atual do Primer. O problema ocorre até no componente de exemplo na página deles.

Mudanças realizadas

Substituí o TextInput.Action pelo nosso Tooltip customizado junto de um IconButton.

Também aproveitei o PR para remover outros "ajustes" que o TextInput.Action exigia no passado, mas que já não eram mais necessários na versão atual do Primer. Um determinava a altura mínima de todos os campos de texto dos formulários para ficar no mesmo padrão do campo de senha, e o outro ajuste era na posição padrão do Tooltip do TextInput.Action.

Tipo de mudança

  • Correção de bug

Checklist:

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

Copy link

vercel bot commented Feb 1, 2024

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

Name Status Preview Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview Feb 1, 2024 11:03pm

@Rafatcb Rafatcb added front Envolve modificações no frontend bug Comportamento diferente do esperado labels Feb 2, 2024
Copy link
Collaborator

@Rafatcb Rafatcb left a comment

Choose a reason for hiding this comment

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

Bem observado 👍

Eu tinha visto esse issue no repositório do Primer quando estávamos mexendo no PR #1589, achei que a essa altura já teriam mergeado. Na hora não me toquei que tínhamos algum Tooltip em um TextInput.

@aprendendofelipe aprendendofelipe merged commit 76dfea9 into main Feb 2, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the tooltip-password-input branch February 2, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Comportamento diferente do esperado front Envolve modificações no frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants