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

[Bug]: Order of images does not persist #4964

Open
pigsonthewing opened this issue May 21, 2022 · 7 comments · May be fixed by #5189
Open

[Bug]: Order of images does not persist #4964

pigsonthewing opened this issue May 21, 2022 · 7 comments · May be fixed by #5189
Labels

Comments

@pigsonthewing
Copy link

pigsonthewing commented May 21, 2022

Summary

If I select several images to share, they are presented in a different order to both that i which I select them, and in which they were taken.

[Not clear if #4392 is the same issue]

Steps to reproduce

  1. In device's gallery, select several images
  2. Share to Commons app
  3. Enter caption & description of first image
  4. Select "next"
  5. Repeat 3 & 4 for all images

Expected behaviour

The images should be presented in the order in which they were selected (or alternatively, ordered by timestamp)

Actual behaviour

The order of the images is apparently random.

Device name

Nord Plus One

Android version

Android 11

Commons app version

3.1.1~1c9267ca0

Device logs

No response

Screen-shots

No response

Would you like to work on the issue?

Prefer not

@nicolas-raoul
Copy link
Member

I can reproduce this.
It results in confusing titles.

Kyle-ZixinFeng added a commit to Kyle-ZixinFeng/apps-android-commons that referenced this issue Oct 19, 2022
zhihaoyuu added a commit to zhihaoyuu/apps-android-commons that referenced this issue Oct 19, 2022
…t-persist-#4964

Fix order of images does not persist commons-app#4964
@Kyle-ZixinFeng
Copy link

Hallo, I think I maybe can try to slove this bug, Could you assign this for me, please?

@nicolas-raoul
Copy link
Member

@cakepineapple When you have implemented your currently assigned issue, please ask again. Thanks for your interest!

@nicolas-raoul
Copy link
Member

@Kyle-ZixinFeng How is your progress? :-)

@JesusMurguia JesusMurguia linked a pull request Mar 24, 2023 that will close this issue
@nicolas-raoul
Copy link
Member

No news from Kyle, re-assigning to Jesus. Kyle if you are still around do not hesitate to collaborate with Jesus. :-)

@JesusMurguia
Copy link

JesusMurguia commented Mar 25, 2023

Can you assign this to me, please, i can work on this

@nicolas-raoul
Copy link
Member

@JesusMurguia Any progress to share? 🙂

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

Successfully merging a pull request may close this issue.

4 participants