[Libreoffice-commits] core.git: 2 commits - chart2/Library_chartcontroller.mk chart2/source include/sfx2 include/svx sfx2/source svx/source

Markus Mohrhard markus.mohrhard at googlemail.com
Wed Aug 19 15:15:20 PDT 2015


 chart2/Library_chartcontroller.mk                      |    1 
 chart2/source/controller/sidebar/ChartAreaPanel.cxx    |   17 ++
 chart2/source/controller/sidebar/ChartAreaPanel.hxx    |    3 
 chart2/source/controller/sidebar/ChartColorWrapper.cxx |  101 +++++++++++++++++
 chart2/source/controller/sidebar/ChartColorWrapper.hxx |   47 +++++++
 include/sfx2/sidebar/SidebarToolBox.hxx                |    2 
 include/svx/PaletteManager.hxx                         |    5 
 include/svx/sidebar/AreaPropertyPanelBase.hxx          |    3 
 include/svx/tbcontrl.hxx                               |    4 
 sfx2/source/sidebar/SidebarToolBox.cxx                 |    8 +
 svx/source/tbxctrls/PaletteManager.cxx                 |   10 +
 svx/source/tbxctrls/colorwindow.hxx                    |    7 +
 svx/source/tbxctrls/tbcontrl.cxx                       |   24 ++--
 13 files changed, 220 insertions(+), 12 deletions(-)

New commits:
commit acd818d9735a6fa4e38016b16ba7efc880058714
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Thu Aug 20 00:13:19 2015 +0200

    update the color button when we select an object
    
    Change-Id: Ib45a02248819c35844ef83435c33ab1c82f5c4dc

