Bug 130601 - Allow overlap checkbox doesn't work for shapes
Summary: Allow overlap checkbox doesn't work for shapes
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.4 all versions
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected
: 145307 (view as bug list)
Depends on:
Blocks: Anchor-and-Text-Wrap
  Show dependency treegraph
 
Reported: 2020-02-11 21:52 UTC by Petr Valach
Modified: 2024-08-04 10:24 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Petr Valach 2020-02-11 21:52:53 UTC
Description:
Using checkbox Allow overlap on inserted objects in Writer doesn't work. After using nothing happens.

Steps to Reproduce:
1.Open Writer.
2.Insert some objects into document (for example squares).
3.Put one object acroos another.
4.Mark one object and from menu choose Wrap|Edit.
5.Uncheck Allow overlap.
6.Do the same with another object.

Actual Results:
Nothing happens.

Expected Results:
Behavior described in video: https://www.youtube.com/watch?v=DzanTIo941g&feature=youtu.be.


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
Version: 6.4.0.3
Build ID: b0a288ab3d2d4774cb44b62f04d5d28733ac6df8
CPU threads: 2; OS: Linux 5.3; UI render: default; VCL: gtk3; 
Locale: cs-CZ (cs_CZ.UTF-8); UI-Language: en-US
Calc: threaded

This issue was consulted with Miklós Vajna.
Comment 1 psidiumcode 2021-09-15 19:50:47 UTC
*** Bug 134114 has been marked as a duplicate of this bug. ***
Comment 2 Kira Tubo 2023-09-21 01:49:13 UTC
Seems to start occurring around 6.4.0.0.alpha1, possibly when "Allow overlap" gets introduced to the UI in the Shape's Wrap > Options dialog window. Only occurs when Wrap > Settings > "Though" (default) is used. Other setting options (Parallel, Optimal, etc.) seem fine. 

Version: 6.4.0.0.alpha1 (x64)
Build ID: cc57df8f942f239d29cb575ea5a7cb01405db787
CPU threads: 6; OS: Windows 10.0 Build 22621; UI render: GL; VCL: win; 
Locale: en-US (en_US); UI-Language: en-US
Calc: CL
Comment 3 Kira Tubo 2023-09-21 01:54:42 UTC
Also still reproducible on daily master build

Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 486ae5db6987411d5e394de94b2b077099d03856
CPU threads: 6; OS: Windows 10.0 Build 22621; UI render: Skia/Raster; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: CL threaded
Comment 4 raal 2024-05-29 17:29:39 UTC
This seems to have begun at the below commit in bibisect repository/OS bibisect-linux-64-6.4.
Adding Cc: to Miklos Vajna ; Could you possibly take a look at this one?
Thanks
 e147c0e5536c40045215355620c3a981b772d2a2 is the first bad commit
commit e147c0e5536c40045215355620c3a981b772d2a2
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Thu Sep 26 02:59:19 2019 +0200

    source 472535a4f8aa95a17d996e96a67df60e72df7b32
79501: Related: tdf#124600 sw anchored object allow overlap: add UI | https://gerrit.libreoffice.org/c/core/+/79501

Before this commit is not possible uncheck Allow overlap - option is not available.
Comment 5 Miklos Vajna 2024-05-30 07:37:30 UTC
Thanks for the bisect!

It seems to me that the "avoid overlap" layout code should be extended here. It clearly works for the bug 124600 case, but not for the case of this bug. That was added in d37096f59e7e0286e55008153591a60bab92b9e8 (Related: tdf#124600 sw anchored object allow overlap: add layout, 2019-09-19).

I agree that this is worth fixing, but it's not per see a regression, rather one more (new) case to handle in the "avoid overlap" layout implementation.

I hope this makes sense. :-)
Comment 6 Buovjaga 2024-08-04 10:24:09 UTC
*** Bug 145307 has been marked as a duplicate of this bug. ***