entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 38 Releases Activity

Retirer ExifRestorer (#75178) #152

Closed
bdauvergne wants to merge 1 commits from wip/75178-Retirer-l-application-d-ExifRest into main
pull from: wip/75178-Retirer-l-application-d-ExifRest
merge into: entrouvert:main
Conversation 3 Commits 1 Files Changed 1 +10 -206
bdauvergne commented 2023-03-06 20:55:44 +01:00
Owner
Copy Link

It was added to keep image orientations but all modern browsers now
comply with the orientation Exif tag.

It was added to keep image orientations but all modern browsers now comply with the orientation Exif tag.
bdauvergne added 1 commit 2023-03-06 20:55:45 +01:00
gitea/wcs/pipeline/head This commit looks good Details
e23e44336c misc: remove copy of Exif tags when resizing images (#75178) 
It was added to keep image orientations but all modern browsers now
comply with the orientation Exif tag.
bdauvergne commented 2023-03-06 21:48:03 +01:00
Author
Owner
Copy Link

J'ai testé sur chrome et firefox en modifiant une image via @mogrify -orient@ et ça marche bien. Je testerai sur un iPhone demain.

J'ai testé sur chrome et firefox en modifiant une image via @mogrify -orient@ et ça marche bien. Je testerai sur un iPhone demain.
fpeters requested changes 2023-03-07 08:49:55 +01:00
fpeters left a comment
Owner
Copy Link

Mais il n'y a pas que l'orientation dans les données exif.

Mais il n'y a pas que l'orientation dans les données exif.
bdauvergne commented 2023-03-07 08:52:54 +01:00
Author
Owner
Copy Link

Mais il n'y a pas que l'orientation dans les données exif.

Ok mais est-ce qu'on peut préciser ce qu'on souhaite conserver ? Dans l'autre ticket je rapporte un commentaire de Nico dans le ticket initial qui parle juste d'orientation et d'"interpolation linéaire" (je ne vois pas de quoi il parle). De ton coté je ne vois pas de justification à l'ajout d'ExifRestorer dans le ticket. Donc à part pour l'orientation où le besoin semblait évident en 2019 je ne sais pas à quoi ça sert.

> Mais il n'y a pas que l'orientation dans les données exif. Ok mais est-ce qu'on peut préciser ce qu'on souhaite conserver ? Dans l'autre ticket je rapporte un commentaire de Nico dans le ticket initial qui parle juste d'orientation et d'"interpolation linéaire" (je ne vois pas de quoi il parle). De ton coté je ne vois pas de justification à l'ajout d'ExifRestorer dans le ticket. Donc à part pour l'orientation où le besoin semblait évident en 2019 je ne sais pas à quoi ça sert.
fpeters commented 2023-03-07 09:12:28 +01:00
Owner
Copy Link

Ok mais est-ce qu'on peut préciser ce qu'on souhaite conserver ?

A priori non les collectivités font ce qu'elles veulent avec les fichiers.

> Ok mais est-ce qu'on peut préciser ce qu'on souhaite conserver ? A priori non les collectivités font ce qu'elles veulent avec les fichiers.
fpeters closed this pull request 2023-03-24 17:19:44 +01:00
All checks were successful
gitea/wcs/pipeline/head This commit looks good
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#152
No description provided.
Delete Branch "wip/75178-Retirer-l-application-d-ExifRest"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?