patternfly / patternfly/patternfly-react

Popper - possibly add as a dependency

Abierto
#9,169 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Breaking change :boom:
Lenguaje dominante
TypeScript
Estrellas
862
Forks
392
Merge medio
4 d 8 h
PR fusionados (30 d)
9

Descripción

Is this a new component or an extension of an existing one?
Popper

Describe the feature
A README in our helpers/Popper/thirdparty directory mentions:

  • Delete the thirdparty folder and add them as dependencies to the package.json:
    "react-popper": "2.2.3"
    "@popperjs/core": "2.4.2"

Most likely not as part of this breaking change, but is this something we'd want to look into for a future major release? An alternative would be https://github.com/patternfly/patternfly-react/issues/9089 which would involve just updating our thirdparty code (possibly).

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Revisa el README en helpers/Popper/thirdparty y las referencias de package.json a react-popper y @popperjs/core. Compara este enfoque de dependencias con la alternativa descrita en patternfly-react issue 9089 y aclara qué dirección se desea para una futura versión y qué incluiría reemplazar la carpeta third-party.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
react, typescript
Área
frontend
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.