As mentioned before, in communication with Ivan and Astron, I have
attached a fix for the split button for cell background colour. It is now
possible again to set the background colour to transparent.
Also I fixed an inconsistency for calc font colour, which does not 
influence the behaviour as far I can see, but which is now consistent
with the other split buttons.

On to the next button to be split in two :-)

Winfried
From b7cf81b7aee3aa297ef29c033b20a492eb6c8d5e Mon Sep 17 00:00:00 2001
From: Winfried Donkers <o...@dci-electronics.nl>
Date: Wed, 7 Mar 2012 17:39:03 +0100
Subject: [PATCH] fdo#45671 fix for transparent colour split button

---
 sc/source/ui/view/formatsh.cxx   |    9 +++++++++
 svx/source/tbxctrls/tbcontrl.cxx |    4 ++++
 2 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/sc/source/ui/view/formatsh.cxx b/sc/source/ui/view/formatsh.cxx
index 61522d6..dc75138 100644
--- a/sc/source/ui/view/formatsh.cxx
+++ b/sc/source/ui/view/formatsh.cxx
@@ -1435,6 +1435,15 @@ void ScFormatShell::ExecuteAttr( SfxRequest& rReq )
                 pTabViewShell->ExecuteCellFormatDlg( rReq, TP_FONT );       // wenn ToolBar vertikal
                 break;
 
+            case SID_BACKGROUND_COLOR:
+                {
+                    SvxBrushItem aBrushItem( (const SvxBrushItem&)
+                                     pTabViewShell->GetSelectionPattern()->GetItem( ATTR_BACKGROUND ) );
+                    aBrushItem.SetColor( COL_TRANSPARENT );
+                    pTabViewShell->ApplyAttr( aBrushItem );
+                }
+                break;
+
             case SID_ATTR_ALIGN_LINEBREAK:                  // ohne Parameter als Toggle
                 {
                     const ScPatternAttr* pAttrs = pTabViewShell->GetSelectionPattern();
diff --git a/svx/source/tbxctrls/tbcontrl.cxx b/svx/source/tbxctrls/tbcontrl.cxx
index 2a20d4a..cdc447f 100644
--- a/svx/source/tbxctrls/tbcontrl.cxx
+++ b/svx/source/tbxctrls/tbcontrl.cxx
@@ -2250,6 +2250,10 @@ SvxColorExtToolBoxControl::SvxColorExtToolBoxControl(
     switch( nSlotId )
     {
         case SID_ATTR_CHAR_COLOR:
+            addStatusListener( OUString( RTL_CONSTASCII_USTRINGPARAM( ".uno:Color" )));
+            nMode = TBX_UPDATER_MODE_CHAR_COLOR_NEW;
+            break;
+
         case SID_ATTR_CHAR_COLOR2:
             addStatusListener( OUString( RTL_CONSTASCII_USTRINGPARAM( ".uno:CharColorExt" )));
             nMode = TBX_UPDATER_MODE_CHAR_COLOR_NEW;
-- 
1.7.7

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to