Bug 147833 - Clone Formatting added to clicked row +1 row using KDE wayland
Summary: Clone Formatting added to clicked row +1 row using KDE wayland
Status: VERIFIED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.3.1.3 release
Hardware: All Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: Wayland KDE, KF5
  Show dependency treegraph
 
Reported: 2022-03-07 20:43 UTC by robby.engelmann
Modified: 2023-03-23 21:41 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Sample doc (8.13 KB, application/vnd.oasis.opendocument.spreadsheet)
2022-10-29 08:25 UTC, Michael Weghorn
Details
Screencast of the issue (6.39 MB, video/x-matroska)
2022-10-29 08:27 UTC, Michael Weghorn
Details

Note You need to log in before you can comment on or make changes to this bug.
Description robby.engelmann 2022-03-07 20:43:30 UTC
Clone Formatting tools does not apply the selected format to the clicked row but one row below.

Using KDE wayland.
Comment 1 Michael Weghorn 2022-10-29 08:25:20 UTC
Created attachment 183321 [details]
Sample doc
Comment 2 Michael Weghorn 2022-10-29 08:27:17 UTC
Created attachment 183322 [details]
Screencast of the issue
Comment 3 Michael Weghorn 2022-10-29 08:33:33 UTC
I can reproduce on Debian testing in a Plasma Wayland session, s. attached screencast attachment 183322 [details]:

1) open attached sample doc attachment 183321 [details]
2) select cell A1
3) click the "Clone Formatting" toolbar icon
4) click cell A2

Actual result:

Cell A3 gets yellow background.

Expected result:

Cell A2 should get yellow background.

It works as expected when explicitly choosing Qt's X11 platform plugin instead (environment variable QT_QPA_PLATFORM=xcb).

What's also noticeable is that the cursor does not change to one showing the "clone formatting" icon when using the Wayland backend.
This may be a LO issue, or one in the Qt library.

Version: 7.5.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 097a2de6428f834d28fa78af203d3c0b6577656e
CPU threads: 4; OS: Linux 6.0; UI render: default; VCL: kf5 (cairo+wayland)
Locale: en-GB (en_GB.UTF-8); UI: en-US
Calc: threaded

Version: 7.5.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 097a2de6428f834d28fa78af203d3c0b6577656e
CPU threads: 4; OS: Linux 6.0; UI render: default; VCL: qt6 (qfont+wayland)
Locale: en-GB (en_GB.UTF-8); UI: en-US
Calc: threaded
Comment 4 Michael Weghorn 2023-02-23 10:33:34 UTC
I can no longer reproduce on a current Debian testing with

Version: 7.6.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 6d404f9ccad3ac6a7ffb83876ae7a656bc2d4547
CPU threads: 12; OS: Linux 6.1; UI render: default; VCL: kf5 (cairo+wayland)
Locale: en-GB (en_GB.UTF-8); UI: en-US
Calc: threaded

And I cannot even reproduce with the same version I was using back then, now from the bibisect repository:

Version: 7.5.0.0.alpha0+ (X86_64) / LibreOffice Community
Build ID: 097a2de6428f834d28fa78af203d3c0b6577656e
CPU threads: 12; OS: Linux 6.1; UI render: default; VCL: kf5 (cairo+wayland)
Locale: en-GB (en_GB.UTF-8); UI: en-US
Calc: threaded

The latter shows no cursor any more, which was fixed in the context of tdf#153333, and that was probably also what this comment was talking about:

> What's also noticeable is that the cursor does not change to one showing the "clone > formatting" icon when using the Wayland backend.
> This may be a LO issue, or one in the Qt library.

It looks like cursor handling in KWin or Qt has changed, and this has fixed the issue described here.

-> Closing as WORKSFORME (could also use NOTOURBUG due to the above observations)

@Robby: Could you please retest with latest KDE and Qt versions as well and reopen this issue if you still see the problem then?

Version of packages on my current Debian testing where it works as expected:

kwin-wayland                           4:5.26.90-1
libqt5waylandclient5:amd64 5.15.8-2
Comment 5 robby.engelmann 2023-03-12 20:52:59 UTC
The offset during clone formating may be solved.
However, it is hard to evaluate, because the cursor is invisible during using the tool.

Version informations:

Version: 7.5.1.2 (X86_64) / LibreOffice Community
Build ID: 50(Build:2)
CPU threads: 8; OS: Linux 6.2; UI render: default; VCL: kf5 (cairo+wayland)
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: CL threaded

Operating System: openSUSE Tumbleweed 20230311
KDE Plasma Version: 5.27.2
KDE Frameworks Version: 5.105.0
Qt Version: 5.15.8
Kernel Version: 6.2.2-1-default (64-bit)
Graphics Platform: Wayland
Processors: 8 × Intel® Core™ i7-10510U CPU @ 1.80GHz
Memory: 31.0 GiB of RAM
Graphics Processor: Mesa Intel® UHD Graphics
Manufacturer: TUXEDO
Product Name: TUXEDO
System Version: Not Applicable
Comment 6 Michael Weghorn 2023-03-13 05:57:14 UTC
(In reply to robby.engelmann from comment #5)
> The offset during clone formating may be solved.
> However, it is hard to evaluate, because the cursor is invisible during
> using the tool.

This is a Qt bug, but there's a solution in LO that avoids the issue, s. tdf#153333. Can you please retest with a version that includes the corresponding change?

There are e.g. daily development builds available at [1]. [2] describes how to install them in parallel to an existing LO version. (Alternatively, you can wait until 7.5.2 is released.)

[1] https://dev-builds.libreoffice.org/daily/master/current.html
[2] https://wiki.documentfoundation.org/Installing_in_parallel/Linux
Comment 7 robby.engelmann 2023-03-23 20:56:15 UTC
waited and everything is fine now with 7.5.2
Comment 8 Michael Weghorn 2023-03-23 21:41:32 UTC
(In reply to robby.engelmann from comment #7)
> waited and everything is fine now with 7.5.2

Great, thanks for confirming.