I really like the New Views UI and the creation of a view has really improved. I think it serves both new and experienced users very well. One small thing I'm missing is an 'unsorted' option in the 'sorted by' select box. Sometimes I do not want to choose one of 'Newest first' 'Oldest first' or 'Title'.

CommentFileSizeAuthor
#5 Create-view-allow-sorted-by-none.patch1.15 KBmstrelan

Comments

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Fixed

That's actually a great idea from my perspective.

Added it and commited to 7.x-3.x

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented

Great, thanks!

mstrelan’s picture

mstrelan commented
Status: Fixed » Needs work

Just a VERY minor problem. The comment in code "Check if we are allowed to sort by creation date." should come AFTER the "Unsorted" option.

dawehner’s picture

dawehner
Primary language German
commented
Category: feature » bug
Status: Needs work » Active

Feel free to create a patch.

mstrelan’s picture

mstrelan commented
Status: Active » Needs review
StatusFileSize
new Create-view-allow-sorted-by-none.patch1.15 KB

I had a funny feeling :P

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
commented
Status: Needs review » Reviewed & tested by the community

Bonus points for perfectionism! I must admit I didn't test the patch, but it sure looks good to me.

dawehner’s picture

dawehner
Primary language German
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Commited.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.