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.

Sign up for GitHub

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

Jump to bottom

Inconsistent photo sorting by date taken #1339

Closed
sushimustwrite opened this issue Jul 23, 2013 · 6 comments
Closed

Inconsistent photo sorting by date taken #1339

sushimustwrite opened this issue Jul 23, 2013 · 6 comments

Comments

@sushimustwrite
Copy link
Member

The current photo sorting is reverse chronological order (date taken). But within a given date, the photos show in forward chronological order. This doesn't make much sense. Is this intentional? And if so, why?
@hfiguiere
Copy link
Member

Known bug. See bug #1286

@sushimustwrite
Copy link
Member Author

No it's not. At least not covered by that one (which I did find and read through, by the way). This issue covers photos that are dated properly but are sorted in an order that isn't consistent with the default overall sort order. Per this comment on that issue the ordering is not in itself a bug. I'm asking whether to consider a more consistent ordering.

@sneakypete81
Copy link
Contributor

This is #735.

The current sort order is quite confusing for new users, and a real pain for albums that cover events spanning multiple days.

@sushimustwrite
Copy link
Member Author

That's it. Kinda surprised that one didn't turn up in my searching (or I just didn't notice) and many times going through these bugs. Thanks.

@hfiguiere hfiguiere mentioned this issue Jul 24, 2013
Open
@jmathai jmathai mentioned this issue Jul 26, 2013
Open
@jmathai
Copy link
Member

jmathai commented Jul 26, 2013

Taking a step back and opening this up for discussion with the hopes that we can decide on a good default based on all the perspectives we get. #1341

jmathai added a commit that referenced this issue Oct 18, 2013
@jmathai
Changing date sort logic to not group by date. See #1341 for discussion.
30d90bb 
Closes #1341 (meta)
Closes #735
Closes #1339
Closes #1286
Closes #1340
@hfiguiere
Copy link
Member

Real fix for this attached to PR #1526

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jmathai @sneakypete81 @hfiguiere @sushimustwrite