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

[5.x] Add UI for listing field sortable config #10259

Merged
merged 5 commits into from
Jun 25, 2024

Conversation

ryanmitchell
Copy link
Contributor

Its totally possible I'm being dumb and not noticing it elsewhere, but it seems theres no UI for controlling whether a field is sortable in a listing view (even though there is a config setting for it).

This PR adds a toggle field to the common field options, which is only shown when the field is listable (or hidden by default) and when its not a computed field:

CleanShot 2024-06-04 at 09 34 21@2x

ryanmitchell and others added 5 commits June 4, 2024 09:32
the condition isn't right, it should also be shown when listable is "hidden by default"
but then there was issues doing "lisable: not true" and i dont want to go down that rabbit hole.

other fields in that settings view are still shown when they arent applicable anyway.
@jasonvarga jasonvarga merged commit 055f752 into statamic:5.x Jun 25, 2024
16 checks passed
@ryanmitchell ryanmitchell deleted the fix/sortable-ui branch June 25, 2024 18:02
@ryanmitchell
Copy link
Contributor Author

Thank you

duncanmcclean added a commit to statamic/docs that referenced this pull request Jul 31, 2024
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.

2 participants