isVisible failing with element from a different window
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 39/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Localiza la función isVisible en la base de código react-component/util e inspecciona su comprobación de instancia de Element. Reproduce el comportamiento con un elemento de otra ventana y, a continuación, verifica que la comprobación de visibilidad gestione correctamente ese elemento utilizando el contexto de ventana del documento propietario.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hey,
I wanted to report a small issue with isVisible function.
I'm working on an application that opens new windows to show some components. We rely on react component for tooltips and popup and we realized that they are failing on those components that run on the new windows. After some investigation we realised that the issue came from the isVisible function that fails when the element comes from another window.
The problem comes from the element instanceof Element that always fails because the `Element' class instance is window dependant.
There is a fix for which is to rely on element.ownerDocument.defaultView.Element instead.
Would accept a PR for this issue?
Best,
- Lenguaje dominante
- TypeScript
- Estrellas
- 670
- Forks
- 206
- Merge medio
- 18 h 59 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de react-component/util
-
isEqual: false 'circular references' warning for a reference repeated across sibling keysPosiblemente ocupada @savkaoleg la tomó hace 19 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
react-component/util#816 ·
-
injectCSS method not attaching csp nonce passed to it. it is used in antd components internallyAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
react-component/util#671 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
react-component/util#738 · 1 comentario · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
react-component/util#710 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
react-component/util#647 · 5 comentarios ·
Todos los issues de react-component/util
Issues similares
-
fix(data-lake): wizard source step still previews the local slug, not the server-disambiguated oneAbiertodata-lake
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
enhancement good first issue priority: low size: XS
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Empty label or headline exports the editor hint ("LABEL" / "Headline goes here") into the PNGAbierto
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
-
Spray wall wizard: Done button on the hold review step sits under the navigation header (iOS)Abiertobug ios mobile priority:P1
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
streamplace/streamplace#1351 ·
Los mantenedores suelen responder en 2 días