Commit graph

497012 commits

Author SHA1 Message Date
Bogdan Buzea
6b04c3ff86 Resave with newer Glade version
Change-Id: Ia4b40678632d825143c7e9ac2e8719b7e109a64a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169410
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 13:44:09 +02:00
Bogdan Buzea
9a07403258 Resave with newer Glade version
Change-Id: Ieaa8a90ddef6a49a3dfbb59575bf8defebeaa854
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169382
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 13:43:03 +02:00
Bogdan Buzea
aa1c24888d Resave with newer Glade version
Change-Id: Id2bc88f90d5e73ffa2c99d0fc4c3aa1fb55f568d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169386
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 13:42:29 +02:00
Bogdan Buzea
21249bc76a Resave with newer Glade version
Change-Id: I658a8a32cda30ecacd92dda284c924aa0137e14d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169418
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 13:41:23 +02:00
Bogdan Buzea
52c3ff2e32 Resave with newer Glade version
Change-Id: Ia266b96c1cc558ca4eb17194a58a1623c3f3ed54
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169392
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 13:40:30 +02:00
Michael Stahl
9413f2b39c ucb: cmis: remove CertValidationHandler
Bothering users with a dialog for a problematic TLS certificate just
conditions them to click OK and compromise their security.

WebDAV UCP already doesn't show such a dialog since LO 7.3, now remove
the dialog from CMIS UCP too.

Users can add any self-signed CAs they want to use to the operating
system trusted CA store, for example with p11-kit's trust(1).

Change-Id: Iedb8518923f9ac75c33a3b0df6ff795a7810a18a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169338
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2024-06-24 12:38:52 +02:00
Michael Stahl
62529d1eee libcmis,ucb: cmis: improve AllowInsecureProtocols implementation
1. in libcmis, pass the CurlInitProtocolsFunction to all subclasses of
   HttpSession that need it, and add 2 upstream fixes to pass it around

2. Arrange for InitCurl_easy to be called in UCP RepoContent as well

3. If AllowInsecureProtocols is disabled, automatically upgrade
   http connections to https, as is already done in webdav-curl.
   Do this in Content and RepoContent; hopefully should work
   to convert when m_aURL member is initialised;
   the m_xIdentifier on the other hand should have the original
   URL because ContentProviderImplHelper::queryExistingContents()
   caching likely relies on that.

Change-Id: I20d36ed03ba7ce221d6946b1c996071f4130ec7e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169114
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2024-06-24 12:37:02 +02:00
Michael Stahl
e598cfd1d8 ucb: cmis: remove NSS initialization
Upstream curl has removed NSS backend.

Change-Id: Iaf69ceb6b71db4c9553e8d4f7b30a702d0dd0f44
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169336
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
2024-06-24 12:36:33 +02:00
Olivier Hallot
fefc609559 Update git submodules
* Update helpcontent2 from branch 'master'
  to 47eaabaeb6b90551dccb72ad5ba9f241ad7608cf
  - Fix %PRODUCTNAME
    
    Change-Id: I30533473a81ecf7bb9ca3befbe5bac75d010b16d
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/169375
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    Tested-by: Jenkins
2024-06-24 10:53:06 +02:00
Pierre F
b0ad1a7b51 Update git submodules
* Update helpcontent2 from branch 'master'
  to ade7397aed9febc73918d93ffdf6477d3f4d4175
  - explicit \s and \d +  put the note first for full ICU spec. tdf#113975
    
    Change-Id: Ie06115c051def1da7b94357a392796d15a21df34
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/169402
    Tested-by: Jenkins
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2024-06-24 10:52:51 +02:00
Justin Luth
a9bdf3adf3 NFC cleanup duplicate entry in sc/IwyuFilter_sc.yaml #2
found by sed -i  -f ref.sed */*.yaml
--------------------ref.sed --------------------
{
N;
s#\([^\n]*\n\)\1#\1#g;
P;
D;
}
------------------------------------------------

meaning: by default it reads in one line from the file
then the N; command says "add a \n and read in the next line
    which leaves a one-line pattern containing 2 lines separated by \n
then search for line1\n repeated \1
and replace with \1.

likely nothing has happened, so P; print up to the first \n (aka line1)
    so that we don't lose that line
D; delete up to the first \n
    so that it isn't in the pattern buffer any more,
    and next search starts line2
and then repeat.

Change-Id: Ib5d38af77568f59f36083f5348762d852acaa3e8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169047
Reviewed-by: Justin Luth <jluth@mail.com>
Tested-by: Jenkins
2024-06-24 10:12:32 +02:00
Justin Luth
160ef8e6c3 NFC cleanup duplicate entry in framework/IwyuFilter_framework.yaml
found by sed -i  -f ref.sed */*.yaml
--------------------ref.sed --------------------
{
N;
N;
N;
s#\([^\n]*\n\)\([^\n]*\)\n\1\2#\1\2#g;
P;
D;
}
------------------------------------------------

