X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=packages%2Fwx%2Fsrc%2FbbwxColourSelectorButton.cxx;h=f61de883dbe95be403088ca616044c4ae026deda;hb=c429a5e1229f3dbc56e2d0100980c7e4a3506d89;hp=9a4cafd2bc5a913cc45d76f719222763db40a42c;hpb=1e2df7fb6ae571ce1cf72e6af1715f0e34675cbf;p=bbtk.git

diff --git a/packages/wx/src/bbwxColourSelectorButton.cxx b/packages/wx/src/bbwxColourSelectorButton.cxx
index 9a4cafd..f61de88 100644
--- a/packages/wx/src/bbwxColourSelectorButton.cxx
+++ b/packages/wx/src/bbwxColourSelectorButton.cxx
@@ -2,8 +2,8 @@
   Program:   bbtk
   Module:    $RCSfile: bbwxColourSelectorButton.cxx,v $
   Language:  C++
-  Date:      $Date: 2008/11/25 11:17:25 $
-  Version:   $Revision: 1.7 $
+  Date:      $Date: 2008/12/12 08:55:23 $
+  Version:   $Revision: 1.9 $
 =========================================================================*/
 
 /* ---------------------------------------------------------------------
@@ -42,9 +42,29 @@
 #include "bbwxColourSelectorButton.h"
 #include "bbwxPackage.h"
 
+#include <wx/clrpicker.h>
 
 namespace bbwx
 {
+//-------------------------------------------------------------------------
+class wxColourPickerCtrlWidget :public wxPanel
+{
+
+public:
+   wxColourPickerCtrlWidget( ColourSelectorButton* box,
+                             wxWindow *parent,
+                             unsigned char  cr,
+                             unsigned char  cg,
+                             unsigned char  cb  );
+
+   ~wxColourPickerCtrlWidget();
+   void OnColorChange(wxColourPickerEvent& e);
+   void UpdateBox();
+private:
+   wxColourPickerCtrl *picker;
+   ColourSelectorButton *mBox;
+};
+//-------------------------------------------------------------------------
 
 //-------------------------------------------------------------------------
 wxColourPickerCtrlWidget::wxColourPickerCtrlWidget(	ColourSelectorButton* box,
@@ -118,14 +138,18 @@ void wxColourPickerCtrlWidget::UpdateBox()
   
   void ColourSelectorButton::bbUserConstructor() 
   { 
+  // Default input value
     bbSetInputIn("1 1 1");
+  // Initial output value = the same than input
     bbSetOutputOut("1 1 1");
+    // IMPORTANT : Initialize output widget to null pointer to test it after 
     bbSetOutputWidget(0);
   }
 
 
   void ColourSelectorButton::Process() 
   { 
+  // The widget may not be created : have to test it before using it
     wxColourPickerCtrlWidget* w = ( wxColourPickerCtrlWidget* )bbGetOutputWidget();
     if (w) 
       {
@@ -133,6 +157,8 @@ void wxColourPickerCtrlWidget::UpdateBox()
       }
     else 
       {
+      // If widget does not exist yet you have to update the output value according 
+      // to the input (which may have changed if the user set it)
 	bbSetOutputOut(bbGetInputIn());
       }
   }
@@ -154,8 +180,8 @@ void wxColourPickerCtrlWidget::UpdateBox()
     cb = (unsigned char)(255.*b);
 
     wxColourPickerCtrlWidget* w = new wxColourPickerCtrlWidget(this, //bbGetWxParent() , 
-												parent,
-												cr , cg , cb );    
+								parent,
+								cr , cg , cb );    
     
     bbSetOutputWidget( w );
   }