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

Feature/multiinput #869

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Feature/multiinput #869

wants to merge 12 commits into from

Conversation

RafaelGondi
Copy link
Member

@RafaelGondi RafaelGondi commented Mar 7, 2025

Por favor, verifique se o seu pull request está de acordo com o checklist abaixo:

  • A implementação feita possui testes (Caso haja um motivo para não haver testes/haver apenas testes de snapshot, descrever abaixo)
  • A documentação no mdx foi feita ou atualizada, caso necessário
  • O eslint passou localmente

1 - Resumo

  • Implementa componente MultiInput

2 - Tipo de pull request

  • 🧱 Novo componente
  • ✨ Nova feature ou melhoria
  • 🐛 Fix
  • 👨‍💻 Refatoração
  • 📝 Documentação
  • 🎨 Estilo
  • 🤖 Build ou CI/CD

3 - Esse PR fecha alguma issue? Favor referenciá-la

Não

4 - Quais são os passos para avaliar o pull request?

  • Code review;
  • Teste as interações com a API do componente

5 - Imagem ou exemplo de uso:

image

6 - Esse pull request adiciona breaking changes?

  • Sim
  • Não

@github-actions github-actions bot added 🐛 Bug Algo não está funcionando 📃 Documentação Melhorias ou adição de documentação labels Mar 7, 2025
@RafaelGondi RafaelGondi marked this pull request as ready for review March 7, 2025 18:53
});

function addInput() {
internalModel.value.push({
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Dar um "focus" no input novo
  • Adicionar suporte ao enter
  • Ter uma opção para remover label (só passar vazio ficar estranho pq a label continua 'floating', só q vazio)

Sugestões:

  • Desabilitar o botão quando a última opção é vazia (para evitar que o usuário adicione diversas opções vazias desnecessariamente)
  • Adicionar a possibilidade de o usuário poder confirmar antes de deletar da lista
  • Como é um input, eu adicionaria a possibilidade de uma label 'por fora' (Ex.: Checklist, Lista de opções)
  • Plus plus plus: Drag and drop para ordenar a lista

@github-actions github-actions bot added 🧱 Componente Novo componente 🛠️ Build Modificação em algum arquivo de build labels Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Algo não está funcionando 🛠️ Build Modificação em algum arquivo de build 🧱 Componente Novo componente 📃 Documentação Melhorias ou adição de documentação
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants