cui/source/inc/border.hxx                    |    1 +
 cui/source/tabpages/border.cxx               |    7 +++++++
 cui/uiconfig/ui/borderpage.ui                |    4 +++-
 lotuswordpro/source/filter/lwppagelayout.cxx |    7 +++++++
 translations                                 |    2 +-
 5 files changed, 19 insertions(+), 2 deletions(-)

New commits:
commit fc5856263439846c9e692e1ccaddf3312530b1a2
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Mon Sep 27 17:19:35 2021 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Mon Sep 27 20:25:04 2021 +0200

    lock the combobox + spinbutton group width to its original all-shown width
    
    so the notebook page contents don't jump around when the spinbox is
    hidden/shown
    
    Change-Id: Ib6ad47644b55d857dae972340444d7bac67278e8
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122724
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/cui/source/inc/border.hxx b/cui/source/inc/border.hxx
index 2a3d14ed5062..433ab7425e5c 100644
--- a/cui/source/inc/border.hxx
+++ b/cui/source/inc/border.hxx
@@ -129,6 +129,7 @@ private:
 
     std::unique_ptr<SvtLineListBox> m_xLbLineStyle;
     std::unique_ptr<ColorListBox> m_xLbLineColor;
+    std::unique_ptr<weld::Widget> m_xLineWidthGroup;
     std::unique_ptr<weld::ComboBox> m_xLineWidthLB;
     std::unique_ptr<weld::MetricSpinButton> m_xLineWidthMF;
 