diff --git a/chart2/source/controller/sidebar/ChartAreaPanel.cxx b/chart2/source/controller/sidebar/ChartAreaPanel.cxx
index dbf9a68..6377e64 100644
--- a/chart2/source/controller/sidebar/ChartAreaPanel.cxx
+++ b/chart2/source/controller/sidebar/ChartAreaPanel.cxx
@@ -25,6 +25,13 @@ namespace chart { namespace sidebar {
 
 namespace {
 
+SvxColorToolBoxControl* getColorToolBoxControl(sfx2::sidebar::SidebarToolBox* pToolBoxColor)
+{
+    css::uno::Reference<css::frame::XToolbarController> xController = pToolBoxColor->GetFirstController();
+    SvxColorToolBoxControl* pToolBoxColorControl = dynamic_cast<SvxColorToolBoxControl*>(xController.get());
+    return pToolBoxColorControl;
+}
+
 OUString getCID(css::uno::Reference<css::frame::XModel> xModel)
 {
     css::uno::Reference<css::frame::XController> xController(xModel->getCurrentController());
@@ -240,7 +247,7 @@ ChartAreaPanel::ChartAreaPanel(vcl::Window* pParent,
     mxSelectionListener(new ChartSidebarSelectionListener(this)),
     mbUpdate(true),
     mbModelValid(true),
-    maFillColorWrapper(mxModel)
+    maFillColorWrapper(mxModel, getColorToolBoxControl(mpToolBoxColor.get()))
 {
     std::vector<ObjectType> aAcceptedTypes { OBJECTTYPE_PAGE, OBJECTTYPE_DIAGRAM, OBJECTTYPE_DATA_SERIES, OBJECTTYPE_TITLE, OBJECTTYPE_LEGEND};
     mxSelectionListener->setAcceptedTypes(aAcceptedTypes);
@@ -273,8 +280,7 @@ void ChartAreaPanel::Initialize()
     if (xSelectionSupplier.is())
         xSelectionSupplier->addSelectionChangeListener(mxSelectionListener.get());
 
-    css::uno::Reference<css::frame::XToolbarController> xController = mpToolBoxColor->GetFirstController();
-    SvxColorToolBoxControl* pToolBoxColor = dynamic_cast<SvxColorToolBoxControl*>(xController.get());
+    SvxColorToolBoxControl* pToolBoxColor = getColorToolBoxControl(mpToolBoxColor.get());
     pToolBoxColor->setColorSelectFunction(maFillColorWrapper);
 
     updateData();
@@ -427,6 +433,8 @@ void ChartAreaPanel::updateData()
     xPropSet->getPropertyValue("FillTransparenceGradientName") >>= aFillFloatTransparenceName;
     XFillFloatTransparenceItem aFillFloatTransparenceItem = getXTransparencyGradientForName(mxModel, aFillFloatTransparenceName);
     updateFillFloatTransparence(false, true, &aFillFloatTransparenceItem);
+
+    maFillColorWrapper.updateData();
 }
 
 void ChartAreaPanel::modelInvalid()
diff --git a/chart2/source/controller/sidebar/ChartColorWrapper.cxx b/chart2/source/controller/sidebar/ChartColorWrapper.cxx
index d8a8f87..beeb85d 100644
--- a/chart2/source/controller/sidebar/ChartColorWrapper.cxx
+++ b/chart2/source/controller/sidebar/ChartColorWrapper.cxx
@@ -11,6 +11,10 @@
 
 #include "ChartController.hxx"
 
+#include <editeng/colritem.hxx>
+#include <svx/tbcontrl.hxx>
+#include <svx/svxids.hrc>
+
 namespace chart { namespace sidebar {
 
 namespace {
@@ -56,13 +60,15 @@ css::uno::Reference<css::beans::XPropertySet> getPropSet(
 }
 
 ChartColorWrapper::ChartColorWrapper(
-        css::uno::Reference<css::frame::XModel> xModel):
+        css::uno::Reference<css::frame::XModel> xModel,
+        SvxColorToolBoxControl* pControl):
     mxModel(xModel),
+    mpControl(pControl),
     maPropertyName("FillColor")
 {
 }
 
-void ChartColorWrapper::operator()(const OUString& rCommand, const Color& rColor)
+void ChartColorWrapper::operator()(const OUString& , const Color& rColor)
 {
     css::uno::Reference<css::beans::XPropertySet> xPropSet = getPropSet(mxModel);
     assert(xPropSet.is());
@@ -75,6 +81,21 @@ void ChartColorWrapper::updateModel(css::uno::Reference<css::frame::XModel> xMod
     mxModel = xModel;
 }
 
+void ChartColorWrapper::updateData()
+{
+    css::uno::Reference<css::beans::XPropertySet> xPropSet = getPropSet(mxModel);
+    if (!xPropSet.is())
+        return;
+
+    css::uno::Any aAny = xPropSet->getPropertyValue(maPropertyName);
+    sal_uInt32 nColor = 0;
+    aAny >>= nColor;
+    Color aColor(nColor);
+
+    SvxColorItem aItem(aColor, SID_ATTR_FILL_COLOR);
+    mpControl->StateChanged(SID_ATTR_FILL_COLOR, SfxItemState::SET, &aItem);
+}
+
 } }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/chart2/source/controller/sidebar/ChartColorWrapper.hxx b/chart2/source/controller/sidebar/ChartColorWrapper.hxx
index adbe9ef..b0f9071 100644
--- a/chart2/source/controller/sidebar/ChartColorWrapper.hxx
+++ b/chart2/source/controller/sidebar/ChartColorWrapper.hxx
@@ -13,6 +13,8 @@
 #include <com/sun/star/frame/XFramesSupplier.hpp>
 #include <tools/color.hxx>
 
+class SvxColorToolBoxControl;
+
 namespace chart { namespace sidebar {
 
 class ChartColorWrapper
@@ -20,17 +22,21 @@ class ChartColorWrapper
 private:
 
 public:
-    ChartColorWrapper(css::uno::Reference<css::frame::XModel> xModel);
+    ChartColorWrapper(css::uno::Reference<css::frame::XModel> xModel,
+            SvxColorToolBoxControl* pControl);
 
     void operator()(const OUString& rCommand, const Color& rColor);
 
     void updateModel(css::uno::Reference<css::frame::XModel> xModel);
+
+    void updateData();
+
 private:
 
-    // not the chart frame
-    // you need to get the chart frame through getActiveFrame
     css::uno::Reference<css::frame::XModel> mxModel;
 
+    SvxColorToolBoxControl* mpControl;
+
     OUString maPropertyName;
 };
 
commit cf7db8226240ca7f425cf649e164767988c80001
Author: Markus Mohrhard <markus.mohrhard at googlemail.com>
Date:   Wed Aug 19 23:43:39 2015 +0200

    handle new color picker correctly in chart sidebar
    
    This implements the basics for that and implements it in the AreaPanel.
    
    Change-Id: I6d9e5012bbcc2c953d478a09a839f35f2ef64c5b

diff --git a/chart2/Library_chartcontroller.mk b/chart2/Library_chartcontroller.mk
index e06edd1..9f8b65f 100644
--- a/chart2/Library_chartcontroller.mk
+++ b/chart2/Library_chartcontroller.mk
@@ -191,6 +191,7 @@ $(eval $(call gb_Library_add_exception_objects,chartcontroller,\
     chart2/source/controller/sidebar/Chart2PanelFactory \
     chart2/source/controller/sidebar/ChartAreaPanel \
     chart2/source/controller/sidebar/ChartAxisPanel \
+    chart2/source/controller/sidebar/ChartColorWrapper \
     chart2/source/controller/sidebar/ChartElementsPanel \
     chart2/source/controller/sidebar/ChartErrorBarPanel \
     chart2/source/controller/sidebar/ChartLinePanel \
diff --git a/chart2/source/controller/sidebar/ChartAreaPanel.cxx b/chart2/source/controller/sidebar/ChartAreaPanel.cxx
index 3230e7c..dbf9a68 100644
--- a/chart2/source/controller/sidebar/ChartAreaPanel.cxx
+++ b/chart2/source/controller/sidebar/ChartAreaPanel.cxx
@@ -19,6 +19,8 @@
 #include <svx/xflftrit.hxx>
 #include <svx/unomid.hxx>
 
+#include <svx/tbcontrl.hxx>
+
 namespace chart { namespace sidebar {
 
 namespace {
@@ -237,7 +239,8 @@ ChartAreaPanel::ChartAreaPanel(vcl::Window* pParent,
     mxListener(new ChartSidebarModifyListener(this)),
     mxSelectionListener(new ChartSidebarSelectionListener(this)),
     mbUpdate(true),
-    mbModelValid(true)
+    mbModelValid(true),
+    maFillColorWrapper(mxModel)
 {
     std::vector<ObjectType> aAcceptedTypes { OBJECTTYPE_PAGE, OBJECTTYPE_DIAGRAM, OBJECTTYPE_DATA_SERIES, OBJECTTYPE_TITLE, OBJECTTYPE_LEGEND};
     mxSelectionListener->setAcceptedTypes(aAcceptedTypes);
@@ -270,6 +273,10 @@ void ChartAreaPanel::Initialize()
     if (xSelectionSupplier.is())
         xSelectionSupplier->addSelectionChangeListener(mxSelectionListener.get());
 
+    css::uno::Reference<css::frame::XToolbarController> xController = mpToolBoxColor->GetFirstController();
+    SvxColorToolBoxControl* pToolBoxColor = dynamic_cast<SvxColorToolBoxControl*>(xController.get());
+    pToolBoxColor->setColorSelectFunction(maFillColorWrapper);
+
     updateData();
 }
 
diff --git a/chart2/source/controller/sidebar/ChartAreaPanel.hxx b/chart2/source/controller/sidebar/ChartAreaPanel.hxx
index 48df2ac..aa66339 100644
--- a/chart2/source/controller/sidebar/ChartAreaPanel.hxx
+++ b/chart2/source/controller/sidebar/ChartAreaPanel.hxx
@@ -30,6 +30,7 @@
 
 #include "ChartSidebarModifyListener.hxx"
 #include "ChartSidebarSelectionListener.hxx"
+#include "ChartColorWrapper.hxx"
 
 class XFillFloatTransparenceItem;
 class XFillTransparenceItem;
@@ -92,6 +93,8 @@ private:
 
     bool mbUpdate;
     bool mbModelValid;
+
+    ChartColorWrapper maFillColorWrapper;
 };
 
 } } // end of namespace svx::sidebar
diff --git a/chart2/source/controller/sidebar/ChartColorWrapper.cxx b/chart2/source/controller/sidebar/ChartColorWrapper.cxx
new file mode 100644
index 0000000..d8a8f87
--- /dev/null
+++ b/chart2/source/controller/sidebar/ChartColorWrapper.cxx
@@ -0,0 +1,80 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include "ChartColorWrapper.hxx"
+
+#include "ChartController.hxx"
+
+namespace chart { namespace sidebar {
+
+namespace {
+
+OUString getCID(css::uno::Reference<css::frame::XModel> xModel)
+{
+    css::uno::Reference<css::frame::XController> xController(xModel->getCurrentController());
+    css::uno::Reference<css::view::XSelectionSupplier> xSelectionSupplier(xController, css::uno::UNO_QUERY);
+    if (!xSelectionSupplier.is())
+        return OUString();
+
+    css::uno::Any aAny = xSelectionSupplier->getSelection();
+    if (!aAny.hasValue())
+        return OUString();
+
+    OUString aCID;
+    aAny >>= aCID;
+
+    return aCID;
+}
+
+css::uno::Reference<css::beans::XPropertySet> getPropSet(
+        css::uno::Reference<css::frame::XModel> xModel)
+{
+    OUString aCID = getCID(xModel);
+    css::uno::Reference<css::beans::XPropertySet> xPropSet =
+        ObjectIdentifier::getObjectPropertySet(aCID, xModel);
+
+    ObjectType eType = ObjectIdentifier::getObjectType(aCID);
+    if (eType == OBJECTTYPE_DIAGRAM)
+    {
+        css::uno::Reference<css::chart2::XDiagram> xDiagram(
+                xPropSet, css::uno::UNO_QUERY);
+        if (!xDiagram.is())
+            return xPropSet;
+
+        xPropSet.set(xDiagram->getWall());
+    }
+
+    return xPropSet;
+}
+
+}
+
+ChartColorWrapper::ChartColorWrapper(
+        css::uno::Reference<css::frame::XModel> xModel):
+    mxModel(xModel),
+    maPropertyName("FillColor")
+{
+}
+
+void ChartColorWrapper::operator()(const OUString& rCommand, const Color& rColor)
+{
+    css::uno::Reference<css::beans::XPropertySet> xPropSet = getPropSet(mxModel);
+    assert(xPropSet.is());
+
+    xPropSet->setPropertyValue(maPropertyName, css::uno::makeAny(rColor.GetColor()));
+}
+
+void ChartColorWrapper::updateModel(css::uno::Reference<css::frame::XModel> xModel)
+{
+    mxModel = xModel;
+}
+
+} }
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/chart2/source/controller/sidebar/ChartColorWrapper.hxx b/chart2/source/controller/sidebar/ChartColorWrapper.hxx
new file mode 100644
index 0000000..adbe9ef
--- /dev/null
+++ b/chart2/source/controller/sidebar/ChartColorWrapper.hxx
@@ -0,0 +1,41 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#ifndef INCLUDED_CHART2_SOURCE_CONTROLLER_SIDEBAR_CHARTCOLORWRAPPER_HXX
+#define INCLUDED_CHART2_SOURCE_CONTROLLER_SIDEBAR_CHARTCOLORWRAPPER_HXX
+
+#include <com/sun/star/frame/XFramesSupplier.hpp>
+#include <tools/color.hxx>
+
+namespace chart { namespace sidebar {
+
+class ChartColorWrapper
+{
+private:
+
+public:
+    ChartColorWrapper(css::uno::Reference<css::frame::XModel> xModel);
+
+    void operator()(const OUString& rCommand, const Color& rColor);
+
+    void updateModel(css::uno::Reference<css::frame::XModel> xModel);
+private:
+
+    // not the chart frame
+    // you need to get the chart frame through getActiveFrame
+    css::uno::Reference<css::frame::XModel> mxModel;
+
+    OUString maPropertyName;
+};
+
+} }
+
+#endif
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/include/sfx2/sidebar/SidebarToolBox.hxx b/include/sfx2/sidebar/SidebarToolBox.hxx
index a395969..574603b2 100644
--- a/include/sfx2/sidebar/SidebarToolBox.hxx
+++ b/include/sfx2/sidebar/SidebarToolBox.hxx
@@ -61,6 +61,8 @@ public:
                        const css::uno::Reference<css::frame::XToolbarController>& rxController,
                        const OUString& rsCommandName);
 
+    css::uno::Reference<css::frame::XToolbarController> GetFirstController();
+
 private:
     Image maItemSeparator;
 
diff --git a/include/svx/PaletteManager.hxx b/include/svx/PaletteManager.hxx
index 7ad0cdb..561dcac 100644
--- a/include/svx/PaletteManager.hxx
+++ b/include/svx/PaletteManager.hxx
@@ -50,6 +50,8 @@ class PaletteManager
     std::deque<Color>       maRecentColors;
     std::vector<std::unique_ptr<Palette>> m_Palettes;
 
+    std::function<void(const OUString&, const Color&)> maColorSelectFunction;
+
 public:
     PaletteManager();
     ~PaletteManager();
@@ -69,6 +71,9 @@ public:
 
     void        SetBtnUpdater(svx::ToolboxButtonColorUpdater* pBtnUpdater);
     void        PopupColorPicker(const OUString& aCommand);
+
+    void        SetColorSelectFunction(std::function<void(const OUString&, const Color&)> aColorSelectFunction);
+
     static void DispatchColorCommand(const OUString& aCommand, const Color& rColor);
 };
 
diff --git a/include/svx/sidebar/AreaPropertyPanelBase.hxx b/include/svx/sidebar/AreaPropertyPanelBase.hxx
index 9fdbba1..24ebced 100644
--- a/include/svx/sidebar/AreaPropertyPanelBase.hxx
+++ b/include/svx/sidebar/AreaPropertyPanelBase.hxx
@@ -23,6 +23,7 @@
 #include <vcl/ctrl.hxx>
 #include <sfx2/sidebar/SidebarPanelBase.hxx>
 #include <sfx2/sidebar/ControllerItem.hxx>
+#include <sfx2/sidebar/SidebarToolBox.hxx>
 #include <svx/xgrad.hxx>
 #include <svx/itemwin.hxx>
 #include <svx/xfillit0.hxx>
@@ -124,7 +125,7 @@ protected:
     VclPtr<FixedText>                                          mpColorTextFT;
     VclPtr<SvxFillTypeBox>                                     mpLbFillType;
     VclPtr<SvxFillAttrBox>                                     mpLbFillAttr;
-    VclPtr<ToolBox>                                            mpToolBoxColor; // for new color picker
+    VclPtr<sfx2::sidebar::SidebarToolBox>                                            mpToolBoxColor; // for new color picker
     VclPtr<FixedText>                                          mpTrspTextFT;
     VclPtr<ListBox>                                            mpLBTransType;
     VclPtr<MetricField>                                        mpMTRTransparent;
diff --git a/include/svx/tbcontrl.hxx b/include/svx/tbcontrl.hxx
index 9e589ac..584b99e 100644
--- a/include/svx/tbcontrl.hxx
+++ b/include/svx/tbcontrl.hxx
@@ -225,6 +225,7 @@ public:
     Color GetColor();
 };
 
+typedef std::function<void(const OUString&, const Color&)> ColorSelectFunction;
 class SVX_DLLPUBLIC SvxColorToolBoxControl : public SfxToolBoxControl
 {
     using SfxToolBoxControl::StateChanged;
@@ -233,6 +234,7 @@ class SVX_DLLPUBLIC SvxColorToolBoxControl : public SfxToolBoxControl
     PaletteManager mPaletteManager;
     BorderColorStatus maBorderColorStatus;
     bool bSidebarType;
+    ColorSelectFunction maColorSelectFunction;
     DECL_LINK(SelectedHdl, Color*);
 public:
     SFX_DECL_TOOLBOX_CONTROL();
@@ -247,6 +249,8 @@ public:
     // XSubToolbarController
     virtual sal_Bool SAL_CALL opensSubToolbar() throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE;
     virtual void SAL_CALL updateImage() throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE;
+
+    void setColorSelectFunction(ColorSelectFunction aColorSelectFunction);
 };
 
 class SVX_DLLPUBLIC SvxFrameToolBoxControl : public SfxToolBoxControl
diff --git a/sfx2/source/sidebar/SidebarToolBox.cxx b/sfx2/source/sidebar/SidebarToolBox.cxx
index ce02e9e..542088a 100644
--- a/sfx2/source/sidebar/SidebarToolBox.cxx
+++ b/sfx2/source/sidebar/SidebarToolBox.cxx
@@ -205,6 +205,14 @@ sal_uInt16 SidebarToolBox::GetItemIdForSubToolbarName (const OUString& rsSubTool
     return 0;
 }
 
+css::uno::Reference<css::frame::XToolbarController> SidebarToolBox::GetFirstController()
+{
+    if (maControllers.empty())
+        return css::uno::Reference<css::frame::XToolbarController>();
+
+    return maControllers.begin()->second.mxController;
+}
+
 void SidebarToolBox::RegisterHandlers()
 {
     if ( ! mbAreHandlersRegistered)
diff --git a/svx/source/tbxctrls/PaletteManager.cxx b/svx/source/tbxctrls/PaletteManager.cxx
index 658348f..8f0ff65 100644
--- a/svx/source/tbxctrls/PaletteManager.cxx
+++ b/svx/source/tbxctrls/PaletteManager.cxx
@@ -36,7 +36,8 @@ PaletteManager::PaletteManager() :
     mnCurrentPalette(0),
     mnColorCount(0),
     mpBtnUpdater(NULL),
-    mLastColor(COL_AUTO)
+    mLastColor(COL_AUTO),
+    maColorSelectFunction(PaletteManager::DispatchColorCommand)
 {
     LoadPalettes();
     mnNumOfPalettes += m_Palettes.size();
@@ -217,6 +218,11 @@ void PaletteManager::SetBtnUpdater(svx::ToolboxButtonColorUpdater* pBtnUpdater)
     mpBtnUpdater = pBtnUpdater;
 }
 
+void PaletteManager::SetColorSelectFunction(std::function<void(const OUString&, const Color&)> aColorSelectFunction)
+{
+    maColorSelectFunction = aColorSelectFunction;
+}
+
 void PaletteManager::PopupColorPicker(const OUString& aCommand)
 {
     // The calling object goes away during aColorDlg.Execute(), so we must copy this
@@ -230,7 +236,7 @@ void PaletteManager::PopupColorPicker(const OUString& aCommand)
             mpBtnUpdater->Update( aColorDlg.GetColor() );
         mLastColor = aColorDlg.GetColor();
         AddRecentColor( mLastColor );
-        DispatchColorCommand(aCommandCopy, mLastColor);
+        maColorSelectFunction(aCommandCopy, mLastColor);
     }
 }
 
diff --git a/svx/source/tbxctrls/colorwindow.hxx b/svx/source/tbxctrls/colorwindow.hxx
index 97cccc5..9cb27fd 100644
--- a/svx/source/tbxctrls/colorwindow.hxx
+++ b/svx/source/tbxctrls/colorwindow.hxx
@@ -29,6 +29,8 @@
 #include <svx/PaletteManager.hxx>
 #include <vcl/lstbox.hxx>
 
+#include <functional>
+
 class BorderColorStatus;
 
 class SvxColorWindow_Impl : public SfxPopupWindow
@@ -50,6 +52,8 @@ private:
     PaletteManager&     mrPaletteManager;
     BorderColorStatus&  mrBorderColorStatus;
 
+    std::function<void(const OUString&, const Color&)> maColorSelectFunction;
+
     DECL_LINK( SelectHdl, SvxColorValueSet* );
     DECL_LINK( SelectPaletteHdl, void *);
     DECL_LINK( AutoColorClickHdl, void * );
@@ -66,7 +70,8 @@ public:
                          sal_uInt16 nSlotId,
                          const ::com::sun::star::uno::Reference< ::com::sun::star::frame::XFrame >& rFrame,
                          const OUString& rWndTitle,
-                         vcl::Window* pParentWindow);
+                         vcl::Window* pParentWindow,
+                         std::function<void(const OUString&, const Color&)> maColorSelectFunction);
     virtual ~SvxColorWindow_Impl();
     virtual void        dispose() SAL_OVERRIDE;
     void                StartSelection();
diff --git a/svx/source/tbxctrls/tbcontrl.cxx b/svx/source/tbxctrls/tbcontrl.cxx
index bc0912f..528b4ea 100644
--- a/svx/source/tbxctrls/tbcontrl.cxx
+++ b/svx/source/tbxctrls/tbcontrl.cxx
@@ -1215,7 +1215,8 @@ SvxColorWindow_Impl::SvxColorWindow_Impl( const OUString&            rCommand,
                                           sal_uInt16                 nSlotId,
                                           const Reference< XFrame >& rFrame,
                                           const OUString&            rWndTitle,
-                                          vcl::Window*                    pParentWindow ):
+                                          vcl::Window*                    pParentWindow,
+                                          std::function<void(const OUString&, const Color&)> aFunction):
 
     SfxPopupWindow( nSlotId, pParentWindow,
                     "palette_popup_window", "svx/ui/colorwindow.ui",
@@ -1223,7 +1224,8 @@ SvxColorWindow_Impl::SvxColorWindow_Impl( const OUString&            rCommand,
     theSlotId( nSlotId ),
     maCommand( rCommand ),
     mrPaletteManager( rPaletteManager ),
-    mrBorderColorStatus( rBorderColorStatus )
+    mrBorderColorStatus( rBorderColorStatus ),
+    maColorSelectFunction(aFunction)
 {
     get(mpPaletteListBox,     "palette_listbox");
     get(mpButtonAutoColor,    "auto_color_button");
@@ -1365,7 +1367,7 @@ IMPL_LINK(SvxColorWindow_Impl, SelectHdl, SvxColorValueSet*, pColorSet)
     if ( maSelectedLink.IsSet() )
         maSelectedLink.Call(&aColor);
 
-    PaletteManager::DispatchColorCommand(maCommand, aColor);
+    maColorSelectFunction(maCommand, aColor);
     return 0;
 }
 
@@ -1407,7 +1409,7 @@ IMPL_LINK_NOARG(SvxColorWindow_Impl, AutoColorClickHdl)
     if ( maSelectedLink.IsSet() )
         maSelectedLink.Call(&aColor);
 
-    PaletteManager::DispatchColorCommand(maCommand, aColor);
+    maColorSelectFunction(maCommand, aColor);
     return 0;
 }
 
@@ -2537,8 +2539,9 @@ VclPtr<vcl::Window> SvxFontNameToolBoxControl::CreateItemWindow( vcl::Window *pP
 SvxColorToolBoxControl::SvxColorToolBoxControl(
     sal_uInt16 nSlotId,
     sal_uInt16 nId,
-    ToolBox& rTbx ) :
-    SfxToolBoxControl( nSlotId, nId, rTbx )
+    ToolBox& rTbx ):
+    SfxToolBoxControl( nSlotId, nId, rTbx ),
+    maColorSelectFunction(PaletteManager::DispatchColorCommand)
 {
     if ( dynamic_cast< sfx2::sidebar::SidebarToolBox* >(&rTbx) )
         bSidebarType = true;
@@ -2611,6 +2614,12 @@ SvxColorToolBoxControl::~SvxColorToolBoxControl()
 {
 }
 
+void SvxColorToolBoxControl::setColorSelectFunction(ColorSelectFunction aColorSelectFunction)
+{
+    maColorSelectFunction = aColorSelectFunction;
+    mPaletteManager.SetColorSelectFunction(aColorSelectFunction);
+}
+
 VclPtr<SfxPopupWindow> SvxColorToolBoxControl::CreatePopupWindow()
 {
     SvxColorWindow_Impl* pColorWin =
@@ -2622,7 +2631,8 @@ VclPtr<SfxPopupWindow> SvxColorToolBoxControl::CreatePopupWindow()
                             GetSlotId(),
                             m_xFrame,
                             SVX_RESSTR( RID_SVXITEMS_EXTRAS_CHARCOLOR ),
-                            &GetToolBox() );
+                            &GetToolBox(),
+                            maColorSelectFunction);
 
     switch( GetSlotId() )
     {


More information about the Libreoffice-commits mailing list