chart2/source/controller/sidebar/ChartColorWrapper.cxx |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 5f193d851e3567e1018a61763846698e02ce38ec
Author:     Dennis Francis <dennis.fran...@collabora.com>
AuthorDate: Sun Feb 23 22:38:15 2020 +0530
Commit:     Andras Timar <andras.ti...@collabora.com>
CommitDate: Fri Feb 28 22:54:43 2020 +0100

    chart2: Fix the color uno command in ChartColorWrapper
    
    The uno command for color depends on the property name. For LineColor
    it should be .uno:XLineColor and the only other case is FillColor for
    which it should be .uno:FillColor. Without this fix, on selecting
    the chart for editing the first time, the sidebar line-color control
    is disabled as a side-effect.
    
    Change-Id: Ia71ed2f6d9e0f31523f1415f3ee089fd9d7d1b2d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89304
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Andras Timar <andras.ti...@collabora.com>
    (cherry picked from commit 9a80969f3115fc33778005861442f91127344dc0)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/89677
    Tested-by: Jenkins

diff --git a/chart2/source/controller/sidebar/ChartColorWrapper.cxx 
b/chart2/source/controller/sidebar/ChartColorWrapper.cxx
index 9a8c568a863d..d9903b12c275 100644
--- a/chart2/source/controller/sidebar/ChartColorWrapper.cxx
+++ b/chart2/source/controller/sidebar/ChartColorWrapper.cxx
@@ -96,12 +96,15 @@ void ChartColorWrapper::updateModel(const 
css::uno::Reference<css::frame::XModel
 
 void ChartColorWrapper::updateData()
 {
+    static const OUStringLiteral aLineColor = "LineColor";
+    static const OUStringLiteral aCommands[2] = {".uno:XLineColor", 
".uno:FillColor"};
+
     css::uno::Reference<css::beans::XPropertySet> xPropSet = 
getPropSet(mxModel);
     if (!xPropSet.is())
         return;
 
     css::util::URL aUrl;
-    aUrl.Complete = ".uno:FillColor";
+    aUrl.Complete = (maPropertyName == aLineColor) ? aCommands[0] : 
aCommands[1];
 
     css::frame::FeatureStateEvent aEvent;
     aEvent.FeatureURL = aUrl;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to