meaning: by default it reads in one line from the file
then the 3 N; commands say "add a \n and read in the next line
    which leaves a one-line pattern containing 4 lines separated by \n
then search for line1\nline2\n repeated \1\2
and replace with \1\2.

likely nothing has happened, so P; print up to the first \n (aka line1)
    so that we don't lose that line
D; delete up to the first \n
    so that it isn't in the pattern buffer any more,
    and next search starts line2
and then repeat.
    Note that the pattern gets rather huge
    because while we only remove one line at a time,
    we unnecessarily append 3 more lines (instead of just one).
    But that doesn't seem to cause the search to fail...

Change-Id: I440bca873483dc7f1716881113807fe121fe6a4c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169045
Tested-by: Jenkins
Reviewed-by: Gabor Kelemen <gabor.kelemen.extern@allotropia.de>
Reviewed-by: Justin Luth <jluth@mail.com>
2024-06-24 10:12:21 +02:00
Noel Grandin
168eb2e64b tdf#144208 speedup doc with lots of redline(15)
SwXText is heavily used. Reduce the allocation cost of
this class by de-pimpl'ing this class.

Change-Id: I01b06c978075ae312131dfff29b4f304caf69e18
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169359
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
2024-06-24 09:20:05 +02:00
Noel Grandin
ba28df4fb2 tdf#144208 speedup doc with lots of redline(14)
SwXTextRange is heavily used. Reduce the allocation cost of
this class by de-pimpl'ing this class.

One strange thing I noticed is that the destructor of the Impl indicates
that deleting the bookmark via deleteMark() has to happen under the SolarMutex,
but there are two places in the existing code which violate this already.

Change-Id: I171a0d82d6d020d9a43808c424f308722770783f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169354
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2024-06-24 09:19:52 +02:00
Samuel Mehrbrodt
2d60017a66 ods: Don't export changes author/date when in privacy mode
Change-Id: I8c542f398bb70da96a64b61a9931398b6a78ea72
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169258
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
2024-06-24 07:51:52 +02:00
Bogdan Buzea
dae21034a4 Resave with newer Glade version
Change-Id: I78c7d3d4ae4643fa9f669177263d8c7bf4f681e5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169377
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 05:23:38 +02:00
Bogdan Buzea
7cfb1165ff Resave with newer Glade version
Change-Id: I86065d81a449360f09268fe617f8f09e7a917411
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169385
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 05:18:29 +02:00
Bogdan Buzea
ce117075a6 Resave with newer Glade version
Change-Id: I21efa82c67aa8e7a7af7a9fcb80f61ee64adaba9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169383
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 05:17:18 +02:00
Bogdan Buzea
6806659182 Resave with newer Glade version
Change-Id: I6abcf8d983550e92a2ccc768de47c2ccb6d8583b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169381
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 05:16:06 +02:00
Bogdan Buzea
87cf6f4dda Resave with newer Glade version
Change-Id: I56f5bbeed4850a742403ffd7dd43fca25101ad17
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169380
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 05:15:32 +02:00
Bogdan Buzea
e2e107aebb Resave with newer Glade version
Change-Id: I49158bd13ace3cd0b5e577e7964ae3b64822ac25
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169379
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 05:13:08 +02:00
Bogdan Buzea
2b7a3b870d Resave with newer Glade version
Change-Id: I83aa9920a5bdb339aeaed1b659b2f6786fa1390f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169420
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 05:11:52 +02:00
Bogdan Buzea
8b3e6cc614 Resave with newer Glade version
Change-Id: Ia740b979dcb8e21f7031020b8e2c708cfa43c0f2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169407
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 05:11:04 +02:00
Bogdan Buzea
2722385c10 Resave with newer Glade version
Change-Id: I39c00ad75b1dd2c769ed89a3b1aa2e028ab64efb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169419
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 05:09:59 +02:00
Bogdan Buzea
ec92c4883b Resave with newer Glade version
Change-Id: I214857e837fb0293228df934e8ccdbc89395ceb2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169412
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 05:09:27 +02:00
Tomaž Vajngerl
c32128b35d editeng: add convenience creators to ESelection All, NotFound
ESelection::All() select all text
ESelection::NotFound() sets seletion to not found state
ESelection::NoSelection() sets selection to no / invalid selection

