We have a simple user interactive crop tool in .uno:Crop, but the context menu is executing .uno:GrafAttrCrop.
Jay - and what is the benefit of one over the other? I'm setting this to NEEDINFO as there is literally no explanation as to why we should do this or what it would add.
Sorry for the limited explanation. :D Presently when you right-click on an image in impress, the context menu gives the option 'Crop Image...' which opens up a crop dialog which allows you to manually change values to crop an image (the same as going to Format > Crop Image). This would be fine if this was the only means of available to crop an image, but the toolbar has a 'Crop Image' button which allows you to easily crop an image by dragging the corners of the image with the mouse. So rather than linking to the crop dialog (.uno:GrafAttrCrop) in the context menu, the context menu should link to the interactive crop tool (.uno:Crop).
Crop dialog (https://help.libreoffice.org/Common/Crop_1) is still accessible via menu Format. Setting as NEW, cropping with interactive crop tool is more user friendly.
I'm taking this
Committed patch: https://gerrit.libreoffice.org/#/c/13472/ Should be available shortly in 4.5.
Joel committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=fdec5e9f9a42214918290d4a329c97e98826627a fdo#86627 - Change Image Crop Behavior It will be available in 4.5.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.
Joel committed a patch related to this issue. It has been pushed to "libreoffice-4-4": http://cgit.freedesktop.org/libreoffice/core/commit/?id=0b0860414cefded03fdbc0a4490aaac39236ff09&h=libreoffice-4-4 fdo#86627 - Change Image Crop Behavior (4.4) It will be available in 4.4.0.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.
The patch was actually tested?
I think Jay tested it - but it works fine on my system. Why...is it not working for you?
(In reply to Joel Madero from comment #9) > I think Jay tested it - but it works fine on my system. Why...is it not > working for you? It does — sorry for the noise, I had a brainfart and thought I was testing the latest build.