[Proposal] Add topmost className prop and merge it into trigger element
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Comienza por la ruta de clonación de hijos de la implementación de Trigger descrita en el issue e inspecciona cómo se combinan las props con el elemento trigger. Añade el comportamiento propuesto de className de nivel superior, conservando el className del hijo, y verifica que el trigger resultante reciba ambas clases.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Due to this issue: https://github.com/ant-design/ant-design/issues/10325, I suggest that we add className as a prop of Trigger for convenience.
Current
const props = this.props;
const children = props.children;
const child = React.Children.only(children);
const newChildProps = { key: 'trigger' };
// Merge some event handlers into `newChildProps`
const trigger = React.cloneElement(child, newChildProps);
Expected
import classNames from 'classnames';
// ...
const props = this.props;
const children = props.children;
const child = React.Children.only(children);
const newChildProps = {
key: 'trigger',
className: classNames(props.className, child.props.className),
};
// Merge some event handlers into `newChildProps`
const trigger = React.cloneElement(child, newChildProps);
| name | type | default | description |
|---|---|---|---|
| className | string | additional className added to trigger |
@yesmeck Shall I create a PR directly? Thanks.
- Lenguaje dominante
- TypeScript
- Estrellas
- 391
- Forks
- 245
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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/trigger
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
react-component/trigger#593 · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
react-component/trigger#623 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
react-component/trigger#618 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
react-component/trigger#555 ·
-
points: ['cc', 'cc'] 无法自适应调整位置 Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
react-component/trigger#550 · 1 comentario ·
Todos los issues de react-component/trigger
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
bug v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
modelcontextprotocol/inspector#2458 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
railmapgen/rmp-gallery#4068 ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
carbon-design-system/ibm-products#9907 ·