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

EXIF data are removed #368

Open
artonge opened this issue Aug 14, 2023 · 4 comments
Open

EXIF data are removed #368

artonge opened this issue Aug 14, 2023 · 4 comments
Labels
Possible Enhancment A feature might be useful to get added in the editor but not decided yet whether to consider or not

Comments

@artonge
Copy link

artonge commented Aug 14, 2023

When editing a picture, its EXIF data are removed.

I think it is expected for those data to be kept in the new version of the picture.

Here is a simple example:

Screenshot from 2023-08-14 10-38-02

@AhmeeedMostafa
Copy link
Collaborator

@artonge yes it is expected to lose them, as we already create a new image and we don't consider/care for the exif data as already this is a new image created in any case, so if u need to consider it u should do that from ur side by reading the exif data before (in the getting edited image) and apply it after (in the new edited image)

I'm adding this as possible enhancement but not sure if we will consider it in our features plan or not as it is still possible to be applied from ur side.

@AhmeeedMostafa AhmeeedMostafa added the Possible Enhancment A feature might be useful to get added in the editor but not decided yet whether to consider or not label Sep 11, 2023
@skjnldsv
Copy link

I'm adding this as possible enhancement but not sure if we will consider it in our features plan or not as it is still possible to be applied from ur side.

If we find a way and open a PR for it, will you consider it?

@AhmeeedMostafa
Copy link
Collaborator

@skjnldsv yes but depending on its implementation size (if it will take big size or cause any performance issue we won't otherwise it's fine to consider),
and it should be configured whether to enable/disable it through the properties.

@artonge
Copy link
Author

artonge commented Sep 15, 2023

Maybe a lead if no lib already exists:
https://getaround.tech/exif-data-manipulation-javascript/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Possible Enhancment A feature might be useful to get added in the editor but not decided yet whether to consider or not
Projects
None yet
Development

No branches or pull requests

3 participants