nextcloud / ios

📱 Nextcloud iOS App
https://itunes.apple.com/us/app/nextcloud/id1125420102
GNU General Public License v3.0
1.93k stars 875 forks source link

Live Photos viewed in Nextcloud iOS app appear in SDR instead of HDR #2452

Open IamPicklerick333 opened 1 year ago

IamPicklerick333 commented 1 year ago

Description:

I have been using the Nextcloud iOS app to automatically upload Live Photos in their original quality. However, when I view these photos within the app, they are displayed in SDR instead of HDR.

To further investigate, I performed the following tests:

I downloaded a Live Photo within the Nextcloud app, and it downloaded the SDR version of the photo. I selected "Open with..." from the app's menu and chose "Save Image." This action saved the photo to the iOS Photos app, where I was able to view it in HDR. However, it was no longer recognized as a Live Photo.

Steps to reproduce

  1. Use the Nextcloud iOS app to automatically upload Live Photos.
  2. Open the Nextcloud app and navigate to the uploaded Live Photos.
  3. Observe that the Live Photos are displayed in SDR instead of HDR.

Expected behaviour

When viewing or downloading Live Photos within the Nextcloud iOS app, they should be displayed in HDR to preserve their original dynamic range.

Actual behaviour

The images are being converted from HDR to SDR within the Nextcloud app when viewing photos and when downloading Live Photos.

Reasoning or why should it be changed/implemented?

Images should retain their original quality.

Environment data

iOS version: iOS 16.4.1

Nextcloud iOS app version: 4.8.0.5

Server operating system: Linux 5.19.0-43-generic x86_64

Web server: Apache, nginx

Database: PostgreSQL

PHP version: 8.0.28

Nextcloud version: 26.0.2

mpivchev commented 1 year ago

I think our server does not support Live Photos yet. Am I correct? @juliushaertl @marinofaggiana

juliushaertl commented 1 year ago

Depends on what you mean with support. Those are just files in the end, so they should be supported. Preview generation might be a different topic and depends on the server settings: https://github.com/nextcloud/server/pull/28077

marinofaggiana commented 1 year ago

No @mpivchev but we have open a issue for a probabile implementation of Live on Server, having said that, the problem here is not the live photo but the format of the 'photo' part