#7042 closed feature request (wontfix)
Use exif rotation data when uploading pictures
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 2.5.1 |
Component: | Upload | Keywords: | |
Focuses: | Cc: |
Description
When I upload a picture, the exif rotation data doesn't seem to be used. It should be.
Pretty sure flickr does this; I know gallery (gallery.menalto.com) does; all my local tools here respect it so I'd hate to have to modify the picture just to satisfy wordpress.
Otherwise, the gallery feature is shaping up nicely- well done.
(Doesn't help that there appears to be no way to do the rotation post-upload.)
Change History (24)
#4
@
16 years ago
- Component changed from General to Gallery
- Owner anonymous deleted
- Type changed from defect (bug) to feature request
#7
@
16 years ago
- Component changed from Gallery to Upload
- Severity changed from major to normal
- Summary changed from exif rotation data not used when uploading pictures? to Use exif rotation data when uploading pictures
We can rotate the uploaded image before saving it (when exif rotation is set) then create thumbnail and other sizes. However that will modify the original image, remove exif data, etc.
Perhaps the better option is to rotate only the sub-sized images while creating them and leave the original intact. It's also possible to rotate the original with JS when displaying it.
#9
follow-up:
↓ 17
@
16 years ago
The replacement imagerotate() function is very slow with large images, it copies the image pixel by pixel. Perhaps we should do image rotate and flip only when it's available.
#12
in reply to:
↑ 11
@
16 years ago
Removed this temporarily pending better patch [11899].
Replying to Viper007Bond:
Well, slow rotation is better than no rotation I think.
By slow I mean really slow... For 1.2MB image it takes 0.2 sec. when imagerotate() is available and about 8 sec. otherwise, not even considering the server load. Don't think that's acceptable.
#13
@
16 years ago
Honestly I disagree. 8 seconds is bad, but not terrible for sitting there looking at "crunching". I think the user expects that it could take a bit.
#14
@
16 years ago
I'd very much agree... 8 seconds is quite acceptable as long as I've got something on the screen telling me why. That 8 seconds spent is going to be way better than my wife having her image end up sideways in her blog post, come complaining to me about it, having me walk her (again) through rotating it by hand in an image editor and re-uploading it, and I'd bet she would agree, too.
#15
@
16 years ago
I think this is a really big deal for "regular human users". My mother is baffled and I'm preparing myself to explain exif data... :-)
#17
in reply to:
↑ 9
@
15 years ago
It actually looks like this ticket was fixed in [11746]. Looks like there was some post-commit discussion but that's it. So fixed?
#20
@
15 years ago
- Resolution set to wontfix
- Status changed from new to closed
Between this: (In [11746]) Use exif rotation data when creating sub-sizes of uploaded jpeg images, see #7042; and the addition of rotation post-upload in 2.9, I'm going to close this ticket as wontfix, since it's been open for 21 months but no one has added a patch. If someone wants to revisit, feel free to open when there's a patch to attach.
#21
@
15 years ago
I have no objection, the other patches that have already been committed satisfactorily address this I think.
Postponed to 2.8.