Slideshow viewer not adapting sorting mode of files app

[stephanhoennekes]

Actual behavior:
If the slideshow viewer is opened from the files app, its always using standard ascending/alphabetical order by file name (when clicking the arrows).

Expected behavior:
If a descending order by name or any order by file date or file size is chosen in the files app, the slideshow viewer should adopt this behavior by clicking the arrows as it was the case with the old gallery app.

[jakobroehrl]

@skjnldsv
How to fix this?
Does https://github.com/nextcloud/viewer/blob/master/src/services/FileList.js needs an sort parameter?

[skjnldsv]

@jakobroehrl

viewer/src/utils/fileUtils.js

Line 64 in c534e51

const sortCompare = function(fileInfo1, fileInfo2, key, asc = true) {

viewer/src/views/Viewer.vue

Lines 375 to 378 in 4280f99

	// sort like the files list
	// TODO: implement global sorting API
	// https://github.com/nextcloud/server/blob/a83b79c5f8ab20ed9b4d751167417a65fa3c42b8/apps/files/lib/Controller/ApiController.php#L247
	this.fileList = filteredFiles.sort((a, b) => sortCompare(a, b, 'basename'))

[jakobroehrl]

Thanks @skjnldsv
But how can I query the users sort criteria (by name, size or modified) from files app?

[skjnldsv]

This is unfortunately not implemented yet.
Ideally this should be a proper standard API :/

[jakobroehrl]

So then I can't solve this Issue, right?

[valdearg]

Did we ever get any further with this issue? It's frankly insane that after two years we still can't view images in order.

Is there anything we can do to get this put on some priority list?

[cmprmsd]

Since this issue might take another 10 years of resolving if there are central api changes needed, might it be possible to at least fix the natural sorting of images manually?

Having files behaving differently in the file viewer and slideshow is a no brainer especially on the mobile website.

I'm referring to the wrong sorting of
1
10
100
2
20
200
3
...
in the slideshow whereas the file viewer sorts correctly.

[johnnyq]

Yes this is completely frustrating and should be way up there on the priority list, although it may not be as easy to fix as one would think

[TheAnachronism]

Yes this is completely frustrating and should be way up there on the priority list
Agree, basically makes the entire image display functionality for me useless because I can't just rename around 10k images so they have a name, for Nextcloud to correctly order them.

Why isn't there more discussion or attention on this??

[Jerome-Herbinet]

Hello,

In my humble opinion, this functional bug should be fixed as a matter of urgency because since the acceleration and highlighting of Nextcloud Photos "2.0" and especially of the "viewer" which now incorporates a retouching tool, this will lead to a lot of disappointments (too bad for a "basic" function).

I hope you don't see any aggression in my comment, because I know you have a lot of other things to think about 😉, but I always imagine that when developing software it is wiser to consolidate the basic functionality before adding new features.

[TommySchwartz]

Is this the main and officially thread for adding a fix-this vote for the sorting disorder?
Found I don't know how many threads about this problem but most of them pointed here, and this issue is still open.

For me I don't care about exif sorting for now, I just want the sorting to be as shown in the actual folder, name, size or last changed.

Otherwise really thankful about nextcloud, its the perfect software in this tech dystopian times.

[marco-calautti]

This issue is still present in the latest stable version of Nextcloud. It has been 3 years now. Are there any plans to fix this?

[cmprmsd]

No plans :D

[marco-calautti]

No plans :D

That's something I would like to hear from the developers.

[cmprmsd]

Look at the close comments on some of the other issues.

skjnldsv closed this as not planned on Sep 8, 2022

Sad reality.

[johnnyq]

@cmprmsd I think this will be fixed sooner now since Nextcloud Files 28.0 adopted the newer backend libraries, so it would make sense to fix it now.