Writer has commands to rotate 90 degrees clock and counterclock wise, but no command to rotate it 180 degrees. This command would be useful for inclusion in the image context menu, so a user doesnt need to open the context menu twice to achieve this. Maybe also useful in the image toolbar.
Don't we have something as flip vertical / horizontal? Hmm, that still isn´t the same of course ;)
Yep we do have flip vertical and flip horizontal, but unless both halves of the image are exactly the same, they wont be the same as rotate 180.
Moving this to component Writer (remove ux-advice from the CC list). Also adding EASYHACK, plus NEEDINFO for the code pointers.
Missing code pointer and skill<foo>
Removing keyword 'needsDevEval' as this bug is an easyHack
Changing status: NEEDINFO -> NEW Adding keyword 'needsDevEval' [ninjaedit]
(In reply to Yousuf Philips (jay) from comment #2) > Yep we do have flip vertical and flip horizontal, but unless both halves of > the image are exactly the same, they wont be the same as rotate 180. Applying a flip vertical _and_ flip horizontal _together_, is mathematically the same as a rotation by 180°. From a technical point the methods are different: If you use 90°-rotation, then in case the image is linked, it is embedded automatically and the transformation actually changes the image. As the image itself is changed, the cropping handle tool will work as visually expected after a 180° rotation. But the user is not asked, whether he wants to embed the image. If you use flipping, then only the flip-information is stored and the image remains unchanged. That is possible with linked images too. But the cropping handle tool is faulty in that case and you have to use cropping from the Crop tab of the image properties to get the correct result. I personal have no strong opinion about which way to go.
(In reply to Regina Henschel from comment #7) > Applying a flip vertical _and_ flip horizontal _together_, is mathematically > the same as a rotation by 180°. Rotating by flipping doesnt modify the underlying file, so that would be preferable in my view.
Adding a new UNO command, is surely not an easy hack, first of all it has to be documented carefully, with test cases etc.
(In reply to jan iversen from comment #9) > Adding a new UNO command, is surely not an easy hack, first of all it has to > be documented carefully, with test cases etc. I don't understand. My patch is not correct? Wath sould i do? sorry but I ' m new in these things.
(In reply to filippo from comment #10) > (In reply to jan iversen from comment #9) > > Adding a new UNO command, is surely not an easy hack, first of all it has to > > be documented carefully, with test cases etc. > > I don't understand. My patch is not correct? > Wath sould i do? > sorry but I ' m new in these things. Sorry to confuse you, this was not about your patch, which I think is in review state. This was about this bug in general, which is a bit more than a normal easyHacks, so hats off that you submitted a patch. Feel free to contact me anytime.
giacco committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=5f116966de4c0d007c2a420b7adc74d72e728b89 tdf#87700 command to rotate 180 degrees It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
giacco committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ee7e34696e3981c65dd5a13688976aead450a709 tdf#87700 command to rotate 180 degrees It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to Commit Notification from comment #13) > Affected users are encouraged to test the fix and report feedback. Works like a charm, well done! Now we miss one the icon for this function. @Andreas, could you please do an icon for 180° rotation (90 left and right is in the context menu, for instance).
Patch merged to hide the button by default in the toolbar. https://gerrit.libreoffice.org/#/c/36741/