There ought to be some kind of note around Sort Criteria that warns users that using the default sorting with the table style will completely override any sorts added to the View. A good place to put it might be sort.html.

Comments

merlinofchaos’s picture

merlinofchaos commented

There is a checkbox in the table style that controls this behavior.

Island Usurper’s picture

Island Usurper commented

Yes, I know, but my point is that it took me a while to figure out what was causing the behavior. I kept fiddling with the Sort Criteria because I didn't remember that the table style had a default sort set, and that it was overriding the view's query.

I'm not looking to change the behavior, but just add a note into the inline help to explain what could happen. Since this happened in a default view I've put in my module, I'm going to tweak it to make more sense. Other people might make this mistake, though, so I want to help ease their confusion.

Steel Rat’s picture

Steel Rat commented

I have to agree with the poster. I had no idea why the sort criteria wasn't working at all until I saw this.

mdeaves’s picture

mdeaves commented

Thanks you sooo much. It is a very good point and I truly appreciate you starting this thread!

fnikola’s picture

fnikola commented

+1 for me on this documentation updates. I ran across this too and the Sort Criteria help should indicate that the default sort in the table style will override this behavior.

alynner’s picture

alynner commented

I just spent about an hour trying to figure out why sort criteria was having zero effect, thanks for this post and I agree, it would be nice to have been notified!

dawehner’s picture

dawehner
Primary language German
commented

So someone of you pick this up and write the documentation, this would be cool :)

esmerel’s picture

esmerel commented
Assigned: Unassigned » esmerel
esmerel’s picture

esmerel commented
Status: Active » Closed (duplicate)

There is a patch in #823330: Improve the Advance Help Documentation: Display Section that will clarify this in the documentation.

esmerel’s picture

esmerel commented
Status: Closed (duplicate) » Active

blast, I updated the wrong issue.

esmerel’s picture

esmerel commented
Status: Active » Closed (duplicate)

No, actually I didn't update the wrong one; the patch will update one of the main adv help pages to note that overrides are across the set.