vcl/qt5/QtInstanceBuilder.cxx  |   28 ++++++++++++++++++----------
 vcl/qt5/QtInstanceTreeView.cxx |    2 +-
 2 files changed, 19 insertions(+), 11 deletions(-)

New commits:
commit e7968c5751c21500bb29d56f6d1b23cc0e5a4ea9
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Sun Aug 3 22:58:33 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Mon Aug 4 09:01:38 2025 +0200

    tdf#130857 qt weld: Unbreak "Print" option page
    
    Prevent trying to use native Qt widgets for the
    "LibreOfficeDev Writer" -> "Print" page in the
    "Tools" -> "Options" dialog long as the dialog itself
    isn't supported yet, as that results in that page
    looking broken with when using the qt5 or qt6 VCL
    plugin and starting LO with
    environment variable SAL_VCL_QT_USE_WELDED_WIDGETS=1 set.
    
    Declare that .ui file only as supported when a native
    parent is used.
    
    Change-Id: Ifbd7e30f162fd3bed6426a145ef36827a1df401a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188902
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/vcl/qt5/QtInstanceBuilder.cxx b/vcl/qt5/QtInstanceBuilder.cxx
index cde2892a855b..ba4fa4f30a2d 100644
--- a/vcl/qt5/QtInstanceBuilder.cxx
+++ b/vcl/qt5/QtInstanceBuilder.cxx
@@ -159,7 +159,6 @@ bool QtInstanceBuilder::IsUIFileSupported(const OUString& 
rUIFile, const weld::W
         u"modules/swriter/ui/inserttable.ui"_ustr,
         u"modules/swriter/ui/linenumbering.ui"_ustr,
         u"modules/swriter/ui/numberingnamedialog.ui"_ustr,
-        u"modules/swriter/ui/printoptionspage.ui"_ustr,
         u"modules/swriter/ui/renameautotextdialog.ui"_ustr,
         u"modules/swriter/ui/renameobjectdialog.ui"_ustr,
         u"modules/swriter/ui/rowheight.ui"_ustr,
@@ -212,14 +211,23 @@ bool QtInstanceBuilder::IsUIFileSupported(const OUString& 
rUIFile, const weld::W
 
     // These UI files are only supported inside native Qt dialogs/widgets
     static std::unordered_set<OUString> aSupportedWithQtParent = {
-        u"cui/ui/appearance.ui"_ustr,         
u"cui/ui/graphictestentry.ui"_ustr,
-        u"cui/ui/lineendstabpage.ui"_ustr,    u"cui/ui/linetabpage.ui"_ustr,
-        u"cui/ui/linestyletabpage.ui"_ustr,   u"cui/ui/optlingupage.ui"_ustr,
-        u"cui/ui/possizetabpage.ui"_ustr,     
u"cui/ui/rotationtabpage.ui"_ustr,
-        u"cui/ui/slantcornertabpage.ui"_ustr, 
u"cui/ui/textattrtabpage.ui"_ustr,
-        u"cui/ui/textanimtabpage.ui"_ustr,    
u"cui/ui/textcolumnstabpage.ui"_ustr,
-        u"svt/ui/datewindow.ui"_ustr,         u"svx/ui/colorwindow.ui"_ustr,
-        u"vcl/ui/editmenu.ui"_ustr,           u"xmlsec/ui/certdetails.ui"_ustr,
+        u"cui/ui/appearance.ui"_ustr,
+        u"cui/ui/graphictestentry.ui"_ustr,
+        u"cui/ui/lineendstabpage.ui"_ustr,
+        u"cui/ui/linetabpage.ui"_ustr,
+        u"cui/ui/linestyletabpage.ui"_ustr,
+        u"cui/ui/optlingupage.ui"_ustr,
+        u"cui/ui/possizetabpage.ui"_ustr,
+        u"cui/ui/rotationtabpage.ui"_ustr,
+        u"cui/ui/slantcornertabpage.ui"_ustr,
+        u"cui/ui/textattrtabpage.ui"_ustr,
+        u"cui/ui/textanimtabpage.ui"_ustr,
+        u"cui/ui/textcolumnstabpage.ui"_ustr,
+        u"modules/swriter/ui/printoptionspage.ui"_ustr,
+        u"svt/ui/datewindow.ui"_ustr,
+        u"svx/ui/colorwindow.ui"_ustr,
+        u"vcl/ui/editmenu.ui"_ustr,
+        u"xmlsec/ui/certdetails.ui"_ustr,
         u"xmlsec/ui/certgeneral.ui"_ustr,
     };
 
commit 53d726da551445844e1a8eeb6a19a9add2b00cd5
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Sun Aug 3 22:51:38 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Mon Aug 4 09:01:32 2025 +0200

    tdf#130857 qt weld: Unbreak "Writing Aids" option page
    
    Prevent trying to use native Qt widgets for the
    "Languages and Locales" -> "Writing Aids" page as
    long as the dialog itself isn't supported yet, as that
    results in that page looking broken with when using the
    qt5 or qt6 VCL plugin and starting LO with
    environment variable SAL_VCL_QT_USE_WELDED_WIDGETS=1 set.
    
    Declare that .ui file only as supported when a native
    parent is used.
    
    That file was previously declared as supported in
    
        commit 66a674f915badf9e586b8e3854954d71dc908a4d
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Fri Feb 21 15:33:53 2025 +0100
    
            tdf#130857 qt weld: Support spelling options dialog
    
    because in that scenario it does have native parent
    and works fine there.
    
    Change-Id: I185827e0931245d87424dea71fecfd86ca627236
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188901
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/vcl/qt5/QtInstanceBuilder.cxx b/vcl/qt5/QtInstanceBuilder.cxx
index 4eb3ceea0e14..cde2892a855b 100644
--- a/vcl/qt5/QtInstanceBuilder.cxx
+++ b/vcl/qt5/QtInstanceBuilder.cxx
@@ -83,7 +83,6 @@ bool QtInstanceBuilder::IsUIFileSupported(const OUString& 
rUIFile, const weld::W
         u"cui/ui/numberdialog.ui"_ustr,
         u"cui/ui/objecttitledescdialog.ui"_ustr,
         u"cui/ui/objectnamedialog.ui"_ustr,
-        u"cui/ui/optlingupage.ui"_ustr,
         u"cui/ui/optnewdictionarydialog.ui"_ustr,
         u"cui/ui/password.ui"_ustr,
         u"cui/ui/pastespecial.ui"_ustr,
@@ -215,12 +214,13 @@ bool QtInstanceBuilder::IsUIFileSupported(const OUString& 
rUIFile, const weld::W
     static std::unordered_set<OUString> aSupportedWithQtParent = {
         u"cui/ui/appearance.ui"_ustr,         
u"cui/ui/graphictestentry.ui"_ustr,
         u"cui/ui/lineendstabpage.ui"_ustr,    u"cui/ui/linetabpage.ui"_ustr,
-        u"cui/ui/linestyletabpage.ui"_ustr,   u"cui/ui/possizetabpage.ui"_ustr,
-        u"cui/ui/rotationtabpage.ui"_ustr,    
u"cui/ui/slantcornertabpage.ui"_ustr,
-        u"cui/ui/textattrtabpage.ui"_ustr,    
u"cui/ui/textanimtabpage.ui"_ustr,
-        u"cui/ui/textcolumnstabpage.ui"_ustr, u"svt/ui/datewindow.ui"_ustr,
-        u"svx/ui/colorwindow.ui"_ustr,        u"vcl/ui/editmenu.ui"_ustr,
-        u"xmlsec/ui/certdetails.ui"_ustr,     u"xmlsec/ui/certgeneral.ui"_ustr,
+        u"cui/ui/linestyletabpage.ui"_ustr,   u"cui/ui/optlingupage.ui"_ustr,
+        u"cui/ui/possizetabpage.ui"_ustr,     
u"cui/ui/rotationtabpage.ui"_ustr,
+        u"cui/ui/slantcornertabpage.ui"_ustr, 
u"cui/ui/textattrtabpage.ui"_ustr,
+        u"cui/ui/textanimtabpage.ui"_ustr,    
u"cui/ui/textcolumnstabpage.ui"_ustr,
+        u"svt/ui/datewindow.ui"_ustr,         u"svx/ui/colorwindow.ui"_ustr,
+        u"vcl/ui/editmenu.ui"_ustr,           u"xmlsec/ui/certdetails.ui"_ustr,
+        u"xmlsec/ui/certgeneral.ui"_ustr,
     };
 
     if (aSupportedUIFiles.contains(rUIFile))
commit 44e08e5b7fef8d9aa65e81e6a9a1367b177f473c
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Sun Aug 3 22:36:37 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Mon Aug 4 09:01:26 2025 +0200

    tdf#130857 qt weld: Use text col in TreeView::insert
    
    Don't unconditionally insert the text into the first
    column in QtInstanceTreeView::insert, but into the
    first column that supports text according to the
    underlying GtkTreeViewColumn/GtkCellRenderers in the .ui
    file. (Use QtInstanceTreeView::set_text which implements
    the corresponding logic.)
    
    Together with previous commit
    
        Change-Id: Ic8a59b254f3bac14c3b812f00d0cbfa3740432f2
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Sun Aug 3 22:17:34 2025 +0200
    
            tdf#130857 qt weld: Evaluate GtkCellRenderers to know supported 
types
    
    this fixes the problem mentioned in previous commit
    
        Change-Id: Id94e22e879f712bd967431ec0f9dc34e4480e3dd
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Sun Aug 3 22:15:20 2025 +0200
    
            tdf#130857 qt weld: Support Java "Class Path" dialog
    
    > Currently, both the icon and the path are displayed
    > in the first column of the tree view and the second
    > column remains empty. This will be addressed in
    > upcoming commits.
    
    Change-Id: I16f661fb47fedb889743bb8f91b5447a21f9f88f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188892
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>
    Tested-by: Jenkins

diff --git a/vcl/qt5/QtInstanceTreeView.cxx b/vcl/qt5/QtInstanceTreeView.cxx
index 41b94fe9eea7..3dd253677fea 100644
--- a/vcl/qt5/QtInstanceTreeView.cxx
+++ b/vcl/qt5/QtInstanceTreeView.cxx
@@ -71,7 +71,7 @@ void QtInstanceTreeView::insert(const weld::TreeIter* 
pParent, int nPos, const O
         const QModelIndex aIndex = modelIndex(nPos, 0, aParentIndex);
         QStandardItem* pItem = itemFromIndex(aIndex);
         if (pStr)
-            pItem->setText(toQString(*pStr));
+            set_text(treeIter(nPos), *pStr);
         if (pId)
             pItem->setData(toQString(*pId), ROLE_ID);
 

Reply via email to