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

Some sorting fixes. #1526

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Some sorting fixes. #1526

wants to merge 2 commits into from

Conversation

hfiguiere
Copy link
Member

Bug #1286 - the proper sort is dateTaken.

@hfiguiere
Copy link
Member Author

Rebased now.

@jmathai
Copy link
Member

jmathai commented Aug 20, 2014

@hfiguiere I'll look at this but I'm still confused about the use of dateTaken for sorting. Are you suggesting via this pull request that the current sorting algorithm should be changed (see #1341 for rationale of current sorting).

Just FYI, on the gallery tab the sorting is reverse chronological ordering by date/time uploaded. When viewing an album it's chronological by day and within each day it's chronological by time.

@hfiguiere
Copy link
Member Author

What you are saying that sorting is inconsistent and can't be configured? What I'm proposing is consistent sorting that still can't be configured.

@jmathai
Copy link
Member

jmathai commented Aug 21, 2014

If you look at the development branch you'll see that you can change the sort order; though it is not sticky. The development branch supports user defined sorting.

There's a lot of discussion in #1341 with regard to different sorting on different pages that I don't want to discard.

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.

2 participants