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

Choosing to make an image "first" removes all edits from photos #1294

Closed
tiwane opened this issue Aug 14, 2023 · 2 comments
Closed

Choosing to make an image "first" removes all edits from photos #1294

tiwane opened this issue Aug 14, 2023 · 2 comments
Assignees
@tiwane
Labels
bug

Comments

@tiwane
Copy link

tiwane commented Aug 14, 2023

From https://forum.inaturalist.org/t/duplicating-and-or-cropping-photos-for-detail-changing-order-to-1st-photo-undoes-edits/43860 I can replicate on 1.29.5 (579).

Step 1: Make a new observation with at least 2-3 photos on mobile

Step 2: Edit one or more photos for brightness, contrast, cropping, etc

Step 3: Move one to 1st photo using the “Make 1st” function at top right

Step 4: All automatically reset to original parameters
@tiwane tiwane added the bug label Aug 14, 2023
@tiwane tiwane self-assigned this Aug 14, 2023
@tiwane
Copy link
Author

tiwane commented Aug 14, 2023

Log files at https://inaturalist.slack.com/archives/G041A2Z1W/p1692055826344689

@tiwane
Copy link
Author

tiwane commented Aug 21, 2023

Looks good in 1.29.7+581

budowski added a commit that referenced this issue Aug 24, 2023
@budowski
#1294 - remember photo edits when setting a photo as first
914913b
@kueda kueda closed this as completed Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tiwane tiwane

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kueda @tiwane