Hello community,

here is the log from the commit of package libreoffice for openSUSE:Factory 
checked in at 2020-07-02 23:57:59
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/libreoffice (Old)
 and      /work/SRC/openSUSE:Factory/.libreoffice.new.3060 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "libreoffice"

Thu Jul  2 23:57:59 2020 rev:209 rq:818250 version:7.0.0.0.beta2

Changes:
--------
--- /work/SRC/openSUSE:Factory/libreoffice/libreoffice.changes  2020-06-29 
21:18:04.581730349 +0200
+++ /work/SRC/openSUSE:Factory/.libreoffice.new.3060/libreoffice.changes        
2020-07-02 23:58:51.305373295 +0200
@@ -1,0 +2,19 @@
+Wed Jul  1 20:15:15 UTC 2020 - Atri Bhattacharya <[email protected]>
+
+- Update libreoffice-stuck-widgets-in-sidebar.patch with
+  an additional fix from upstream (tdf#134360).
+
+-------------------------------------------------------------------
+Tue Jun 30 14:05:33 UTC 2020 - Atri Bhattacharya <[email protected]>
+
+- Add patches from upstream to fix regressions:
+  * libreoffice-stuck-widgets-in-sidebar.patch: some sidebar
+    widgets getting stuck in a disabled state (boo#1173410,
+    tdf#134360).
+  * libreoffice-select-correct-item-in-gallery.patch: select the
+    item in gallery before dragging it (boo#1173409, tdf#134285).
+  * libreoffice-clipped-sidebar-paragraph-widget.patch: fix too
+    wide and clipped paragraph widget in sidebar (boo#1173404,
+    tdf#134357).
+
+-------------------------------------------------------------------

New:
----
  libreoffice-clipped-sidebar-paragraph-widget.patch
  libreoffice-select-correct-item-in-gallery.patch
  libreoffice-stuck-widgets-in-sidebar.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ libreoffice.spec ++++++
--- /var/tmp/diff_new_pack.DGnDQp/_old  2020-07-02 23:58:57.045392300 +0200
+++ /var/tmp/diff_new_pack.DGnDQp/_new  2020-07-02 23:58:57.049392314 +0200
@@ -104,6 +104,9 @@
 Patch3:         mediawiki-no-broken-help.diff
 Patch4:         poppler-0.86.patch
 Patch5:         pyuno-nopwd.patch
+Patch6:         libreoffice-stuck-widgets-in-sidebar.patch
+Patch7:         libreoffice-select-correct-item-in-gallery.patch
+Patch8:         libreoffice-clipped-sidebar-paragraph-widget.patch
 # try to save space by using hardlinks
 Patch990:       install-with-hardlinks.diff
 # save time by relying on rpm check rather than doing stupid find+grep
@@ -963,6 +966,9 @@
 %patch3
 %patch4 -p1
 %patch5 -p1
+%patch6 -p1
+%patch7 -p1
+%patch8 -p1
 %patch990 -p1
 %patch991 -p1
 








++++++ libreoffice-clipped-sidebar-paragraph-widget.patch ++++++
>From f8e8b0d1747e16394d43eec7a1ef2b33b5678619 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <[email protected]>
Date: Tue, 30 Jun 2020 17:13:24 +0100
Subject: [PATCH] tdf#134357 we can reduce the min toolbar sidebar width

since,

commit 754bea8c6a17889f9d4d23bf98847b6b67253220
Date:   Wed Apr 29 15:06:42 2020 +0100

    remove the padding around the dropdown button of GtkMenuToolButtons

the GtkMenuToolButton aren't as wide anymore so we don't need a min of 5
GtkToolButtons widths for tdf#130197, now the max of either 4 GtkToolButtons or
that of 1 GtkToolButtons + 2 GtkMenuToolButton is sufficient

Change-Id: I7296805b4b3c0d86ce3d08cadc8c2659104af619
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97502
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <[email protected]>
---

diff --git a/svx/source/dialog/dlgctrl.cxx b/svx/source/dialog/dlgctrl.cxx
index f63196c..75f9319 100644
--- a/svx/source/dialog/dlgctrl.cxx
+++ b/svx/source/dialog/dlgctrl.cxx
@@ -1446,9 +1446,11 @@
         // use the, filled-in by dispatcher, width of measurewidth as the width
         // of a "standard" column in a two column panel
         std::unique_ptr<weld::Builder> 
xBuilder(Application::CreateBuilder(&rToolbar, "svx/ui/measurewidthbar.ui"));
-        std::unique_ptr<weld::Toolbar> 
xToolbar(xBuilder->weld_toolbar("measurewidth"));
-        std::unique_ptr<ToolbarUnoDispatcher> xDispatcher(new 
ToolbarUnoDispatcher(*xToolbar, *xBuilder, rFrame));
-        nColumnWidth = xToolbar->get_preferred_size().Width();
+        std::unique_ptr<weld::Toolbar> 
xToolbar1(xBuilder->weld_toolbar("measurewidth1"));
+        std::unique_ptr<ToolbarUnoDispatcher> xDispatcher1(new 
ToolbarUnoDispatcher(*xToolbar1, *xBuilder, rFrame));
+        std::unique_ptr<weld::Toolbar> 
xToolbar2(xBuilder->weld_toolbar("measurewidth2"));
+        std::unique_ptr<ToolbarUnoDispatcher> xDispatcher2(new 
ToolbarUnoDispatcher(*xToolbar2, *xBuilder, rFrame));
+        nColumnWidth = std::max(xToolbar1->get_preferred_size().Width(), 
xToolbar2->get_preferred_size().Width());
         eSize = rToolbar.get_icon_size();
     }
     rToolbar.set_size_request(nColumnWidth, -1);
diff --git a/svx/uiconfig/ui/measurewidthbar.ui 
b/svx/uiconfig/ui/measurewidthbar.ui
index ad96cc7..b901963 100644
--- a/svx/uiconfig/ui/measurewidthbar.ui
+++ b/svx/uiconfig/ui/measurewidthbar.ui
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<!-- Generated with glade 3.22.1 -->
+<!-- Generated with glade 3.22.2 -->
 <interface domain="svx">
   <requires lib="gtk+" version="3.18"/>
   <object class="GtkBox">
@@ -7,65 +7,43 @@
     <property name="can_focus">False</property>
     <property name="orientation">vertical</property>
     <child>
-      <object class="GtkToolbar" id="measurewidth">
+      <object class="GtkToolbar" id="measurewidth1">
         <property name="visible">True</property>
         <property name="can_focus">False</property>
         <property name="toolbar_style">icons</property>
         <property name="show_arrow">False</property>
         <property name="icon_size">2</property>
         <child>
-          <object class="GtkToolButton" id=".uno:MergeCells">
+          <object class="GtkMenuToolButton" id=".uno:TableCellBackgroundColor">
             <property name="visible">True</property>
             <property name="can_focus">False</property>
             <property name="use_underline">True</property>
           </object>
           <packing>
             <property name="expand">False</property>
-            <property name="homogeneous">True</property>
+            <property name="homogeneous">False</property>
           </packing>
         </child>
         <child>
-          <object class="GtkToolButton" id=".uno:SplitCell">
+          <object class="GtkMenuToolButton" id=".uno:SetBorderStyle">
             <property name="visible">True</property>
             <property name="can_focus">False</property>
             <property name="use_underline">True</property>
           </object>
           <packing>
             <property name="expand">False</property>
-            <property name="homogeneous">True</property>
+            <property name="homogeneous">False</property>
           </packing>
         </child>
         <child>
-          <object class="GtkToolButton" id=".uno:SplitTable">
+          <object class="GtkToolButton" id=".uno:InsertFormula">
             <property name="visible">True</property>
             <property name="can_focus">False</property>
             <property name="use_underline">True</property>
           </object>
           <packing>
             <property name="expand">False</property>
-            <property name="homogeneous">True</property>
-          </packing>
-        </child>
-        <child>
-          <object class="GtkToolButton" id=".uno:DeleteColumns">
-            <property name="visible">True</property>
-            <property name="can_focus">False</property>
-            <property name="use_underline">True</property>
-          </object>
-          <packing>
-            <property name="expand">False</property>
-            <property name="homogeneous">True</property>
-          </packing>
-        </child>
-        <child>
-          <object class="GtkToolButton" id=".uno:DeleteTable">
-            <property name="visible">True</property>
-            <property name="can_focus">False</property>
-            <property name="use_underline">True</property>
-          </object>
-          <packing>
-            <property name="expand">False</property>
-            <property name="homogeneous">True</property>
+            <property name="homogeneous">False</property>
           </packing>
         </child>
       </object>
@@ -75,5 +53,63 @@
         <property name="position">0</property>
       </packing>
     </child>
+    <child>
+      <object class="GtkToolbar" id="measurewidth2">
+        <property name="visible">True</property>
+        <property name="can_focus">False</property>
+        <property name="toolbar_style">icons</property>
+        <property name="show_arrow">False</property>
+        <property name="icon_size">2</property>
+        <child>
+          <object class="GtkToolButton" id=".uno:InsertRowsBefore">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="use_underline">True</property>
+          </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="homogeneous">False</property>
+          </packing>
+        </child>
+        <child>
+          <object class="GtkToolButton" id=".uno:InsertRowsAfter">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="use_underline">True</property>
+          </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="homogeneous">False</property>
+          </packing>
+        </child>
+        <child>
+          <object class="GtkToolButton" id=".uno:InsertColumnsBefore">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="use_underline">True</property>
+          </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="homogeneous">False</property>
+          </packing>
+        </child>
+        <child>
+          <object class="GtkToolButton" id=".uno:InsertColumnsAfter">
+            <property name="visible">True</property>
+            <property name="can_focus">False</property>
+            <property name="use_underline">True</property>
+          </object>
+          <packing>
+            <property name="expand">False</property>
+            <property name="homogeneous">False</property>
+          </packing>
+        </child>
+      </object>
+      <packing>
+        <property name="expand">False</property>
+        <property name="fill">True</property>
+        <property name="position">1</property>
+      </packing>
+    </child>
   </object>
 </interface>

++++++ libreoffice-select-correct-item-in-gallery.patch ++++++
>From 8753cc269a350cc468e4f54f961f304a35144467 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <[email protected]>
Date: Thu, 25 Jun 2020 21:05:26 +0100
Subject: [PATCH] tdf#134285 select the item before dragging it

so we don't get the previously selected item

Change-Id: Ifd6e46d97f1de0de86403b9c237ee8d2cd3a6cd3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97137
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <[email protected]>
---

diff --git a/svx/source/gallery2/galctrl.cxx b/svx/source/gallery2/galctrl.cxx
index 6604eee..47ef1757 100644
--- a/svx/source/gallery2/galctrl.cxx
+++ b/svx/source/gallery2/galctrl.cxx
@@ -400,6 +400,7 @@
 
 bool GalleryIconView::StartDrag()
 {
+    Select();
     return mpParent->StartDrag();
 }
 
++++++ libreoffice-stuck-widgets-in-sidebar.patch ++++++
>From a76fcd9f9718b269c0c84534a75114433369bf92 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <[email protected]>
Date: Mon, 29 Jun 2020 20:02:26 +0100
Subject: [PATCH] tdf#134360 some sidebar widgets getting stuck in a disabled 
state

Change-Id: I655525679dd2471e9d0c1a5af246901594b188ed
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97387
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <[email protected]>
---

diff --git a/svx/source/sidebar/paragraph/ParaPropertyPanel.cxx 
b/svx/source/sidebar/paragraph/ParaPropertyPanel.cxx
index 324a2e5..12a9eba 100644
--- a/svx/source/sidebar/paragraph/ParaPropertyPanel.cxx
+++ b/svx/source/sidebar/paragraph/ParaPropertyPanel.cxx
@@ -254,7 +254,12 @@
         break;
     }
 
-    if( pState && eState >= SfxItemState::DEFAULT )
+    bool bDisabled = eState == SfxItemState::DISABLED;
+    mxLeftIndent->set_sensitive(!bDisabled);
+    mxRightIndent->set_sensitive(!bDisabled);
+    mxFLineIndent->set_sensitive(!bDisabled);
+
+    if (pState && eState >= SfxItemState::DEFAULT)
     {
         const SvxLRSpaceItem* pSpace = static_cast<const 
SvxLRSpaceItem*>(pState);
         maTxtLeft = pSpace->GetTextLeft();
@@ -314,13 +319,7 @@
         nfVal = static_cast<long>(mxFLineIndent->normalize( nfVal ));
         mxFLineIndent->set_value( nfVal, FieldUnit::MM_100TH );
     }
-    else if( eState == SfxItemState::DISABLED )
-    {
-        mxLeftIndent->set_sensitive(false);
-        mxRightIndent->set_sensitive(false);
-        mxFLineIndent->set_sensitive(false);
-    }
-    else
+    else if (eState != SfxItemState::DISABLED )
     {
         mxLeftIndent->set_text("");
         mxRightIndent->set_text("");
@@ -335,6 +334,10 @@
     mxTopDist->set_max( mxTopDist->normalize( MAX_DURCH ), 
MapToFieldUnit(m_eULSpaceUnit) );
     mxBottomDist->set_max( mxBottomDist->normalize( MAX_DURCH ), 
MapToFieldUnit(m_eULSpaceUnit) );
 
+    bool bDisabled = eState == SfxItemState::DISABLED;
+    mxTopDist->set_sensitive(!bDisabled);
+    mxBottomDist->set_sensitive(!bDisabled);
+
     if( pState && eState >= SfxItemState::DEFAULT )
     {
         const SvxULSpaceItem* pOldItem = static_cast<const 
SvxULSpaceItem*>(pState);
@@ -355,12 +358,7 @@
         nVal = mxBottomDist->normalize( nVal );
         mxBottomDist->set_value( nVal, FieldUnit::MM_100TH );
     }
-    else if(eState == SfxItemState::DISABLED )
-    {
-        mxTopDist->set_sensitive(false);
-        mxBottomDist->set_sensitive(false);
-    }
-    else
+    else if (eState != SfxItemState::DISABLED )
     {
         mxTopDist->set_text("");
         mxBottomDist->set_text("");
>From c4eb732bc38156d2d6c373c6ce3520eecf323d32 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <[email protected]>
Date: Tue, 30 Jun 2020 20:05:30 +0100
Subject: [PATCH] Related: tdf#134360 table row/height widgets can get stuck 
insensitive

Change-Id: I1be53f33f60c510757c67f1da8f863539ecbffdb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97573
Tested-by: Caolán McNamara <[email protected]>
Reviewed-by: Caolán McNamara <[email protected]>
---

diff --git a/sw/source/uibase/sidebar/TableEditPanel.cxx 
b/sw/source/uibase/sidebar/TableEditPanel.cxx
index 4d76cc0..46f75cd 100644
--- a/sw/source/uibase/sidebar/TableEditPanel.cxx
+++ b/sw/source/uibase/sidebar/TableEditPanel.cxx
@@ -45,6 +45,9 @@
     {
         case SID_ATTR_TABLE_ROW_HEIGHT:
         {
+            bool bDisabled = eState == SfxItemState::DISABLED;
+            m_xRowHeightEdit->set_sensitive(!bDisabled);
+
             if (pState && eState >= SfxItemState::DEFAULT)
             {
                 const SfxUInt32Item* pItem = static_cast<const 
SfxUInt32Item*>(pState);
@@ -55,18 +58,16 @@
                     m_xRowHeightEdit->set_value(nNewHeight, FieldUnit::TWIP);
                 }
             }
-            else if (eState == SfxItemState::DISABLED)
-            {
-                m_xRowHeightEdit->set_sensitive(false);
-            }
-            else
-            {
+            else if (eState != SfxItemState::DISABLED)
                 m_xRowHeightEdit->set_text("");
-            }
+
             break;
         }
         case SID_ATTR_TABLE_COLUMN_WIDTH:
         {
+            bool bDisabled = eState == SfxItemState::DISABLED;
+            m_xColumnWidthEdit->set_sensitive(!bDisabled);
+
             if (pState && eState >= SfxItemState::DEFAULT)
             {
                 const SfxUInt32Item* pItem = static_cast<const 
SfxUInt32Item*>(pState);
@@ -77,14 +78,9 @@
                     m_xColumnWidthEdit->set_value(nNewWidth, FieldUnit::TWIP);
                 }
             }
-            else if (eState == SfxItemState::DISABLED)
-            {
-                m_xColumnWidthEdit->set_sensitive(false);
-            }
-            else
-            {
+            else if (eState != SfxItemState::DISABLED)
                 m_xColumnWidthEdit->set_text("");
-            }
+
             break;
         }
     }




Reply via email to