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

Photo picker for stories cannot pick new photos #5478

Closed
3 tasks done
WhyNotHugo opened this issue Nov 10, 2022 · 25 comments
Closed
3 tasks done

Photo picker for stories cannot pick new photos #5478

WhyNotHugo opened this issue Nov 10, 2022 · 25 comments
Labels

Comments

@WhyNotHugo
Copy link

WhyNotHugo commented Nov 10, 2022

  • I have searched open and closed issues for duplicates
  • I am submitting a bug report for existing functionality that does not work as intended
  • This isn't a feature request or a discussion topic

Bug description

It seems Signal uses its own custom photo picker instead of the iOS one. The problem is that a custom photo picker can only enumerate photos for which I’ve granted Signal access. So, in my case, I only see a single one.

The only workaround is to go into iOS settings > Signals > Photos > Edit Selected Photos, pick the new photo I want to upload there (eg: grant Signal access to it), then jump back into Signal and redo the above steps, and the photo is pickable.

Clearly the workaround is a huge pain, and I suspect most iOS users don't even know how to dig into settings to fix this. The photo also has to be picked twice: once in settings and once again in signal.

Steps to reproduce

Many months ago, I used the "share photo" option on a conversation on Signal. At the time, I picked a single photo to share (the one I was about to send). This is relevant context; the first time you use Signal's photo picker it will work, but only the first time.

  • Click on the stories tab
  • Tap on the button to upload a new one
  • Only the photo to which Signal has already been granted access is show.

Actual result

I can only pick the photo to which Signal has already been granted access.

Expected result

I should be able to pick another, different, photo.

Device info

Device: iPhone 11 Pro

iOS version: 16.0

Signal version: 5.59.10

Possible solution

The obvious solution here is: simply use iOS image picker. The current approach won't work because it only lists photos to which Signal has been granted access (but Signal has no shortcut to change this without digging into settings).

I'm not entirely sure of the exact API (I'm an iOS user, not developer), but apps like Fastmail get this right: when I want to attach a photo to an email, it pops up iOS's photo picker, which allows me to pick any photo on my phone.

Alternatives considered

Instagram tries to work around this by showing a link on the photo picker to use the native iOS photo picker to select new photos. With this approach, one can easily grant access to new photos, but when uploading a new story, one has to pick the photo twice, first on the iOS photo picker, and then on Instagram's internal one. I don't know how they consider this good UX.

@EvanHahn-Signal
Copy link
Contributor

Thanks for filing this. We'll figure out the right thing to do here.

@stale
Copy link

stale bot commented Feb 13, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 13, 2023
@WhyNotHugo
Copy link
Author

WhyNotHugo commented Feb 13, 2023 via email

@stale stale bot removed the stale label Feb 13, 2023
@stale
Copy link

stale bot commented May 15, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 15, 2023
@WhyNotHugo
Copy link
Author

Still an issue on 6.22.0.9

@stale stale bot removed the stale label May 15, 2023
@Miguel-Signal
Copy link

Hi @WhyNotHugo

Just to confirm, have you enabled permissions for All Photos within your iOS Settings for Signal?

iPhone Settings > Signal > Photos > Allow Photos or All Photos
iPhone Settings > Privacy > Photos > Allow Photos or All Photos
iPhone Settings > General > Restrictions > Disable restrictions > then restart the phone and try again.

@WhyNotHugo
Copy link
Author

WhyNotHugo commented May 16, 2023 via email

@stale
Copy link

stale bot commented Aug 18, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 18, 2023
@WhyNotHugo
Copy link
Author

WhyNotHugo commented Aug 18, 2023 via email

@stale stale bot removed the stale label Aug 18, 2023
Copy link

stale bot commented Nov 16, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 16, 2023
@WhyNotHugo
Copy link
Author

WhyNotHugo commented Nov 16, 2023 via email

@stale stale bot removed the stale label Nov 16, 2023
Copy link

stale bot commented Feb 15, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 15, 2024
@dabo248
Copy link

dabo248 commented Feb 15, 2024

It's still an issue and bothering me every time I want to send a photo.

@stale stale bot removed the stale label Feb 15, 2024
Copy link

stale bot commented May 16, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 16, 2024
@dabo248
Copy link

dabo248 commented May 16, 2024

bump

@stale stale bot removed the stale label May 16, 2024
@killalau
Copy link

killalau commented Jun 27, 2024

In my settings, there are only 2 options "None" and "Add Photos Only", there is no way for me to add a phone in-app in general. Do we need a "Full Access" on photos?

My spec:

  • Device: iPhone 15 Plus
  • iOS Version: 17.5.1
  • Signal Version: 7.17(171)

@ArthurHNL
Copy link

ArthurHNL commented Jul 1, 2024

The problem here is that Signal implemented its own photo picker instead of using the one provided by iOS. This requires Signal to have access to all photos of the user for the user to be able to pick a desired photo. A workaround is going into the settings to manually select photos that Signal is allowed access to and then selecting these photos again in Signal but this, to my opinion, not user friendly.

To properly fix this, in my opinion, Signal should migrate to using the system photos picker that does not require any permissions. Or at the very least provide a link to the settings app where access can be granted.

@ArthurHNL
Copy link

In my settings, there are only 2 options "None" and "Add Photos Only", there is no way for me to add a phone in-app in general. Do we need a "Full Access" on photos?

My spec:

  • Device: iPhone 15 Plus
  • iOS Version: 17.5.1
  • Signal Version: 7.17(171)

@killalau

It is possible to grant access to the full photo library (at least on my iPhone the option is there), though simply not desired in my opinion. Using the system photo picker also provides a unified experience for users across apps.

If you really do not have the option, maybe you are using a work/school managed device that has policies preventing this?

These are the options I have (pardon my Dutch, “beperkte toegang” means limited access and “volledige toegang” means full access).

IMG_0245

@akwick
Copy link

akwick commented Jul 31, 2024

@WhyNotHugo My workaround for this is issue is to open Photos, select the image/video I want to share, click on sharing, and share it via Signal. In the case that I want to sent the same image/video to multiple people, I forward it via Signal. I found it "less" painful than changing the settings. 😊

Copy link

stale bot commented Nov 4, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 4, 2024
@SaltyPeppermint
Copy link

This issue is still not resolved

@moritzbindewald
Copy link

Pls fix this is utterly annoying...

Copy link

stale bot commented Dec 8, 2024

This issue has been closed due to inactivity.

@stale stale bot closed this as completed Dec 8, 2024
@WhyNotHugo
Copy link
Author

WhyNotHugo commented Dec 8, 2024 via email

@WhyNotHugo
Copy link
Author

See #5921 as a follow-up

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

No branches or pull requests

9 participants