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(accordion): ajusta tipo do botão para evitar envio de formulário #2352

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

pedrodominguesp
Copy link
Member

@pedrodominguesp pedrodominguesp commented Jan 15, 2025

Alterado o tipo do botão dentro do componente accordion-item-body de submit para button, evitando que o formulário seja enviado ao pressionar a tecla Enter.
Essa alteração impede que o Accordion feche inesperadamente quando a tecla Enter é pressionada em campos de entrada.
fixes DTHFUI-10166

Qual o comportamento atual?
Ao pressionar a tecla Enter em um input que está dentro do componente accordion-item-body o Componente Accordion realiza o evento de fechar todos os itens.

Qual o novo comportamento?
A tecla Enter não dispara o envio do formulário, pois o botão foi alterado para o tipo button, mantendo os itens do Accordion abertos.

Simulação
Samples no portal.
app_dthfui10166.zip

@pedrodominguesp pedrodominguesp force-pushed the accordion/DTHFUI-10166 branch 2 times, most recently from ad7e4b6 to 80e5be2 Compare January 23, 2025 14:08
@pedrodominguesp pedrodominguesp changed the title fix(accordion): corrige eventos de teclado e alinha tag ao título fix(accordion): ajusta tipo do botão para evitar envio de formulário Jan 23, 2025
Alterado o tipo do botão de submit para button, eliminando o comportamento de envio de eventos para formulários ao pressionar a tecla Enter.
fixes DTHFUI-10166
@anabye anabye merged commit 6a98a70 into 17.x.x Jan 23, 2025
8 checks passed
@anabye anabye deleted the accordion/DTHFUI-10166 branch January 23, 2025 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants