Page MenuHomePhabricator

Provide an interface to rotate an image
Open, MediumPublicFeature

Description

As Brion writes,

[Allowing uploaders to] preview & fix the rotation *during* the upload process
will be a big help. There should be a bugzilla entry ready for comments
somewhere.

Since I couldn't find one, this one is open.


Version: 1.18.x
Severity: enhancement

Details

Reference
bz32875
TitleReferenceAuthorSource BranchDest Branch
[jobs-api] support services in jobsrepos/cloud/toolforge/jobs-api!71raymond-ndibesupport_servicesmain
builds-builder: bump to 0.0.78-20231018094911-aa5433e6repos/cloud/toolforge/toolforge-deploy!118sstefanovabump_builds-buildermain
dev: Refactor shell scriptsrepos/cloud/toolforge/builds-builder!17sstefanovaslavina/refactor-scriptsmain
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 12:05 AM
bzimport set Reference to bz32875.
bzimport added a subscriber: Unknown Object (MLST).

I personally feel rotation shouldn't really happen during the image rendering process. Instead when uploading there should be a warning, saying image looks rotated, do you want to fix, with a preview of before and after the rotation, and forcing user to chose which one. If the user chooses the rotated version, we should physically rotate the image (And remove orientation), otherwise we should just leave the image as is.

(Also, it should be noted there is some discussion of this around bug 6672 comment 36 )

There appears to be a pretty fantastic user interface for this already implemented as a gadget for the RotateBot:
http://commons.wikimedia.org/wiki/Help:RotateLink

With a proper backend API, it doesn't seem like it'd be too difficult to plug this in in some way.

Filed bug 33186 (T35186) as the API counterpart to this bug

vivekee047 wrote:

Hey,

Is this still an issue.
Please check the bug 6672.Which is similar to this
Either both are same and this has to be marked as resolved
Or else can you tell me whats the difference betwwen these bugs

@Rob Lanphier:could you provide some information regarding API for rotating the image.Is it done or not.

Thanks

(In reply to comment #5)

@Rob Lanphier:could you provide some information regarding API for rotating the
image.Is it done or not.

Since bug 33186 is not closed, this is not done.

jgerber wrote:

*** Bug 45200 has been marked as a duplicate of this bug. ***

Again, please only allow 90 degree rotations. We do NOT, under ANY circumstances want people guessing at rotations at upload, then doing rough crops before any original copy has even been saved. 90 degree rotations are safe both at upload or later. Anything more than that, and we must save the original copy first

Remember that images degrade with every rotation. Further, we're rotating rectangles, not circles: if you rotate a rectangle anything but 90 degrees, you end up with either triangular gaps in the corners where the rotated image is put into a larger box, or you have to crop, throwing information away. **We do not want to encourage throwing away information before it's even saved, using a Web-based device that people will quickly eyeball the rotation and hit save on, because what else are they going to do.

Only 90 degree rotations are wanted by the community, don't scope creep this to the point it has to be rejected.

Earlier in the ticket concerned was raised about image degradation. This would certainly be a consideration for raster formats..

However some images on Commons are SVG. Is there a parameter that can be supplied to the SVG rendering back end allowing for the base coordinate system to be modified, or the SVG tweaked by wrapping an additional group and transformation matrix into the SVG passed to the back end used for rendering SVG? (This would be relatively straight forward to pass in from the File syntax, and would meet the use case mentioned in T185380)

Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 12:24 PM