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

Hidden gallery is not hidden when picking a photo #6096

Closed
ara4n opened this issue May 1, 2022 · 0 comments · Fixed by #6143
Closed

Hidden gallery is not hidden when picking a photo #6096

ara4n opened this issue May 1, 2022 · 0 comments · Fixed by #6143
Labels
A-Composer A-Media O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@ara4n
Copy link
Member

ara4n commented May 1, 2022

Steps to reproduce

  1. Hide a photo in the iOS photos app
  2. Select a photo to send in Element
  3. Observe that the Hidden gallery is shown in the selection list… and shows the thumbnail of the most recent hidden photo in the UI
  4. Therefore it’s impossible to select a photo without displaying hidden photos
  5. This could be incredibly disasterous for someone demoing iOS, for obvious reasons

Outcome

What did you expect?

Hidden photos should be hidden

What happened instead?

Hidden photo get outed whenever you select a photo to send somewhere

Your phone model

12 pro max

Operating system version

15.4

Application version

latest tf

Homeserver

No response

Will you send logs?

No
@ara4n ara4n added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label May 1, 2022
@pixlwave pixlwave added A-Composer A-Media S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely labels May 11, 2022
@pixlwave pixlwave mentioned this issue May 11, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Composer A-Media O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Hide the hidden album's thumbnail. element-hq/element-ios
2 participants
@ara4n @pixlwave