vcl/qt5/QtInstanceBuilder.cxx  |    2 ++
 vcl/qt5/QtInstanceTextView.cxx |   10 +++++++++-
 xmlsecurity/inc/bitmaps.hlst   |    2 +-
 3 files changed, 12 insertions(+), 2 deletions(-)

New commits:
commit 96596b48681af169256119e84a5a150aedbe4422
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Wed Jul 30 13:49:09 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Thu Jul 31 18:49:46 2025 +0200

    tdf#130857 xmlsecurity: Fix path for unvalidated cert icon
    
    The path was missing an "s" in "xmlsecurity".
    
    As a consequence, the icon couldn't be loaded, and thus
    couldn't be displayed.
    
    For the qt6 VCL plugin with SAL_VCL_QT_USE_WELDED_WIDGETS=1,
    it triggered first a warning, and then an assert:
    
        warn:vcl:3089384:3089384:vcl/source/bitmap/BitmapEx.cxx:103: 
BitmapEx::BitmapEx(): could not load image 
xmlecurity/res/notcertificate_40x56.png via icon theme breeze
        soffice.bin: 
/home/michi/development/git/libreoffice/include/vcl/qt/QtUtils.hxx:48: QPixmap 
toQPixmap(const BitmapEx &): Assertion `!aPixmap.isNull() && "Failed to create 
icon pixmap"' failed.
    
    Support for that dialog was added in previous commit
    
        Change-Id: Ib0ec32ff50744f4e265a743f2b0e5f240253ee5f
        Author: Michael Weghorn <m.wegh...@posteo.de>
        Date:   Wed Jul 30 11:51:12 2025 +0200
    
            tdf#130857 qt weld: Support "View Certificate" dialog
    
    The issue could be triggered as follows when a GPG certificate
    that is not considered validated is used:
    
    * start Writer
    * save doc
    * "File" -> "Digital Signatures" -> "Digital Signatures"
    * press "Sign Document"
    * select an existing certificate/GPG key
    * press "View Certificate"
    
    Now, the icon is shown as expected with all of gtk3, gen and qt6.
    
    Path was spelled the wrong way way ever since
    
        commit 7bd87eb30ca5df683fa3bd94ca5be9dbca362c03
        Date:   Wed Apr 26 09:35:24 2017 +0100
    
            remove xmlsecurity bitmaps from .src files
    
    Change-Id: Icbe2f56dc0b831a0d56fc2707fed6480cb0b9bae
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188600
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/xmlsecurity/inc/bitmaps.hlst b/xmlsecurity/inc/bitmaps.hlst
index 42248818392f..f11359db61bf 100644
--- a/xmlsecurity/inc/bitmaps.hlst
+++ b/xmlsecurity/inc/bitmaps.hlst
@@ -9,7 +9,7 @@
 
 #pragma once
 
-inline constexpr OUString BMP_STATE_NOT_VALIDATED = 
u"xmlecurity/res/notcertificate_40x56.png"_ustr;
+inline constexpr OUString BMP_STATE_NOT_VALIDATED = 
u"xmlsecurity/res/notcertificate_40x56.png"_ustr;
 
 #define BMP_CERT_OK                     "xmlsecurity/res/certificate_16.png"
 #define BMP_CERT_NOT_OK                 "xmlsecurity/res/notcertificate_16.png"
commit 2441ba4e65538560177621560b16c5078d15e4fc
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Wed Jul 30 11:51:12 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Thu Jul 31 18:49:40 2025 +0200

    tdf#130857 qt weld: Support "View Certificate" dialog
    
    This means that native Qt widgets are used for that dialog
    now when using the qt5 or qt6 VCL plugin and starting LO with
    environment variable SAL_VCL_QT_USE_WELDED_WIDGETS=1 set.
    
    This dialog can be triggered as follows:
    
    * start Writer
    * save doc
    * "File" -> "Digital Signatures" -> "Digital Signatures"
    * press "Sign Document"
    * select an existing certificate/GPG key
    * press "View Certificate"
    
    Change-Id: Ib0ec32ff50744f4e265a743f2b0e5f240253ee5f
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188599
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/vcl/qt5/QtInstanceBuilder.cxx b/vcl/qt5/QtInstanceBuilder.cxx
index e1ea56634e0d..31bfced4044c 100644
--- a/vcl/qt5/QtInstanceBuilder.cxx
+++ b/vcl/qt5/QtInstanceBuilder.cxx
@@ -190,6 +190,7 @@ bool QtInstanceBuilder::IsUIFileSupported(const OUString& 
rUIFile, const weld::W
         u"xmlsec/ui/macrosecuritydialog.ui"_ustr,
         u"xmlsec/ui/securitylevelpage.ui"_ustr,
         u"xmlsec/ui/securitytrustpage.ui"_ustr,
+        u"xmlsec/ui/viewcertdialog.ui"_ustr,
         u"writerperfect/ui/exportepub.ui"_ustr,
     };
 
@@ -202,6 +203,7 @@ bool QtInstanceBuilder::IsUIFileSupported(const OUString& 
rUIFile, const weld::W
         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 eb248d7c15b4027065afa1aedd9afe6a5d5e1e14
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Wed Jul 30 11:50:01 2025 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Thu Jul 31 18:49:35 2025 +0200

    tdf#130857 qt weld: Implement QtInstanceTextView::set_monospace
    
    Used e.g. by the "View Certificate" dialog for which
    support will be added in an upcoming commit.
    
    Change-Id: Ibd2c8236df812481f81274be8d0c53e268fcf7d2
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/188598
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/vcl/qt5/QtInstanceTextView.cxx b/vcl/qt5/QtInstanceTextView.cxx
index fab6f2809561..cb37716a43b1 100644
--- a/vcl/qt5/QtInstanceTextView.cxx
+++ b/vcl/qt5/QtInstanceTextView.cxx
@@ -91,7 +91,15 @@ bool QtInstanceTextView::get_editable() const
     return bEditable;
 }
 
-void QtInstanceTextView::set_monospace(bool) { assert(false && "Not 
implemented yet"); }
+void QtInstanceTextView::set_monospace(bool bMonospace)
+{
+    SolarMutexGuard g;
+
+    GetQtInstance().RunInMainThread([&] {
+        m_pTextEdit->setFont(bMonospace ? 
QFontDatabase::systemFont(QFontDatabase::FixedFont)
+                                        : QApplication::font());
+    });
+}
 
 void QtInstanceTextView::set_max_length(int) { assert(false && "Not 
implemented yet"); }
 

Reply via email to