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

FEATURE: Make taxonomies sortable #70

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lorenzulrich
Copy link

Since all Taxonomy items are nodes, the sortingIndex of the nodes is already respected. Instead of just having an alphabetical sorting by title, we allow sorting taxonomies in the backend module. This is helpful if taxonomies are used e.g. as a filter in the frontend and you explicitly want to sort the items.

@lorenzulrich
Copy link
Author

@mficzel As discussed in Slack, this is a first version of my feature that allows sorting taxonomies. What do you think?

sitegeist-taxonomy-sortable.mp4

@mficzel
Copy link
Member

mficzel commented Dec 10, 2023

Not that sure this is consistent across dimensions. that would be important to me.

It will probably have to be implemented again for 9.0 as this is in many aspects a rewrite.

@lorenzulrich
Copy link
Author

Good point. I checked and it isn't consistent (as it is with nodes in the document tree). But it can be solved checking otherNodeVariants. I will update the code and push a new version.

Since all Taxonomy items are nodes, the sortingIndex of the nodes is already
respected. Instead of just having an alphabetical sorting by title, we allow
sorting taxonomies in the backend module. This is helpful if taxonomies are used
e.g. as a filter in the frontend and you explicitly want to sort the items.
@lorenzulrich
Copy link
Author

Not that sure this is consistent across dimensions. that would be important to me.

Just to let you know, the current version now sets the sorting index across all dimensions.

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.

None yet

2 participants