improve Table API with regards to Sorting
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 55/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- Calme
- Stack technique
- react, typescript
- Domaine
- frontend
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- TypeScript
- Étoiles
- 23
- Forks
- 10
- Merge moyen
- 5 j 14 h
- PR mergées (30 j)
- 9
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Aucun guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de react-ui-org/react-ui
-
maintenance
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
react-ui-org/react-ui#784 ·
Les mainteneurs répondent en général sous 1 jour
-
accessibility bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
react-ui-org/react-ui#775 ·
Les mainteneurs répondent en général sous 1 jour
-
Update ESLint to version 10Ouvertemaintenance
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
react-ui-org/react-ui#753 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Supported versionsOuvertemaintenance
Difficulté 4/5 3-5 jours Accessibilité débutants 42/100
react-ui-org/react-ui#780 ·
Les mainteneurs répondent en général sous 1 jour
-
feature
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
react-ui-org/react-ui#778 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de react-ui-org/react-ui
Issues similaires
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulté 1/5 Moins d'une heure Accessibilité débutants 83/100
lingdojo/kana-dojo#33701 · 1 commentaire · 5 réactions ·
Les mainteneurs répondent en général sous 1 jour
-
tech-debt
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
rookslog/zlibrary-mcp#408 ·
Les mainteneurs répondent en général sous 1 jour
-
mp4box.js — 20-byte `sgpd` box with large `entry_count` causes unbounded loop and heap OOM abortOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
status: needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
mastra-ai/mastra#29641 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour