Hello community,

here is the log from the commit of package plasma5-desktop for openSUSE:Factory 
checked in at 2020-02-19 12:40:38
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/plasma5-desktop (Old)
 and      /work/SRC/openSUSE:Factory/.plasma5-desktop.new.26092 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "plasma5-desktop"

Wed Feb 19 12:40:38 2020 rev:119 rq:774530 version:5.18.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/plasma5-desktop/plasma5-desktop.changes  
2020-02-14 16:24:46.983171388 +0100
+++ 
/work/SRC/openSUSE:Factory/.plasma5-desktop.new.26092/plasma5-desktop.changes   
    2020-02-19 12:40:49.179656876 +0100
@@ -1,0 +2,12 @@
+Fri Feb 14 22:40:49 UTC 2020 - Wolfgang Bauer <wba...@tmo.at>
+
+- Add patch to fix Alt+D shortcut handling (kde#416695):
+  * Set-toolbox-to-invisible-when-its-hidden.patch
+
+-------------------------------------------------------------------
+Fri Feb 14 13:01:17 UTC 2020 - Fabian Vogt <fab...@ritter-vogt.de>
+
+- Add patch to fix saving font settings (kde#416358):
+  * 0001-KCM-Fonts-force-need-save-to-false-during-load-to-av.patch
+
+-------------------------------------------------------------------

New:
----
  0001-KCM-Fonts-force-need-save-to-false-during-load-to-av.patch
  Set-toolbox-to-invisible-when-its-hidden.patch

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

Other differences:
------------------
++++++ plasma5-desktop.spec ++++++
--- /var/tmp/diff_new_pack.uc3bMB/_old  2020-02-19 12:40:50.183658810 +0100
+++ /var/tmp/diff_new_pack.uc3bMB/_new  2020-02-19 12:40:50.191658825 +0100
@@ -39,6 +39,8 @@
 Patch1:         0001-Use-themed-user-face-icon-in-kickoff.patch
 # PATCH-FIX-UPSTREAM
 Patch2:         0001-unlock-widgets.patch
+Patch3:         0001-KCM-Fonts-force-need-save-to-false-during-load-to-av.patch
+Patch4:         Set-toolbox-to-invisible-when-its-hidden.patch
 BuildRequires:  extra-cmake-modules >= 1.8.0
 BuildRequires:  fdupes
 BuildRequires:  glib2-devel

++++++ 0001-KCM-Fonts-force-need-save-to-false-during-load-to-av.patch ++++++
>From 0c4da0774e4388b6937b5aa98124b696debcb601 Mon Sep 17 00:00:00 2001
From: Benjamin Port <benjamin.p...@enioka.com>
Date: Thu, 13 Feb 2020 21:14:43 +0100
Subject: [PATCH] [KCM Fonts] force need save to false during load to avoid
 state to be true too early

Summary:
This will resolve a bug (apply never enabled). Bug occurs (at least) when 
kdeglobals contains QFont serialization without styleName (old style)

BUG: 416358

Reviewers: #plasma, broulik, ervin, crossi, meven, ngraham, davidedmundson

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D27384
---
 kcms/fonts/fonts.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kcms/fonts/fonts.cpp b/kcms/fonts/fonts.cpp
index 32e1239ad..f771f6e51 100644
--- a/kcms/fonts/fonts.cpp
+++ b/kcms/fonts/fonts.cpp
@@ -572,8 +572,8 @@ void KFonts::load()
     // previews
     engine()->addImageProvider("preview", new 
PreviewImageProvider(m_settings->font()));
 
-    // reload state after loading by emiting a settings signal
-    emit m_settings->activeFontChanged();
+    // KCM expect save state to be false at this point (can be true because of 
setNearestExistingFonts
+    setNeedsSave(false);
 }
 
 void KFonts::save()
-- 
2.23.0

++++++ Set-toolbox-to-invisible-when-its-hidden.patch ++++++
>From 7a48de1efa224e616d9f24f1fa1e4cdab0bce196 Mon Sep 17 00:00:00 2001
From: David Edmundson <k...@davidedmundson.co.uk>
Date: Fri, 14 Feb 2020 14:16:30 +0200
Subject: Set toolbox to invisble when it's hidden

Summary:
opacity was bound to editMode, but this did not affect visibility or
enabled.

Not only is this somewhat wasteful, enabled is very important as
otherwise shortcuts for buttons get intercepted to match the accelerator
applied to the internal buttons. This breaks all the multi-key shortcuts
that plasma uses as alt+d gets taken.

BUG: 416695
FIXED-IN: 5.18.1

Test Plan:
Mad gammaray skillz to find the bug

Checked alt+d, a now opens the widget explorer
Checked edit mode through the context menu still showed the toolbox

Reviewers: #plasma, ngraham

Reviewed By: ngraham

Subscribers: ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D27386
---
 toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml | 1 +
 toolboxes/desktoptoolbox/contents/ui/ToolBoxRoot.qml   | 2 --
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml 
b/toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml
index e706b69..e8055ec 100644
--- a/toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml
+++ b/toolboxes/desktoptoolbox/contents/ui/ToolBoxButton.qml
@@ -44,6 +44,7 @@ Item {
     }
     transformOrigin: Item.Center
     opacity: plasmoid.editMode
+    visible: plasmoid.editMode
     Behavior on opacity {
         OpacityAnimator {
             duration: units.longDuration
diff --git a/toolboxes/desktoptoolbox/contents/ui/ToolBoxRoot.qml 
b/toolboxes/desktoptoolbox/contents/ui/ToolBoxRoot.qml
index 4478d6e..9921a26 100644
--- a/toolboxes/desktoptoolbox/contents/ui/ToolBoxRoot.qml
+++ b/toolboxes/desktoptoolbox/contents/ui/ToolBoxRoot.qml
@@ -54,10 +54,8 @@ Item {
 
     ToolBoxButton {
         id: toolBoxButton
-        visible: false
         Component.onCompleted: {
             placeToolBox(plasmoid.configuration.ToolBoxButtonState);
-            toolBoxButton.visible = true
         }
     }
 
-- 
cgit v1.1




Reply via email to