Introduce max paragraph and position constants in ESelection,
use for EE_PARA_APPEND, EE_PARA_ALL, EE_PARA_MAX_COUNT,
EE_TEXTPOS_ALL, EE_TEXTPOS_MAX_COUNT.

Also simplify some ESelection constructs.

Change-Id: Ib110c5a730a1deabe4f988baa5a600249b3a31e9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169356
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
Tested-by: Jenkins
2024-06-24 02:14:51 +02:00
Bogdan Buzea
2f9c7d0e1e Resave with newer Glade version
Change-Id: Ief1e76e4888d33d353b73d26a2aca914b5b087dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169405
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 02:14:05 +02:00
Bogdan Buzea
9bfa4637dd Resave with newer Glade version
Change-Id: Id6148c524201850b19c345c70dee6c205b255e2d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169404
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 02:13:21 +02:00
Bogdan Buzea
0bc317e7a0 Resave with newer Glade version
Change-Id: I84dcacf811ce007144ecb8d1473d4b6887887422
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169397
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 02:12:14 +02:00
Bogdan Buzea
e824393f51 Resave with newer Glade version
Change-Id: I397231ded31f3aea7227b0251cf45abaa165faed
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169413
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 02:09:03 +02:00
Bogdan Buzea
5412dba69b Resave with newer Glade version
Change-Id: I970e33e166efe611dbaae16ab0ed6491298a6751
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169409
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 02:08:13 +02:00
Bogdan Buzea
d55b8b4728 Resave with newer Glade version
Change-Id: I011489b4d87a078af6083001d076f71140b98a0a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169401
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-24 02:04:11 +02:00
Bogdan Buzea
321f1533cd Resave with newer Glade version
Change-Id: I814a09020221fd19f6739475cbf8b06f4ab52e18
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169408
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-24 02:01:53 +02:00
Hossein
1305edfbc2 tdf#149256 Ship LibreOfficeKit headers in packages
With LibreOfficeKit (LOK) headers in place, it would be possible to
create C++ applications that can access LibreOffice functionality
without building LibreOffice, only by installing SDK and build tools.

Change-Id: I8efc8ac1cec7b3b89e4d7eb2c8d15afeb595672c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134863
Tested-by: Hossein <hossein@libreoffice.org>
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
2024-06-23 19:57:30 +02:00
Olivier Hallot
ecb0d08e3c Update git submodules
* Update helpcontent2 from branch 'master'
  to 9acd72cd9faa94ec85826293f1e95da3e6175d35
  - tdf#159596 tdf#159771 Update XMATCH and XLOOKUP pages
    
    + Added 3 as matchtype
    + removed reference for "search-criteria-must-apply-to-whole-cell"
    + set that matchtype 2 and 3 cannot be combined with binary search.
    
    Change-Id: I1fd2171aa92fb0a2df1c9ee85a1fb5f7f755efdb
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/169334
    Reviewed-by: Balazs Varga <balazs.varga.extern@allotropia.de>
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
    Tested-by: Jenkins
