className default is lost when props.className is undefined in UncontrolledTabs
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 45/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- javascript, react
- Domínio
- frontend
Direção de pesquisa
Comece pelo componente UncontrolledTabs e inspecione o spread de objeto de default-props, reproduzindo o caso em que props.className é explicitamente undefined. Confirme o comportamento esperado para a classe padrão "react-tabs" e verifique se a classe continua sendo aplicada sem comprometer um className fornecido explicitamente.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
In the UncontrolledTabs component, default props are merged using object spreading:
const { className, ...attributes } = { ...defaultProps, ...props, };
Even the defaultProps.className is defined ('react-tabs') and props.className is explicitly undefined, the spread operation will overwrite the default and set className to undefined.
As a result, the default class name is lost. It's unclear whether this behavior is intended, but it can lead to unexpected issues, especially when upgrading from older versions where the default class was always applied. Projects relying on the default 'react-tabs' class for styling or selectors may break silently.
- Linguagem predominante
- JavaScript
- Estrelas
- 3.1k
- Forks
- 455
- 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 reactjs/react-tabs
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 25/100
reactjs/react-tabs#584 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
reactjs/react-tabs#571 ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
reactjs/react-tabs#525 · 1 comentário · 1 reação ·
-
Disabled tabs should remain keyboard focusable as per W3C WAI ARIA guidelinesTalvez livre de novo @Jastagar assumiu há 1050 dias e não há nenhum pull request aberto. Aberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
reactjs/react-tabs#521 · 3 comentários ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 42/100
reactjs/react-tabs#507 · 1 comentário ·
Todas as issues de reactjs/react-tabs
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
-
enhancement
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 85/100
holepunchto/hrpc-test#28 ·
-
good first issue
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 88/100
anoopcodehack/DevBoard#616 ·
Mantenedores costumam responder em até 2 dias
-
idnits3
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 63/100
ietf-tools/idnits#319 ·
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 76/100
521xueweihan/HelloGitHub#3939 ·