Skip to content

Enable to choose sort order#737

Open
coooldoggy wants to merge 5 commits intozhihu:masterfrom
coooldoggy:master
Open

Enable to choose sort order#737
coooldoggy wants to merge 5 commits intozhihu:masterfrom
coooldoggy:master

Conversation

@coooldoggy
Copy link
Copy Markdown

we can support multiple options to show album sorting orders and able to fix downloaded file order seems random.
Former version only show album order by date taken, so that user can be confused by the difference of native gallery order.

[Usage]

.albumOrder()

"taken" : order by date taken
"added" : order by date added
"size" : order by file size
default : order by default album order

we can support multiple options to show album sorting orders and able to fix downloaded file order seems random.
Former version only show album order by date taken, so that user can be confused by the difference of native gallery order.

[Usage]

.albumOrder()

"taken" : order by date taken
"added" : order by date added
"size" : order by file size
default : order by default album order
Copy link
Copy Markdown

@creativehackerhacks creativehackerhacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the sorting option.

@Mohammad-Adam
Copy link
Copy Markdown

a sort is a mandatory option, plz consider adding it ASAP

Copy link
Copy Markdown

@sumit1826 sumit1826 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@karique01
Copy link
Copy Markdown

please, i need this option, i can't find it in the v0.5.3-beta3 version, the albumOrder() method it's not in the SelectionCreator class

Copy link
Copy Markdown

@karique01 karique01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done

Copy link
Copy Markdown

@karique01 karique01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done

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.

6 participants