2024-06-23 09:20:17 +02:00
Bogdan Buzea
4420d77a50 Resave with newer Glade version
Change-Id: Ia9330a82d3dd6796cb613e6244c3149de872a852
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169372
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-23 08:17:42 +02:00
Bogdan Buzea
547167a27c Resave with newer Glade version
Change-Id: Ib3b198c2030b86cdd2cc9558407feefdde0b40dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169374
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-23 08:17:06 +02:00
Bogdan Buzea
4bba6c0ef4 Resave with newer Glade version
Change-Id: I1edfac23ae17ffcdf8959265bce51322a34c4f24
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169373
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-23 08:16:36 +02:00
Bogdan Buzea
cb2dffae80 Resave with newer Glade version
Change-Id: I6de6bacbdccd83b08e5e0b41adef23efd51d80dc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169371
Tested-by: Jenkins
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
2024-06-23 08:15:51 +02:00
Bogdan Buzea
6daffa1d9c Resave with newer Glade version
Change-Id: I3b06e8341b71d816df2cfa1f4343d48d4156cfbc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169370
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-23 08:04:04 +02:00
Bogdan Buzea
25287c44b6 Resave with newer Glade version
Change-Id: Ibebe4caede19886e47a9ebbbcaf53c79df8511cf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169369
Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
Tested-by: Jenkins
2024-06-23 08:01:05 +02:00
Tibor Nagy
513c2bdbb1 tdf#157529 PPTX import: fix transparency value
if the paragraph, which uses transformation (Arch, Circle, etc.),
contains multiple texts and the first text is only a space character,
then we use the transparency value of the following text.

Change-Id: Ia87242e34b495802cbecf5dc0dd13d490312f435
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169351
Reviewed-by: Nagy Tibor <tibor.nagy.extern@allotropia.de>
Tested-by: Jenkins
2024-06-22 21:08:52 +02:00
Rafael Lima
c3e80cbaaa tdf#161658 Limit new selection overlay to Calc only
Previous patches [1] and [2] introduced a contrast white line to the selection overlay that was supposed to affect only Calc, but it ended up affecting other areas of LO, such as the star math editor as well as Writer.

This patch makes sure the changes only affect cell selection in Calc.

[1] dc243f0122
[2] 3c0db89809

Change-Id: Ie910120e4b71c55ad8c00a905e1204e291a711f9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169292
Reviewed-by: Rafael Lima <rafael.palma.lima@gmail.com>
Tested-by: Jenkins
2024-06-22 14:38:54 +02:00
Noel Grandin
40e0885130 fix O(n^2) behaviour when fetching field info from EditEngine
Change-Id: I324a1814fc1b3321eed5b29922790600e7092c17
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169344
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
2024-06-22 14:00:49 +02:00
Noel Grandin
858bc4d76b no need to call FormatDoc() after calling CheckIdleFormatter()
CheckIdleFormatter() already does that

Change-Id: Ie6b9e3285303899e3f67cccb0fc5f5625c8db684
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169343
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Jenkins
2024-06-22 14:00:41 +02:00
Justin Luth
b7fce2b26e NFC sw textfld.cxx: make rDoc an actual reference
Change-Id: I84d65e80f27cdc2cbfc4f56c89e5a391e0fd9e1b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169350
Reviewed-by: Justin Luth <jluth@mail.com>
Tested-by: Jenkins
2024-06-22 03:26:18 +02:00
RMZeroFour
4031659233 .NET Bindings: Switch for old windows CLI bindings
This commit adds an --enable-cli/--disable-cli switch to autoconf to
control generation of the old CLI bindings (Windows only). It is
enabled by default, to not be a breaking change to users just yet.
Over time, when the old bindings are deprecated in favor of the new
.NET bindings, it could be set to disabled by default.

