Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a default order prop in List Fields #4518

Merged
merged 7 commits into from
May 8, 2020
Merged

Add a default order prop in List Fields #4518

merged 7 commits into from
May 8, 2020

Conversation

JulienMattiussi
Copy link
Contributor

@JulienMattiussi JulienMattiussi commented Mar 12, 2020 •
edited
Loading

Closes #2736

Issue

In Lists, columns are always sorted by ASC order by default
Sometimes, it's not the main desired behaviour (recent dates before, .....)

Usage

<TextField
    source="title"
    sortBy="title"
    sortByOrder="DESC"
/>

Documentation

image

@JulienMattiussi JulienMattiussi added the WIP Work In Progress label Mar 12, 2020
@JulienMattiussi JulienMattiussi added RFR Ready For Review and removed WIP Work In Progress labels Mar 13, 2020
fzaninotto
fzaninotto requested changes Mar 29, 2020
View reviewed changes
docs/List.md Outdated Show resolved Hide resolved
docs/List.md Outdated Show resolved Hide resolved
packages/ra-core/src/controller/useListParams.ts Outdated Show resolved Hide resolved
packages/ra-core/src/controller/useListParams.ts Outdated Show resolved Hide resolved
@fzaninotto fzaninotto merged commit fdb2cb1 into next May 8, 2020
@fzaninotto fzaninotto deleted the default-order-in-sort branch May 8, 2020 15:17
@fzaninotto fzaninotto added this to the 3.6.0 milestone May 8, 2020
@fzaninotto fzaninotto mentioned this pull request May 8, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fzaninotto fzaninotto fzaninotto approved these changes

Assignees
No one assigned
Labels
RFR Ready For Review
Projects
None yet
Milestone
3.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@JulienMattiussi @fzaninotto