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.

Sign up for GitHub

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

Jump to bottom

fix(popup): ajusta lista ao acionar scroll #1944

Merged
merged 1 commit into from
Feb 6, 2024
Merged

fix(popup): ajusta lista ao acionar scroll #1944

merged 1 commit into from
Feb 6, 2024

Conversation

jnrpalma
Copy link
Contributor

POPUP

DTHFUI-8152

PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
Lista não desaparece ao rolar a tela, quando existem muitos itens associados

Qual o novo comportamento?
Lista desaparece ao rolar a tela, quando existem muitos itens associados.

Simulação
app.zip
& Portal

@jnrpalma jnrpalma force-pushed the popup/DTHFUI-8152 branch 2 times, most recently from d818416 to b109e1d Compare January 26, 2024 15:54
@jnrpalma jnrpalma force-pushed the popup/DTHFUI-8152 branch 4 times, most recently from f8e6846 to c741c29 Compare February 5, 2024 20:05
pedrodominguesp
pedrodominguesp previously approved these changes Feb 6, 2024
View reviewed changes
@pedrodominguesp pedrodominguesp self-requested a review February 6, 2024 00:33
@jnrpalma
fix(popup): ajusta lista ao acionar scroll
5f3b8da 
Ajusta lista ao acionar `scroll` quando existem muitos itens associados.

Fixes DTHFUI-8152
@pedrodominguesp pedrodominguesp merged commit 7fe167f into master Feb 6, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedrodominguesp pedrodominguesp pedrodominguesp approved these changes

@anderson-gregorio-totvs anderson-gregorio-totvs anderson-gregorio-totvs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jnrpalma @pedrodominguesp @anderson-gregorio-totvs