[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Heiko Tietze  changed:

   What|Removed |Added

 CC|libreoffice-ux-advise@lists |
   |.freedesktop.org|

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

BogdanB  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=15
   ||0675
 CC||buzea.bog...@libreoffice.or
   ||g

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Yousuf Philips (jay)  changed:

   What|Removed |Added

 Blocks||114610


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=114610
[Bug 114610] [META] Object rotation bugs and enhancements
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

--- Comment #15 from Yousuf Philips (jay)  ---
Patch merged to hide the button by default in the toolbar.

https://gerrit.libreoffice.org/#/c/36741/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Yousuf Philips (jay)  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Heiko Tietze  changed:

   What|Removed |Added

 CC||kain...@gmail.com

--- Comment #14 from Heiko Tietze  ---
(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).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

--- Comment #13 from Commit Notification 
 ---
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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

filippo  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|filippo.giac...@gmail.com   |libreoffice-b...@lists.free
   ||desktop.org

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:5.4.0

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

--- Comment #12 from Commit Notification 
 ---
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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

jan iversen  changed:

   What|Removed |Added

   Keywords|easyHack, needsDevEval  |

--- Comment #9 from jan iversen  ---
Adding a new UNO command, is surely not an easy hack, first of all it has to be
documented carefully, with test cases etc.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

--- Comment #10 from filippo  ---
(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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

--- Comment #11 from jan iversen  ---
(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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

jan iversen  changed:

   What|Removed |Added

 CC||libreoffice-ux-advise@lists
   ||.freedesktop.org

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Yousuf Philips (jay)  changed:

   What|Removed |Added

 CC|libreoffice-ux-advise@lists |
   |.freedesktop.org|
 Blocks||103152


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=103152
[Bug 103152] [META] Writer image bugs and enhancements
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

--- Comment #8 from Yousuf Philips (jay)  ---
(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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Regina Henschel  changed:

   What|Removed |Added

 CC||rb.hensc...@t-online.de

--- Comment #7 from Regina Henschel  ---
(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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

filippo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|libreoffice-b...@lists.free |filippo.giac...@gmail.com
   |desktop.org |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

filippo  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
   Assignee|filippo.giac...@gmail.com   |libreoffice-b...@lists.free
   ||desktop.org

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

filippo  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|libreoffice-b...@lists.free |filippo.giac...@gmail.com
   |desktop.org |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=87700

Heiko Tietze  changed:

   What|Removed |Added

   Keywords||easyHack, needsDevEval,
   ||topicUI
 Status|NEW |NEEDINFO
 CC|libreoffice-ux-advise@lists |tietze.he...@gmail.com
   |.freedesktop.org|
  Component|ux-advise   |Writer

--- Comment #3 from Heiko Tietze  ---
Moving this to component Writer (remove ux-advice from the CC list). Also
adding EASYHACK, plus NEEDINFO for the code pointers.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=87700

Cor Nouws c...@nouenoff.nl changed:

   What|Removed |Added

 CC||c...@nouenoff.nl

--- Comment #1 from Cor Nouws c...@nouenoff.nl ---
Don't we have something as flip vertical / horizontal?
Hmm, that still isn´t the same of course ;)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 87700] UNO: New command to rotate image 180 degrees

[bugzilla-daemon]

https://bugs.freedesktop.org/show_bug.cgi?id=87700

--- Comment #2 from Jay Philips philip...@hotmail.com ---
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.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise