Skip to content

feat: adiciona props de placeholder customizáveis no TimeInput#1107

Merged
LEMSantos merged 2 commits into
mainfrom
feature/time-input-placeholders
Jul 13, 2026
Merged

feat: adiciona props de placeholder customizáveis no TimeInput#1107
LEMSantos merged 2 commits into
mainfrom
feature/time-input-placeholders

Conversation

@LEMSantos

Copy link
Copy Markdown
Contributor

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

Adiciona as props hourPlaceholder e minutePlaceholder no componente CdsTimeInput, permitindo definir placeholders separados para os campos de hora e minuto. Ambas são opcionais e possuem valor padrão "00".

Corrige também um bug no PlaygroundBuilder onde props string com valor padrão numérico (ex: "00") eram convertidas indevidamente para Number.

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?

  • Execute o comando npx vitest run src/tests/TimeInput.spec.ts e garanta que todos os testes passaram com sucesso
  • No playground da documentação do TimeInput, valide que os campos de placeholder aparecem como text input (não como select)
  • Verifique que o valor padrão dos placeholders é exibido como "00" ao invés de 0
  • Teste passando hour-placeholder="HH" e minute-placeholder="mm" e valide que os placeholders são aplicados corretamente

5 - Imagem ou exemplo de uso:

<CdsTimeInput
  v-model="time"
  label="Horário"
  hour-placeholder="HH"
  minute-placeholder="mm"
/>

6 - Esse pull request adiciona breaking changes?

  • Sim
  • Não

@github-actions github-actions Bot added the 🐛 Bug Algo não está funcionando label Jul 13, 2026
@greptile-apps

greptile-apps Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Esta PR adiciona as props hourPlaceholder e minutePlaceholder ao componente CdsTimeInput, permitindo personalizar o texto de placeholder dos campos de hora e minuto separadamente. Também corrige um bug no PlaygroundBuilder onde valores string com conteúdo numérico (ex.: "00") eram indevidamente convertidos para Number.

  • TimeInput.vue: Duas novas props opcionais do tipo String com valor padrão '00' foram adicionadas e aplicadas nos quatro campos de input, mantendo compatibilidade retroativa total.
  • PlaygroundBuilder.vue: A conversão para Number agora é guardada por propData.type.name.includes('number'), alinhado com o padrão já utilizado no template.
  • TimeInput.spec.ts: Cinco novos testes cobrem o valor padrão, os placeholders individuais, a combinação simultânea e o modo range.

Confidence Score: 5/5

A PR é segura para merge — as mudanças são aditivas, retrocompatíveis e bem cobertas por testes.

As alterações são pequenas e focadas: duas novas props opcionais com valores padrão que reproduzem o comportamento anterior, uma correção de bug pontual no PlaygroundBuilder com padrão já estabelecido no codebase, e cinco novos testes que cobrem os cenários relevantes.

Nenhum arquivo requer atenção especial.

Important Files Changed

Filename Overview
src/components/TimeInput.vue Adiciona as props hourPlaceholder e minutePlaceholder com valor padrão '00', mantendo compatibilidade retroativa.
docs/docgen/PlaygroundBuilder.vue Corrige bug onde strings numéricas eram convertidas para Number indevidamente; fix consistente com padrão existente.
src/tests/TimeInput.spec.ts Adiciona cinco novos testes cobrindo valores padrão, placeholders customizados e modo range.

Reviews (1): Last reviewed commit: "feat: adiciona props de placeholder cust..." | Re-trigger Greptile

@LEMSantos
LEMSantos merged commit 958a3e7 into main Jul 13, 2026
8 checks passed
@LEMSantos
LEMSantos deleted the feature/time-input-placeholders branch July 13, 2026 17:29
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants