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

Order contained content items in ascending order #7406

Merged
merged 1 commit into from
Aug 26, 2019

Conversation

ramiassi
Copy link
Contributor

As widgets and all other elements are ordered by position in ascending order. Why the contained items are ordered in descending order?

@sebastienros
Copy link
Member

Can you provide a screenshot of the end result? Or some repro showing the issue?

@ramiassi
Copy link
Contributor Author

ramiassi commented Nov 18, 2016

Right now what is happening is like this:
a) If in admin -> widgets:
widget1.Position = 10;
widget2.Position = 20;
widget3.Position = 30;
widget4.Position = 40;
They are displayed in this order:

  1. widget1
  2. widget2
  3. widget3
  4. widget4
    b) after attaching the containable part to a content type and allowing the user to set position:
    if the user set the positions like this:
    containedItem1.Position = 10;
    containedItem2.Position = 20;
    conatinedItem3.Position = 30;
    containedItem4.Position = 40;
    then they are displayed in this order in the container’s item detailed view page:
  5. containedItem4
  6. containedItem3
  7. containedItem2
  8. containedItem1

@OrchardCMS OrchardCMS deleted a comment from dnfclas Dec 11, 2017
@BenedekFarkas BenedekFarkas self-requested a review August 26, 2019 15:41
@BenedekFarkas BenedekFarkas added this to the Orchard 1.11 milestone Aug 26, 2019
@BenedekFarkas BenedekFarkas merged commit 2db5255 into OrchardCMS:dev Aug 26, 2019
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.

5 participants