[Libreoffice-commits] core.git: 4 commits - cui/source cui/uiconfig lotuswordpro/source translations
Caolán McNamara (via logerrit)
logerrit at kemper.freedesktop.org
Mon Sep 27 18:25:25 UTC 2021
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 <caolanm at redhat.com>
AuthorDate: Mon Sep 27 17:19:35 2021 +0100
Commit: Caolán McNamara <caolanm at 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 <caolanm at 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 <caolanm at redhat.com>
AuthorDate: Mon Sep 27 16:27:34 2021 +0100
Commit: Caolán McNamara <caolanm at 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 <caolanm at 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 <caolanm at redhat.com>
AuthorDate: Mon Sep 27 17:25:13 2021 +0100
Commit: Caolán McNamara <caolanm at 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 <caolanm at redhat.com>
Reviewed-by: Caolán McNamara <caolanm at 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+LibreOffice at googlemail.com>
AuthorDate: Mon Sep 27 20:04:15 2021 +0200
Commit: Gerrit Code Review <gerrit at 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
More information about the Libreoffice-commits
mailing list