Page MenuHomePhabricator

Table: enable multi-level sortings
Open, Needs TriagePublic

Description

This task covers the expansion of the Table's sorting behavior T360078 to enable sorting by multiple columns at once.


Acceptance criteria

  • A new prop is added to enable multi-level sorting
    • allowMultiSort: This boolean prop can be used to indicate that more than one column can be sorted at a time. Otherwise, when one column is sorted, the other columns will become unsorted.
  • Sort icon click behavior is updated when multi-level sorting is enabled
  • Add unit tests
  • Add a new demo (that possibly makes an external API call on sort)

Event Timeline

CCiufo-WMF triaged this task as Medium priority.Apr 29 2024, 5:03 PM
CCiufo-WMF added subscribers: Sarai-WMDE, AnneT, CCiufo-WMF.

We decided to tentatively remove this from the scope of the Table MVP. Waiting on @AnneT and @Sarai-WMDE to clarify if this is incorrect.

AnneT renamed this task from Table: add multi-sort to Table: enable multi-level sortings.Apr 30 2024, 4:09 PM
AnneT updated the task description. (Show Details)

@CCiufo-WMF Sarai is going to look for use cases for multi-level sort to determine if this needs to be part of the Table MVP (or perhaps we could remove it from MVP regardless, and Sarai's research will determine if we should prioritize multi-level sort soon)

@lwatson I removed this as a subtask of T303320 to signal that it won't be included in the initial release of the Table component. Are we now intending to include multi-level sorting in the initial release?

Sorry, no intention of including multi-level sorting in the initial Table release. Removed the parent task.

We decided to tentatively remove this from the scope of the Table MVP. Waiting on @AnneT and @Sarai-WMDE to clarify if this is incorrect.

That sounds correct. I'm waiting to receive final confirmation from designers regarding a couple of table use cases, but so far it looks like multi-level sorting can be excluded from the table's MVP. This feature cannot be considered a requirement at the moment, since it's not featured by the detected use cases that the cdx-table should/could potentially cover.

CCiufo-WMF raised the priority of this task from Medium to Needs Triage.Thu, May 9, 4:02 PM
CCiufo-WMF moved this task from Up Next to Backlog on the Design-System-Team board.