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

Trash is sorted differently on web and mobile #11820

Closed
1 of 3 tasks
JordyEGNL opened this issue Aug 15, 2024 · 3 comments · Fixed by #13299
Closed
1 of 3 tasks

Trash is sorted differently on web and mobile #11820

JordyEGNL opened this issue Aug 15, 2024 · 3 comments · Fixed by #13299

Comments

@JordyEGNL
Copy link
Contributor

JordyEGNL commented Aug 15, 2024

The bug

On the web version of Immich the trash items are sorted by newest first.
On the mobile version of Immich (Android in my case) this is the reverse, the oldest assets are shown on top.

Expected behavior: the trash items are also sorted by newest first like the web version

Mobile Web
Screenshot_20240815-162322_Immich.png markup_1000020720.png
The oldest assets are shown at the top The oldest assets are shown at the bottom

The OS that Immich Server is running on

Ubuntu 22.04 LTS

Version of Immich Server

v1.112.0

Version of Immich Mobile App

v1.112.0

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

NA

Your .env content

NA

Reproduction steps

1. Trash items with different dates
2. View the trash on web -> newest on top
3. View the trash on mobile -> oldest on top

Relevant log output

No response

Additional information

No response

@yashrajjain726
Copy link
Contributor

Hey @JordyEGNL , I can start working on this. So, should I make it as same as web sort ?

@JordyEGNL
Copy link
Contributor Author

Hey @JordyEGNL , I can start working on this. So, should I make it as same as web sort ?

In my opinion, yes, the most recently deleted items should be shown at the top

@yashrajjain726
Copy link
Contributor

yashrajjain726 commented Oct 9, 2024

Added changes as required. Please let me know, if any more changes is required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants