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.

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

Sortable resources tables #1744

Merged
merged 4 commits into from
Mar 12, 2020
Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Mar 12, 2020

What is this pull request for?

Add default sort order to resources tables.

Notable changes

Before we where relying on the default sort order of the database. That is, but does not have to be, the primary column in most databases.

This changes this behavior to introduce a default_sort_order that uses the name column (if the resource has one) or the first attribute in an ascending sort order.

Also adds a nice little indicator to the sorted column.

Screenshots

Screen Shot 2020-03-12 at 16 51 20

Screen Shot 2020-03-12 at 16 52 04

Before we where relying on the default sort order of the database.
That is, but does not have to be, the primary column in most databases.

This changes this behavior to introduce a default_sort_order that
uses the name column (if the resource has one) or the first attribute
in an ascending sort order.

Also adds a nice little indicator to the sorted column.
@tvdeyen tvdeyen added this to the 5.0 milestone Mar 12, 2020
@tvdeyen tvdeyen merged commit ec17038 into AlchemyCMS:master Mar 12, 2020
@tvdeyen tvdeyen deleted the sortable-resources-tables branch March 12, 2020 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant