Bug 156424 - Copy/paste of a row with images results in a copy with copied images positioned incorrectly
Summary: Copy/paste of a row with images results in a copy with copied images position...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.5.4.2 release
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:7.6.0
Keywords:
Depends on:
Blocks: Calc-Images Cell-Management
  Show dependency treegraph
 
Reported: 2023-07-22 12:39 UTC by Jim Connell
Modified: 2023-12-21 08:06 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
A Calc spreadsheet with images. (20.60 MB, application/vnd.oasis.opendocument.spreadsheet)
2023-07-22 12:40 UTC, Jim Connell
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Connell 2023-07-22 12:39:17 UTC
Description:
In a Calc spreadsheet with columns containing images, if a row is copied and pasted elsewhere, the image(s) in the row end up positioned incorrectly.

Steps to Reproduce:
1. Open attached spreadsheet.
2. Select and copy row 3.
3. Paste into row 6.

Actual Results:
The copy has the images in the wrong place horizontally.  It's not clear if they have been moved to a different cell or if they were just positioned incorrectly.

Expected Results:
The row copy should look like the original.


Reproducible: Always


User Profile Reset: No

Additional Info:
Version: 7.5.4.2 (X86_64) / LibreOffice Community
Build ID: 36ccfdc35048b057fd9854c757a8b67ec53977b6
CPU threads: 12; OS: Windows 10.0 Build 19045; UI render: default; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: threaded

I don't know if the images in a spreadsheet are associated with the cell explicitly or only by position.  So I don't know if any actual association is correct, or if there is even a way to tell.  The point is that it may not be enough to just fix the position.
Comment 1 Jim Connell 2023-07-22 12:40:31 UTC
Created attachment 188521 [details]
A Calc spreadsheet with images.
Comment 2 Jim Connell 2023-07-22 13:58:23 UTC
Similar to bug 156402.
Comment 3 danomois 2023-07-26 23:42:02 UTC
Hi Jim,

Thank you for this.

So - first I was able replicate the behavior in the top version below.

However, in the bottom version below, which is a developer version I downloaded a few hours ago, the copy/paste issue works without a problem.

That's why I marked it resolved/fixed - even though for sure it is not in the stable version I have.

For reference, I'm pretty new to QA here, so if you feel (or know) this is wrong procedure - please just drop a comment and I can update this.

As for your question about images generally - the libreoffice term is 'anchoring' - and you can right click a cell then go to 'anchoring' for different options.

Here's the Image Anchor help page - I know it's been a topic getting some work because I qa'd an issue around image resizing recently as well.
https://help.libreoffice.org/latest/en-US/text/scalc/guide/change_image_anchor.html?DbPAR=CALC#bm_id471607970579914

Thank you!

Version: 7.5.5.2 (X86_64) / LibreOffice Community
Build ID: 50(Build:2)
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Ubuntu package version: 4:7.5.5~rc2-0ubuntu0.22.04.1~lo1
Calc: threaded

Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 77fca616e0bd79e0b405fd0b3543cf8e94e15df3
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: threaded
Comment 4 ady 2023-07-27 00:09:42 UTC
I am resetting this from FIXED to NEW, for more than one reason.

Setting this to FIXED is inadequate, as there is no commit/patch that is known to solve it. At most, it would be WFM.

Additionally, if indeed this is a problem in a current version (such as 7.5 ATM), and it happens to work correctly in a development version that will only be available as stable in about 6 or 7 months, it could mean several (unnecessary) reports might get filed in the following months. Leaving this report opened as NEW would help users in their searches (not so much when it is set as resolved).

Moreover, there is a chance that, leaving this open for now, someone might find what exactly makes this work correctly in LO 24.2 (which I have not tested myself), thus allowing the possibility to cherry-pick or back-port such patch to LO 7.5 (or 7.6, which is not yet released as stable ATM).

This report might eventually be closed as resolved, either by committing a patch to 7.5, 7.6, or when those versions are no longer supported.
Comment 5 Buovjaga 2023-08-18 12:32:42 UTC
Bibisected the fix with linux-64-7.6 to dfb0d118f6b23730bc632885eb4703a37eeaec16
Comment 6 nikolausricky 2023-12-21 08:06:07 UTC Comment hidden (spam)