Change-Id: Ib60b372459cb0c735275ed17d004d037279357eb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/168751
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
2024-06-22 00:11:16 +02:00
Michael Weghorn
89372e6245 tdf#149461 qt5: Provide a QWindow for OpenGL rendering on X11
When using the qt5 or kf5 VCL plugins on X11/XWayland
(as can be forced using QT_QPA_PLUGIN=xcb on Wayland),
ensure that a QWindow exists for the QtObjectWidget
created in the QtObject ctor, as that is needed for
OpenGL rendering.

This makes OpenGL Impress slide transitions like the
"Tiles" one work.

Don't force native windows on Wayland, as that causes
other issues, see also QtFrame::ResolveWindowHandle
and the bugs referenced in there.

Change-Id: I8fac34d2f1cc7cc723ad4570546276a05641dd88
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169348
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
2024-06-21 23:19:09 +02:00
Michael Weghorn
b296b33cfb tdf#149461 qt6: Provide a QWindow for OpenGL rendering
As described at [1], "QWindow supports rendering using
OpenGL [...]".

Using a QWindow for OpenGL rendering had been introduced
in

    commit 56b19f9a81
    Date:   Fri Dec 14 12:44:20 2018 +0300

        tdf#121247, tdf#121266 KDE5: Add basic support for OpenGL

, but then

    commit 4366e06052
    Date:   Tue May 24 11:34:59 2022 +0200

        tdf#148864 Qt switch QtObjectWindow to QWidget

had switched from QWindow to QWidget, and OpenGL slide
transitions like the "Tiles" one stopped working.

At least for qt6, which now uses QtMultimedia for video
playback (see the tdf#145735 commits), issues like tdf#148864
("Kubuntu 22.04 LTS LibreOffice Impress 7.3.3 Fails to Play Embedded
 Videos") and tdf#125517 ("LO Impress: Can't stop presentation
with video and go to the next slide") related to video playback
appear to be no problem, so switch back to using QWindow there.

Explicitly set the window background to transparent
in `QtOpenGLContext::ImplInit`, as not doing so
caused slide content in presentation mode to not
be updated properly when testing with the attachment 183972
from tdf#149461 on Wayland.
(This was not an issue when running on XWayland, i.e.
with QT_QPA_PLATFORM=xcb).

With this in place, OpenGL transitions work with qt6.
With QT_QPA_PLATFORM=xcb, it looks all fine in my tests
on Debian testing.

With QT_QPA_PLATFORM=wayland, the slide text from attachment 183972
sometimes incorrectly showed up for a fraction of a second after
the slide transition finished, then disappeared again until
it was triggered to show as it should (e.g. by pressing the right
arrow key), s. screencast attachment 194899 in tdf#149461.
A multitude of warnings like the following are shown on stderr
in that case, which don't show up for xcb:

    warn:vcl.opengl:47352:47352:vcl/source/opengl/OpenGLHelper.cxx:709: GL Error 0506 (invalid framebuffer operation) in file /home/michi/development/git/libreoffice/slideshow/source/engine/opengl/TransitionImpl.cxx at line 398

That looks like a separate issue to me, however which would
need further analysis.

Keep using the QtObjectWindow approach for qt5 to
not regress on video playback.

[1] https://doc.qt.io/qt-5/qtgui-index.html#opengl-and-opengl-es-integration

Change-Id: I6e1eb989254e2cbbfada6f719ee0518571df4c42
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169347
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2024-06-21 23:19:02 +02:00
Michael Weghorn
22270a54a2 tdf#149461 qt: Move setting layout to QtObject ctor
Move setting the layout for the QtObjectWidget
from the QtObjectWidget ctor to the QtObject ctor,
in preparation to also use that for QWidgets other
than the QtObjectWidget.

Change-Id: Ifbb0d853659bc8099bed40db2bbe8a8778077aa2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/169346
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2024-06-21 23:18:52 +02:00