[Proposal] Add topmost className prop and merge it into trigger element
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 35/100
- Tipo de issue
- Funcionalidade
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- react, typescript
- Domínio
- frontend
Direção de pesquisa
Comece pelo caminho de clonagem do filho na implementação de Trigger descrito na issue e inspecione como as props são mescladas no elemento trigger. Adicione o comportamento proposto de className de nível superior, preservando o className do filho, e verifique se o trigger resultante recebe ambas as classes.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- TypeScript
- Estrelas
- 391
- Forks
- 246
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de react-component/trigger
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
react-component/trigger#593 · 1 reação ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 58/100
react-component/trigger#623 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 65/100
react-component/trigger#618 · 1 comentário ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 42/100
react-component/trigger#555 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
react-component/trigger#550 · 1 comentário ·
Todas as issues de react-component/trigger
Issues semelhantes
-
area/core status/need-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
google-gemini/gemini-cli#29602 ·
Mantenedores costumam responder em até 1 dia
-
area: backend enhancement priority: low
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
snapotter-hq/SnapOtter#1879 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
Tencent/BrowserSkill#390 ·
Mantenedores costumam responder em até 1 dia
-
good first issue status: needs triaging type: bug version: 2.0
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
medusajs/medusa#17094 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
Mantenedores costumam responder em até 1 dia