[Libreoffice-commits] core.git: svx/source

[Gülşah Köse (via logerrit)]

 svx/source/sidebar/area/AreaPropertyPanelBase.cxx |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 48e84907845daaf633991f88efb85d93c058a88d
Author:     Gülşah Köse <gulsah.k...@collabora.com>
AuthorDate: Thu Jan 19 18:10:11 2023 +0300
Commit:     Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Fri Jan 20 09:51:23 2023 +0000

    Hide fill Attribute widget if fill style is none
    
    When Fill style is none we can not use fill attribute element. So It is
    better to hide.
    
    Change-Id: I88d5e49448040b3afa18fbf66377d254716e7415
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145817
    Tested-by: Jenkins
    Tested-by: Pedro Silva <pedro.si...@collabora.com>
    Reviewed-by: Pedro Silva <pedro.si...@collabora.com>
    Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>

diff --git a/svx/source/sidebar/area/AreaPropertyPanelBase.cxx 
b/svx/source/sidebar/area/AreaPropertyPanelBase.cxx
index deac0eb9dcca..e54002ef9589 100644
--- a/svx/source/sidebar/area/AreaPropertyPanelBase.cxx
+++ b/svx/source/sidebar/area/AreaPropertyPanelBase.cxx
@@ -876,8 +876,7 @@ void AreaPropertyPanelBase::updateFillStyle(bool bDisabled, 
bool bDefaultOrSet,
             default:
             case drawing::FillStyle_NONE:
             {
-                mxLbFillAttr->set_active(-1);
-                mxLbFillAttr->set_sensitive(false);
+                mxLbFillAttr->hide();
                 // "Use slide background" also uses FillStyle_NONE internally,
                 // don't switch listbox in that case (will be handled by 
updateFillUseBackground)
                 nPos = meLastXFS == USE_BACKGROUND ? USE_BACKGROUND : NONE;