improve Table API with regards to Sorting
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 55/100
- Tipo de issue
- Refatoração
- Clareza
- Razoavelmente clara
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- react, typescript
- Domínio
- frontend
Direção de pesquisa
Start by reading the Table component API and its sorting implementation, then find the tests that cover sortable columns. Remove columns.isSortable and add sort.sortableColumns as an array of column names; update the API tests to verify sorting uses those names and that columns, rows, and sort configuration stay aligned.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Presently it is possible to express in Table component API invalid configuration.
It is valid to set columns.isSortable = true and not provide the sort object. We could validate this via a custom Prop validation function, but I think it more elegant to:
- remove the
columns.isSortableparam - add new
sort.sortableColumnsparam holding an array of column names
With this change the user only has to ensure that the column names within params columns, rows, and sort match.
- Linguagem predominante
- TypeScript
- Estrelas
- 23
- Forks
- 10
- Merge médio
- 5d 12h
- PRs com merge (30d)
- 9
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Sem guia de contribuição
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-ui-org/react-ui
-
maintenance
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
react-ui-org/react-ui#784 ·
Mantenedores costumam responder em até 1 dia
-
accessibility bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
react-ui-org/react-ui#775 ·
Mantenedores costumam responder em até 1 dia
-
Update ESLint to version 10Abertamaintenance
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 63/100
react-ui-org/react-ui#753 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Playwright container fails `devEngines` check again because its image does not pin Node.jsTalvez já em andamento @bedrich-schindler assumiu hoje. Abertabug maintenance
Dificuldade 4/5 1-2 dias Facilidade para iniciantes 25/100
react-ui-org/react-ui#782 ·
Mantenedores costumam responder em até 1 dia
-
Supported versionsAbertamaintenance
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 42/100
react-ui-org/react-ui#780 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de react-ui-org/react-ui
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
vercel-labs/durable-harness#53 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
area/docs area/ui enhancement
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 88/100
melgarafael/DeskcommCRM#4098 ·
Mantenedores costumam responder em até 1 dia
-
bug ready-for-agent
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 86/100
kerti/balances-v2#801 ·
Mantenedores costumam responder em até 1 dia
-
lesson-submission pending-review
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 84/100
Ikalus1988/MisakaNet#4510 ·
Mantenedores costumam responder em até 1 dia