luyadev/luya-module-cms

Unexpected container order

hbugdoll opened this issue · 4 comments

What steps will reproduce the problem?

Add multiple containers in the CMS.

What is the expected result?

Containers are sorted like in CMS Settings > Containers and thus by ID.

What do you get instead?

Containers are sorted by name in alphabetical order.

Additional infos

Q A
LUYA Version current

@nadar If you agree I can provide a PR.

nadar commented

In an upcoming release we could add a migration with a sorting field. Maybe we could also use the alias to sort as they are not visible in the UI. So things like 0default 1abc would be possible? What do you think?

nadar commented

@rolandschaub recommendations?

In an upcoming release we could add a migration with a sorting field. Maybe we could also use the alias to sort as they are not visible in the UI. So things like 0default 1abc would be possible? What do you think?

Using the alias may be a workaround (but we heavily use "fixed" container aliases).
A new sorting ID would be the preferred way and consistent to menu item's sort_index.

nadar commented

A new migration would require us to release a new major version (5.0). So i will assign this milestone 👍