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

HEIC support #72

Open
Antoine-lb opened this issue Feb 3, 2024 · 6 comments
Open

HEIC support #72

Antoine-lb opened this issue Feb 3, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Comments

@Antoine-lb
Copy link
Contributor

Antoine-lb commented Feb 3, 2024

There is more than one way to go about it:

  • Convert HEIC images to another format on the fly (like thumbnails)
  • Convert HEIC images definitely so you can get read of HEIC
  • Actually read and display HEIC

I'm not very educated in the matter, and probably there are more options than that, I'm open for comments

@Antoine-lb Antoine-lb self-assigned this Feb 3, 2024
@Antoine-lb Antoine-lb added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Feb 3, 2024
@kdjanz
Copy link

kdjanz commented Feb 3, 2024

I think the rule must be that OneFolder must Never Ever change the user's files without permission. So, you can either just display the HEIC or you can create a temporary JPG to display in the thumbnails - either option would be transparent to the user - we don't know and we don't care.

But if I right click on an image and get the option to "Convert to..." then I am asking you and giving you permission to mess with my files with my knowledge. It must be up to me whether I overwrite the HEIC or just add a duplicate file somewhere else that I can use to share or post or whatever. As part of the Conversion process, it would be nice if you give me the option to delete the duplicated HEIC, but that should not be the default unless the user has set that.

My opinions...

@fscheps
Copy link

fscheps commented Feb 4, 2024

I think if we can have the option to treat HEIC files the same way JPG files are treated, then everything would work well. I am not sure if HEIC files have the same specs as JPG with tags, keywords and all, then I would love to have tags saved on the file itself so then I could do the categorization work only once and no matter where I import my pics in the future all information would be preserved.

@Antoine-lb
Copy link
Contributor Author

@kdjanz Fully agree, we should even make it more clear that "OneFolder never modifies your files without confirmation".

For now the converting on the fly to JPEG seems to be the best option, it solves the problem and is not too much work.

The problem can be divided in two:

  • Converting the image in the full preview
  • Converting the thumbnails

In the full preview we could use heic2any to convert in the fly, seems straightforward.

The thumbnails may be a harder problem, sharp needs packages installed in the computer in order to work with HEIC, so maybe for a first version not show:
SCR-20240204-jutm

@Antoine-lb
Copy link
Contributor Author

Antoine-lb commented Feb 4, 2024

@fscheps yes it should be the same as JPG in regards of the metadata, because we use ExifTool under the hood and they support it pretty well. Tags should be saved in the file itself.

The problem is how to view those images on the app, because Chromium (like Google Chrome) doesn't support HEIC, and OneFolder being built on top of Chromium we don't support it either (e.g. we can't just display a HEIC the same way we do with PNG, we have to find a work around).

@kdjanz
Copy link

kdjanz commented Feb 4, 2024 via email

@AShortcuts
Copy link

This article by another photo management app might be helpful for integrating heic support

https://photostructure.com/getting-started/heif-support/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed question Further information is requested
Projects
Status: No status
Development

No branches or pull requests

4 participants