patternfly / patternfly/patternfly-react
Table sorting needs clarification/revisement
Nobody has claimed this yet.
- 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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