improve Table API with regards to Sorting
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 55/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- react, typescript
- Bereich
- frontend
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 23
- Forks
- 10
- Ø Merge
- 5 T. 12 Std.
- Gemergte PRs (30 T.)
- 9
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Kein Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus react-ui-org/react-ui
-
maintenance
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
react-ui-org/react-ui#784 ·
Maintainer antworten meist innerhalb von 1 Tag
-
accessibility bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
react-ui-org/react-ui#775 ·
Maintainer antworten meist innerhalb von 1 Tag
-
maintenance
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 63/100
react-ui-org/react-ui#753 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Playwright container fails `devEngines` check again because its image does not pin Node.jsEvtl. vergeben @bedrich-schindler hat das heute übernommen. Offenbug maintenance
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 25/100
react-ui-org/react-ui#782 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Supported versionsOffenmaintenance
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 42/100
react-ui-org/react-ui#780 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in react-ui-org/react-ui
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
vercel-labs/durable-harness#53 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
area/docs area/ui enhancement
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
melgarafael/DeskcommCRM#4098 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug ready-for-agent
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
kerti/balances-v2#801 ·
Maintainer antworten meist innerhalb von 1 Tag
-
lesson-submission pending-review
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 84/100
Ikalus1988/MisakaNet#4510 ·
Maintainer antworten meist innerhalb von 1 Tag