Bug 67584

Summary: Other: Writer: Wrong Flip menu items and icons
Product: LibreOffice Reporter: Stanislav Horacek <stanislav.horacek>
Component: UIAssignee: Joel Madero <jmadero.dev>
Status: VERIFIED FIXED QA Contact:
Severity: trivial    
Priority: lowest CC: jmadero.dev, timofeev.i.s
Version: 4.1.0.4 releaseKeywords: regression
Hardware: Other   
OS: Linux (All)   
Whiteboard: BSA target:4.2.0 target:4.1.2
i915 platform: i915 features:
Attachments: screenshot of the menu

Description Stanislav Horacek 2013-07-31 13:37:42 UTC
Created attachment 83364 [details]
screenshot of the menu

Problem description: 

Steps to reproduce:
1. In Writer, choose Format - Flip.

Current behavior:
You see "Flip Horizontally" and "Flip Horizontally" items and icons with horizontal and vertical arrowed line.

Expected behavior:
You should see "Flip Vertically" and "Flip Horizontally" items and icons with vertical and horizontal arrowed line.

              
Operating System: Ubuntu
Version: 4.1.0.4 release
Last worked in: 4.0.2.2 release
Comment 1 Joel Madero 2013-08-01 19:03:21 UTC
Ah - there is the other one - confirmed and taking it
Comment 2 Commit Notification 2013-08-14 05:55:54 UTC
Joel Madero committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=d9ead65397ea4b6d4bcc66cf304653bb71e14c9c

fdo#67584 - Incorrect Flip Labels



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.
Comment 3 Stanislav Horacek 2013-08-15 13:46:39 UTC
Checked in master (Build ID: 1e379e3c9aa62c6cf46c32175d5e8c9d4f4bca09):

Labels are OK now, but - could it be "Flip Vertically" and "Flip Horizontally" instead of "Flip horizontal"?

The icons are still swapped.
Comment 4 Joel Madero 2013-08-27 13:30:17 UTC
I'm not sure if we actually pushed the fix here - I made a mistake with the icons and need to revisit it. Will try to investigate
Comment 5 Stanislav Horacek 2013-08-29 20:53:39 UTC
To the icons: it depends on icon theme. Themes galaxy and hicontrast are right, all others have vertical and horizontal flip icon swapped.
Comment 6 Ivan Timofeev (retired) 2013-09-01 12:28:00 UTC
So, this is a regression from http://cgit.freedesktop.org/libreoffice/core/commit/?id=c991fb49aa174f300be0b978d89bc63e01e0aa2b

uno:FlipHorizontal was removed from WriterCommands.xcu, I guess uno:FlipVertical just need to be removed too. I will try that...

I've just swapped the icons - there was an old bug 42434.
Comment 7 Commit Notification 2013-09-01 16:33:36 UTC
Ivan Timofeev committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=65c85266ce1f8b12a33dfaa8efff365854f17dd5

fdo#67584: remove .uno:FlipVertical from WriterCommands.xcu



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.
Comment 8 Joel Madero 2013-09-01 17:10:51 UTC
Is this bug 100% fixed now? I thought my patch took care of most of it other than the minor label issue and I had sym links for icons -- which I was going to take care of. But if you've solved all of it I'll mark as FIXED
Comment 9 Ivan Timofeev (retired) 2013-09-01 17:18:49 UTC
Yes, but I usually delay closing until a fix is backported to release branches. :) The label is fixed here and the icons are fixed in bug 42434.
Comment 10 Commit Notification 2013-09-01 19:21:18 UTC
Ivan Timofeev committed a patch related to this issue.
It has been pushed to "libreoffice-4-1":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=d04e98d4ef8d15c9e8c76d0a61fd0aa45e75b1e8&h=libreoffice-4-1

fdo#67584: remove .uno:FlipVertical from WriterCommands.xcu


It will be available in LibreOffice 4.1.2.

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.
Comment 11 Stanislav Horacek 2013-09-04 07:16:52 UTC
Checked in 4.1 (Build ID: aa9bef8271ed50a397c959ed53c91ee44b3dcb1) and master (Build ID: dda8877b989ebe330315e94523b983a6803e1372) - both labels and icons are OK; marking as verified. Thank you for solving this!

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.