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

On iOS, profile pic chooser forces camera #2311

Closed
ejheil opened this issue Dec 24, 2018 · 2 comments · Fixed by #2363
Closed

On iOS, profile pic chooser forces camera #2311

ejheil opened this issue Dec 24, 2018 · 2 comments · Fixed by #2363

Comments

@ejheil
Copy link
Contributor

ejheil commented Dec 24, 2018

While trying to do this:

Add a profile photo to a new Known site using Safari in iOS.

I encountered this error:

I couldn't upload one, only take a new one with the camera. This is because of the HTML5 media capture api: https://mobilehtml5.org/ts/?id=23

The text for the button says "select a user picture" so this is counterintuitive -- taking a new picture isn't "selecting" one.

Suggestion: just use a regular upload, or else provide two buttons, one for "take picture" and one for "upload picture."

Some other notes:

Give us some context:

  • It'd also be really handy if you could tell us the contents of your version.known file
    version = "0.9.9-a"
    build = 2018122001

  • What database are you using? (e.g. mongo, mysql, postgres)

mysql

  • Any warnings or errors in your admin/diagnostics page?

nope

  • If this is a programming bug, can you include examples of any Micropub / API calls / webhook pings you make? Otherwise please don't worry about what this means!
  • Bonus points - are you able to illustrate the issue with a unit test? If so, submit it as a pull request!
@mapkyca
Copy link
Member

mapkyca commented Dec 25, 2018

The capture API seems to have slightly different behaviour on every device, and we ran into this issue last year some time. Something must have changed, because it used to offer a choice between capture and camera roll.

@fozzedout
Copy link

This also happens on my Android device, so maybe it's a more generic change?

mapkyca added a commit to mapkyca/idno that referenced this issue Mar 7, 2019
Bringing this in line with that of the photos plugin, and hopefully avoid issues on mobile where camera capture is forced.

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

Successfully merging a pull request may close this issue.

3 participants