react-component / react-component/trigger
'click' Trigger doesn't hide pop up when user clicks on an iframe in the page
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 391
- Fork
- 245
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
Behaviour:
When using rc-trigger (eg. via Ant Design's Dropdown) with a click type of trigger, the popup will not be closed if a user clicks or taps on an iframe in the page.
Expected Behaviour:
When a user clicks or taps anywhere on the document or an iframe within the document, the popup will be closed. This works fine for contextMenu types of triggers.
Steps to Reproduce:
(Using Ant Design Dropdown)
- Render a view that contains a Dropdown (with
trigger={['click']}) and an iframe that takes up a large part of the screen - Open the Dropdown overlay (popup) by clicking on the Dropdown
- Click on the iFrame
- Outcome: the Dropdown popup will not close
I have submitted a PR to help with fixing this https://github.com/react-component/trigger/pull/96
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Esamina prima pull request #96, quindi riproduci il problema usando un Ant Design Dropdown con trigger={['click']} e un iframe che occupi gran parte della pagina. Verifica che fare clic o toccare l’iframe chiuda il popup, in linea con il comportamento esistente di contextMenu.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- react, typescript
- Ambito
- frontend
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 25/100