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

fix: Impede a inserção de texto do input de município fora do espaço … #308

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MoisesMsa
Copy link
Collaborator

@MoisesMsa MoisesMsa commented Oct 13, 2024

Português (BR) | English (US)

Comunidade

Tipo de alteração

  • 🐞 Correção de problema
  • ✨ Melhoria ou nova funcionalidade
  • 📰 Nova postagem no blog

Issues relacionadas

Resolve #300

Validação

  • Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
  • Validei o Layout responsivo (desktop/mobile) após a implementação
  • Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

Atualmente é possível ultrapassar o espaço delimitado do campo ao quebrar a linha:

369747056-329c7211-cd2c-4b9b-845a-55ba89605030

Documentação

  • A documentação deste repositório foi atualizada (quando necessário).
  • Esta alteração requer que a documentação externa seja atualizada.

Copy link

netlify bot commented Oct 13, 2024

Deploy Preview for querido-diario-plataforma ready!

Name Link
🔨 Latest commit 5d508c4
🔍 Latest deploy log https://app.netlify.com/sites/querido-diario-plataforma/deploys/670c404e613d830008b86187
😎 Deploy Preview https://deploy-preview-308--querido-diario-plataforma.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@lguima lguima left a comment

Choose a reason for hiding this comment

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

talvez o problema seja na solução que gera o autocomplete, chegou a dar uma olhada nele, talvez atualizar versão, se for dependência, ou estilo do text-area fake que é gerado pra não permitir quebra de linha

de qualquer forma, acho válida a solução, valeu mais uma vez @MoisesMsa!

@MoisesMsa
Copy link
Collaborator Author

@lguima vou dar um doble check, mas pelo que entendi é que o campo de editable não nenhum delimitador de altura/comprimento, o "correto" seria tratar com overflow, mas expremeria muito as cidades selecionadas.

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

Successfully merging this pull request may close these issues.

Campo de cidade permite inserir texto fora do campo
3 participants