feat: adiciona props de placeholder customizáveis no TimeInput#1107
Conversation
Greptile SummaryEsta PR adiciona as props
Confidence Score: 5/5A 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.
|
| 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
Por favor, verifique se o seu pull request está de acordo com o checklist abaixo:
1 - Resumo
Adiciona as props
hourPlaceholdereminutePlaceholderno componenteCdsTimeInput, 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
PlaygroundBuilderonde props string com valor padrão numérico (ex:"00") eram convertidas indevidamente para Number.2 - Tipo de pull request
3 - Esse PR fecha alguma issue? Favor referenciá-la
Não
4 - Quais são os passos para avaliar o pull request?
npx vitest run src/tests/TimeInput.spec.tse garanta que todos os testes passaram com sucesso"00"ao invés de0hour-placeholder="HH"eminute-placeholder="mm"e valide que os placeholders são aplicados corretamente5 - Imagem ou exemplo de uso:
6 - Esse pull request adiciona breaking changes?