opengisch / QField-docs

Documentation for the QField ecosystem
https://docs.qfield.org
Creative Commons Attribution Share Alike 4.0 International
37 stars 75 forks source link

Update pictures.en.md #501

Closed cemno closed 1 month ago

cemno commented 1 month ago

The current explanation is somewhat confusing, as the native camera of most smartphones is capable of geotagging, while the actual problem lies in not providing this information to QField (For example iOS devices). Most non technical users would think there native camera app can geotag images, so it should qualify. This clarification should help understanding that the transfer of this information is the crucial part.

See #5532

m-kuhn commented 1 month ago

Thank you ! IIRC @nirvn was considering to switch the default cam to the internal one, as most systems nowadays block this access and the internal camera got quite powerful.

nirvn commented 1 month ago

@m-kuhn , that is correct. As of QField 3.4, the internal QField camera is the default one.

m-kuhn commented 1 month ago

@cemno thanks a lot for proposing this update, I have opened a pull request with a slightly adjusted version. https://github.com/opengisch/QField-docs/pull/504 What do you think about it?

cemno commented 1 month ago

Thanks, a complete rewrite due to the changed default behaviour is indeed the best. I would suggest mentioning the on/off button for geotagging (with the camera ux update this would be a camera setting within the camera itself). This would make it clear what to look for when deciding whether or not to use this feature. Should I make a suggestion at #504? Or where would be the right place?

m-kuhn commented 1 month ago

Thanks for your feedback, such an addition would be great. I have just merged #504, so you can suggest anything on top of the current master version

suricactus commented 1 month ago

I am closing this due to the conflict and agreement to open a follow-up PR.