Description: UI: Label "type" & Image in the image properties dialog not really helpful (self-explaining) Steps to Reproduce: 1. open a file with image attachment 168083 [details] 2. Right click it -> Properties Actual Results: The tabs 'Type' and 'Image' not really helpful. Not sure how 'type' fit the content of the tab. And calling a tab 'Image' within an image dialog?. Which is partly about flipping/rotating.. Expected Results: No clear cut proposal. This is 'literally' an enhancement request (but without alternative proposition). There is already a bug for duplicate settings (bug 132853). The image tab matches the "Position and size" section in the sidebar. So maybe position and size? Or Size en orientation? And moving "Link file name" in Image to "options" Type tab could be something like "anchoring & position". However some similarity between sidebar & tab structure maybe desired. --- Prefer first some getting some feedback if my report is resonating at all :P Reproducible: Always User Profile Reset: No Additional Info: Version: 7.2.0.0.alpha0+ (x64) Build ID: 796c7f612603490dda9277ced0f6ab3cce3bc116 CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: default; VCL: win Locale: nl-NL (nl_NL); UI: en-US Calc: CL
This dialog is shared with other kind of frame objects, e.g. text frame, OLE-object, chart, Math object. Only the tabs "Image" and "Crop" are specific for Images. I agree, that "Type" does not describe the tab content.
The 'Properties' dialog (by context menu, or Format -> Images menu) for Image frames is only present in Writer. Likewise for other Object Frames. Images present on Impress or Draw canvas have no similar dialog. Nor do images (and other objects) inserted onto calc sheets. But since all modules do provide the block of image controls--'Crop', 'Edit with External Tool', 'Replace', 'Compress', 'Save'[1]--would an image/graphic specific 'Image Type' or 'Format' dialog make for more consistent UX cross module? =-ref-= [1] .uno:Crop, .uno:ExternalEdit, .uno:ChangePicture, .uno:CompressGraphic, .uno:SaveGrapic
(In reply to V Stuart Foote from comment #2) In principle pro harmonization and sharing (although writer having some dedicated settings like wrap/anchoring) But I think this not simply limited to GUI and also requiring harmonization of internals: From: https://bugs.documentfoundation.org/show_bug.cgi?id=107843#c3 -- "Impress 8" produces an OLE. There crop is not needed, because it is done by the frame. "Drawing Format" produces a shape. In Draw/Impress the command .uno:GrafAttrCrop contains the cropping. This command is not available in Writer. Therefore not the original image is inserted, but an image that only has the part, that remains after cropping. --> Feature request: Implement .uno:GrafAttrCrop for those shapes, that would be able to be cropped in Draw/Impress and then insert the shape in original size and crop it. "GDI" contains an image wrapped into a vector format. "Bitmap" contains an image directly. Both "GDI" and "Bitmap" contain only the part, that remains after cropping. That has always been this way, so I would say it is more a feature request than a bug. I think, that the current behavior can be improved. The images can be inserted in original size and the same cropping as in Draw/Impress can then be applied automatically. For such images cropping is available in Writer as part of .uni:GraphicDialog. I don't know, whether the clipboard already contains the needed information. Cropping is part of the style of family "Graphic" in file format. LibreOffice distinguishes between images as shapes and Writer-images, but nevertheless uses the same kind of style in file format, only naming one "gr" and the other "fr". Currently this distinction is necessary, because the capabilities of "Draw-Images" and "Writer-Images" are different. The aim should be, to get rid of "Writer-Images". This issue belongs to this aim, regardless of naming it "bug" or "feature request". ---
Isn't just a renaming sufficient? Quick proposal: Type => Position and Size (as we do for shapes; the tab is also used for other objects), Image => Flip and Rotation (or we keep Image here).
We discussed the proposal in the design meeting. Renaming "Type" to "Position and Size" was accepted but "Image" could be just "Rotation" (without Flip). Chart and formula objects also use "Type" for position and size information, while shapes or text boxes have "Position and Size" in the same tab. And shapes use "Rotation" (but have no flip option in this tab). Code pointer: sw/uiconfig/swriter/ui/picturedialog.ui <object class="GtkLabel" id="type"> <property name="label" translatable="yes" context="picturedialog|type">Type</property> <object class="GtkLabel" id="picture"> <property name="label" translatable="yes" context="picturedialog|picture">Image</property>
Why is/was the tab called "Type"? Maybe add the word "Properties" to the dialog title, so that the window would be named "Image Properties"?
(In reply to sdc.blanco from comment #6) > Maybe add the word "Properties" to the dialog title, so that the window > would be named "Image Properties"? No further input. Here is patch -- without changing dialog title. Have also made this change for the Frame dialog -- and will do the same for the Object dialog (because they have the same tab). https://gerrit.libreoffice.org/c/core/+/134189
Seth Chaiklin committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/0e5f4f3ac15e22cfcf4447704105aaa0d1e241ee tdf#138844 rename "Type" and "Image" tabs in Image Properties dialog It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Seth Chaiklin committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/help/commit/9f035d777141d5f039a85d6f64b627eaf73b8079 (related)tdf#138844, (related)tdf#149018 "Type" -> "Position and Size"
Seth Chaiklin committed a patch related to this issue. It has been pushed to "libreoffice-7-4": https://git.libreoffice.org/help/commit/2823fbf593dc4f9d87f3d14731bf997cc7de7deb (related)tdf#138844, (related)tdf#149018 "Type" -> "Position and Size"