From 7f2e7e49759b4b5aa80addd8c6a49405e7dcf547 Mon Sep 17 00:00:00 2001 From: Gülşah Köse Date: Thu, 19 Jan 2023 18:10:11 +0300 Subject: Hide fill attribute widget if fill style is none MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Reviewed-by: Pedro Silva Reviewed-by: Mike Kaganski Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145876 Tested-by: Jenkins CollaboraOffice Reviewed-by: Gülşah Köse --- svx/source/sidebar/area/AreaPropertyPanelBase.cxx | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'svx') diff --git a/svx/source/sidebar/area/AreaPropertyPanelBase.cxx b/svx/source/sidebar/area/AreaPropertyPanelBase.cxx index 428d35d94d59..9c19924d6c62 100644 --- a/svx/source/sidebar/area/AreaPropertyPanelBase.cxx +++ b/svx/source/sidebar/area/AreaPropertyPanelBase.cxx @@ -246,14 +246,13 @@ IMPL_LINK_NOARG(AreaPropertyPanelBase, SelectFillTypeHdl, weld::ComboBox&, void) default: case NONE: { - mxLbFillAttr->show(); + mxLbFillAttr->hide(); mxLbFillGradFrom->hide(); mxLbFillGradTo->hide(); mxGradientStyle->hide(); mxMTRAngle->hide(); mxToolBoxColor->hide(); mxBmpImport->hide(); - mxLbFillAttr->set_sensitive(false); // #i122676# need to call a single SID_ATTR_FILL_STYLE change setFillStyle(XFillStyleItem(drawing::FillStyle_NONE)); @@ -800,8 +799,7 @@ void AreaPropertyPanelBase::updateFillStyle(bool bDisabled, bool bDefaultOrSet, default: case drawing::FillStyle_NONE: { - mxLbFillAttr->set_active(-1); - mxLbFillAttr->set_sensitive(false); + mxLbFillAttr->hide(); nPos = NONE; break; } @@ -1096,7 +1094,7 @@ void AreaPropertyPanelBase::Update() { case eFillStyle::NONE: { - mxLbFillAttr->show(); + mxLbFillAttr->hide(); mxLbFillGradFrom->hide(); mxLbFillGradTo->hide(); mxMTRAngle->hide(); -- cgit