diff --git a/cui/source/tabpages/border.cxx b/cui/source/tabpages/border.cxx
index 70ec64c1a3bc..59ecc1cfd2ee 100644
--- a/cui/source/tabpages/border.cxx
+++ b/cui/source/tabpages/border.cxx
@@ -294,6 +294,7 @@ SvxBorderTabPage::SvxBorderTabPage(weld::Container* pPage, 
weld::DialogControlle
     , m_xLbLineStyle(new 
SvtLineListBox(m_xBuilder->weld_menu_button("linestylelb")))
     , m_xLbLineColor(new 
ColorListBox(m_xBuilder->weld_menu_button("linecolorlb"),
                 [this]{ return GetDialogController()->getDialog(); }))
+    , m_xLineWidthGroup(m_xBuilder->weld_widget("gridlinewidth"))
     , m_xLineWidthLB(m_xBuilder->weld_combo_box("linewidthlb"))
     , m_xLineWidthMF(m_xBuilder->weld_metric_spin_button("linewidthmf", 
FieldUnit::POINT))
     , m_xSpacingFrame(m_xBuilder->weld_container("spacing"))
@@ -524,6 +525,12 @@ SvxBorderTabPage::SvxBorderTabPage(weld::Container* pPage, 
weld::DialogControlle
     m_xWndPresets->SetSelectHdl( LINK( this, SvxBorderTabPage, SelPreHdl_Impl 
) );
     m_xWndShadows->SetSelectHdl( LINK( this, SvxBorderTabPage, SelSdwHdl_Impl 
) );
 
+    // lock the group to its original width where both widgets are shown so the
+    // notebook page contents don't jump around when the spinbox is
+    // hidden/shown
+    Size aOrigGroupSize(m_xLineWidthGroup->get_preferred_size());
+    m_xLineWidthGroup->set_size_request(aOrigGroupSize.Width(), -1);
+
     FillValueSets();
     FillLineListBox_Impl();
 
commit c156fc0b844d6d7e31dc5b6a5e12ef71d8f631c8
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Mon Sep 27 16:27:34 2021 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Mon Sep 27 20:24:44 2021 +0200

    border page combobox defaults to taking up a lot of space
    
    Change-Id: I29d7debcd39a23b224c8c4660a7edf2f1e9e4a90
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122723
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/cui/uiconfig/ui/borderpage.ui b/cui/uiconfig/ui/borderpage.ui
index 2847f0b736cb..d748cd1ca395 100644
--- a/cui/uiconfig/ui/borderpage.ui
+++ b/cui/uiconfig/ui/borderpage.ui
@@ -251,17 +251,20 @@
               </packing>
             </child>
             <child>
+              <!-- n-columns=2 n-rows=1 -->
               <object class="GtkGrid" id="gridlinewidth">
                 <property name="visible">True</property>
                 <property name="can_focus">False</property>
                 <property name="row_spacing">0</property>
                 <property name="column_spacing">6</property>
+                <property name="halign">start</property>
                 <property name="margin-start">0</property>
                 <property name="margin-top">0</property>
                 <child>
                   <object class="GtkComboBoxText" id="linewidthlb">
                     <property name="visible">True</property>
                     <property name="can_focus">False</property>
+                    <property name="hexpand">False</property>
                     <items>
                       <item translatable="yes" 
context="borderpage|linewidthlb">Thin</item>
                       <item translatable="yes" 
context="borderpage|linewidthlb">Medium</item>
@@ -277,7 +280,6 @@
                   <object class="GtkSpinButton" id="linewidthmf">
                     <property name="visible">True</property>
                     <property name="can_focus">True</property>
-                    <property name="halign">start</property>
                     <property name="activates_default">True</property>
                     <property name="adjustment">adjustment1</property>
                     <property name="digits">2</property>
commit 1010c16c5f4c582adfa9d24036e9f98137e5ede7
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Mon Sep 27 17:25:13 2021 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Mon Sep 27 20:23:59 2021 +0200

    ofz#39374: avoid infinite loop
    
    Change-Id: I0aec664279dea99df865761f34f64aa4fa3d9e0d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122725
    Tested-by: Caolán McNamara <caol...@redhat.com>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/lotuswordpro/source/filter/lwppagelayout.cxx 
b/lotuswordpro/source/filter/lwppagelayout.cxx
index cc8d6b274360..0a72db8f7d30 100644
--- a/lotuswordpro/source/filter/lwppagelayout.cxx
+++ b/lotuswordpro/source/filter/lwppagelayout.cxx
@@ -74,6 +74,7 @@
 #include <lwpchangemgr.hxx>
 #include <lwpfilehdr.hxx>
 #include <lwpglobalmgr.hxx>
+#include <o3tl/sorted_vector.hxx>
 #include <sal/log.hxx>
 #include <vcl/print.hxx>
 
@@ -548,8 +549,11 @@ LwpPageLayout* LwpPageLayout::GetOddChildLayout()
     {
         rtl::Reference<LwpVirtualLayout> xLay(
             dynamic_cast<LwpVirtualLayout*>(GetChildHead().obj().get()));
+        o3tl::sorted_vector<LwpVirtualLayout*> aSeen;
         while (xLay.is())
         {
+            aSeen.insert(xLay.get());
+
             if (xLay->GetLayoutType() == LWP_PAGE_LAYOUT)
             {
                 LwpPageLayout* pPageLayout = 
static_cast<LwpPageLayout*>(xLay.get());
@@ -560,6 +564,9 @@ LwpPageLayout* LwpPageLayout::GetOddChildLayout()
                 }
             }
             
xLay.set(dynamic_cast<LwpVirtualLayout*>(xLay->GetNext().obj().get()));
+
+            if (aSeen.find(xLay.get()) != aSeen.end())
+                throw std::runtime_error("loop in conversion");
         }
     }
     return nullptr;
commit 10c8bd1ed8fbc4cf387803fcb9b70dcfe776b4ab
Author:     Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
AuthorDate: Mon Sep 27 20:04:15 2021 +0200
Commit:     Gerrit Code Review <ger...@gerrit.libreoffice.org>
CommitDate: Mon Sep 27 20:04:15 2021 +0200

    Update git submodules
    
    * Update translations from branch 'master'
      to 0cb98568b1070b0178a398e2befd36808364e37e
      - update translations for master
    
        and force-fix errors using pocheck
    
        Change-Id: Ib11aec7e448b083f697b9705d29795cd2e5a55f0
    
      - update translations for master
    
        and foce-fix errors using pocheck
    
        Change-Id: I12d26e666d46993fd4ff26616c09be9e5bf82fcd

diff --git a/translations b/translations
index 4460c7babd57..0cb98568b107 160000
--- a/translations
+++ b/translations
@@ -1 +1 @@
-Subproject commit 4460c7babd574c5040a528d10fe51001c27680bc
+Subproject commit 0cb98568b1070b0178a398e2befd36808364e37e

Reply via email to