patternfly / patternfly/patternfly-react

Table sorting needs clarification/revisement

Open
#11,006 10 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

PF Team Stale
Dominant language
TypeScript
Stars
862
Forks
392
Avg merge
4d 8h
Merged PRs (30d)
9

Description

Describe the enhancement or change
From the 9/16/2024 design share session, sorting was a topic mentioned, including how "favorites" sorting should work (since typically there's no text to sort by, only an icon).

I gave my initial thoughts on this in the design share, mentioning that if you view it in a sort of alphabetical order, "Favorited" comes before "Unfavorited"/"Not favorited", so following that logic an ascending sort order (with the arrow UP icon) should sort rows so that favorited rows are first (right now our favoritable example does the opposite). Of course this logic kind of breaks depending on the verbiage you're using: "Starred" comes alphabetically after "Not starred". Perhaps if we view it more as an 'active" and "inactive" state universally that logic would fit in all situations, though.

What we should do:

  • Investigate how "favorited" rows should be sorted based on the sort type and what icon should be used; we should see if there's a common pattern for this elsewhere, then from there we may need to update logic in our code.

  • Depending on the outcome of the above, we should make sure we make applicable updates in the core repo as well as update any design guidelines

Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?

Any other information?


Jira Issue: PF-16

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Begin with the PatternFly table's favoritable example and review the sorting behavior described in the issue. Research common conventions for sorting favorited or active/inactive rows, then identify the required updates in the core repository and design guidelines. Done means the behavior and sort icon are agreed and the applicable documentation and implementation changes are defined.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
design, frontend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.