acl-services / acl-services/paprika

Popover - add callback for onAfterOpen and onAfterClose

Aberta
#316 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
JavaScript
Estrelas
54
Forks
10
Merge médio
1d 12h
PRs com merge (30d)
6

Descrição

# Feature request

It would be nice to be able to focus elements in the content of the popover after they have mounted via callbacks.

Suggest adding `onAfterOpen` and `onAfterClose` callbacks in a similar way to how it is done in TakeOver component

## Prerequisites
**This is just a checklist, please delete this section after**
Please answer the following questions for yourself before submitting a feature request.
- [ ] I checked to make sure that this feature has not already been suggested
- [ ] Please label as, low / medium / high priority

*Please fill in as much detail as possible*

## Is your feature request related to a problem?
*A clear and concise description of what the problem is*

## Describe the solution you'd like
*A clear and concise description of what you want to happen.*

## Describe alternatives you've considered
*A clear and concise description of any alternative solutions or features you've considered.*

## Additional context
*Add any other context or screenshots about the feature request here.*

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

The issue references the Popover and TakeOver components only. Start by finding the Popover component source and the TakeOver implementation to mirror its lifecycle callback pattern. Then inspect any Popover tests/docs in the repo to add or adjust coverage for onAfterOpen/onAfterClose behavior. Done means the new callbacks are exposed in Popover and their invocation points are validated by existing or new component tests/docs.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
javascript, react
Domínio
frontend
Tipo de issue
Funcionalidade
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
56/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.