sfx2/source/sidebar/DeckLayouter.cxx |   11 +++++++++++
 1 file changed, 11 insertions(+)

New commits:
commit 241dbf223c17800ceff7023452f3b9c79e77ebc0
Author:     Samuel Mehrbrodt <samuel.mehrbr...@cib.de>
AuthorDate: Wed Jul 10 14:35:24 2019 +0200
Commit:     Samuel Mehrbrodt <samuel.mehrbr...@cib.de>
CommitDate: Thu Jul 11 07:26:00 2019 +0200

    Warn when LayoutSize constraints were not respected
    
    Change-Id: I5145bc4e10928ed456b36617de3345512b7a35d9
    Reviewed-on: https://gerrit.libreoffice.org/75351
    Tested-by: Jenkins
    Reviewed-by: Samuel Mehrbrodt <samuel.mehrbr...@cib.de>

diff --git a/sfx2/source/sidebar/DeckLayouter.cxx 
b/sfx2/source/sidebar/DeckLayouter.cxx
index 120906307e9a..a27f6f7847ca 100644
--- a/sfx2/source/sidebar/DeckLayouter.cxx
+++ b/sfx2/source/sidebar/DeckLayouter.cxx
@@ -377,6 +377,17 @@ void GetRequestedSizes (
                 if (xPanel.is())
                 {
                     aLayoutSize = 
xPanel->getHeightForWidth(rContentBox.GetWidth());
+                    if (!(0 <= aLayoutSize.Minimum && aLayoutSize.Minimum <= 
aLayoutSize.Preferred
+                          && aLayoutSize.Preferred <= aLayoutSize.Maximum))
+                    {
+                        SAL_WARN("sfx.sidebar", "Please follow LayoutSize 
constraints: 0 ≤ "
+                                                "Minimum ≤ Preferred ≤ 
Maximum."
+                                                " Currently: Minimum: "
+                                                    << aLayoutSize.Minimum
+                                                    << " Preferred: " << 
aLayoutSize.Preferred
+                                                    << " Maximum: " << 
aLayoutSize.Maximum);
+                    }
+
                     sal_Int32 nWidth = xPanel->getMinimalWidth();
 
                     uno::Reference<frame::XDesktop2> xDesktop
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to