Changeset 118

Show
Ignore:
Timestamp:
08/15/10 22:24:46 (18 months ago)
Author:
Carsten
Message:

CaWE: The tools were in a single "radio group" in the menu and in several radio groups in the toolbar, and their synchronization code was flawed as described at < http://thread.gmane.org/gmane.comp.lib.wxwidgets.general/70613>. These changes fix the problem and close #35.

Location:
cafu/trunk/CaWE
Files:
13 modified

Legend:

Unmodified
Added
Removed

  • cafu/trunk/CaWE/ChildFrame.cpp

    r108 r118  
    459459    wxToolBar* ToolbarTools=new wxToolBar(this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxTB_FLAT | wxTB_NODIVIDER); 
    460460    ToolbarTools->SetToolBitmapSize(wxSize(21, 18)); 
     461 
     462    // Note that we cannot have separators between these tools, because they must all be in the same radio group. 
    461463    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_SELECTION,             "Selection",               wxBitmap("CaWE/res/Tool_Selection.png", wxBITMAP_TYPE_PNG),      wxNullBitmap, "Selection",             "Selection" ); 
    462464    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_CAMERA,                "Camera",                  wxBitmap("CaWE/res/Tool_Camera.png", wxBITMAP_TYPE_PNG),         wxNullBitmap, "Camera"                 ); 
    463     ToolbarTools->AddSeparator(); 
    464465    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_NEWBRUSH,              "New Brush",               wxBitmap("CaWE/res/Tool_NewBrush.png", wxBITMAP_TYPE_PNG),       wxNullBitmap, "New Brush"              ); 
    465466    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_NEWENTITY,             "New Entity",              wxBitmap("CaWE/res/Tool_NewEntity.png", wxBITMAP_TYPE_PNG),      wxNullBitmap, "New Entity"             ); 
    … …  
    468469    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_NEWLIGHT,              "New Light",               wxBitmap("CaWE/res/Tool_NewLight.png", wxBITMAP_TYPE_PNG),       wxNullBitmap, "New Light"              ); 
    469470    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_NEWDECAL,              "New Decal",               wxBitmap("CaWE/res/Tool_NewDecal.png", wxBITMAP_TYPE_PNG),       wxNullBitmap, "New Decal"              ); 
    470     ToolbarTools->AddSeparator(); 
    471471    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_EDITSURFACEPROPERTIES, "Edit Surface Properties", wxBitmap("CaWE/res/Tool_EditSurfProps.png", wxBITMAP_TYPE_PNG),  wxNullBitmap, "Edit Surface Properties"); 
    472472    ToolbarTools->AddRadioTool(ID_MENU_TOOLS_TOOL_TERRAINEDITOR,         "Edit Terrain",            wxBitmap("CaWE/res/Tool_EditTerrain.png", wxBITMAP_TYPE_PNG),    wxNullBitmap, "Edit Terrain"           ); 
    … …  
    11341134void ChildFrameT::OnMenuTools(wxCommandEvent& CE) 
    11351135{ 
    1136     switch (CE.GetId()) 
    1137     { 
    1138         case 0: 
    1139         default: 
    1140         { 
    1141             // Find the tool whose wxWidgets event ID matches CE.GetId(). 
    1142             const ArrayT<ToolT*>& Tools=m_ToolManager->GetTools(); 
    1143  
    1144             for (unsigned long ToolNr=0; ToolNr<Tools.Size(); ToolNr++) 
    1145             { 
    1146                 ToolT* Tool=Tools[ToolNr]; 
    1147  
    1148                 if (Tool->GetWxEventID()==CE.GetId()) 
    1149                 { 
    1150                     m_ToolManager->SetActiveTool(Tool->GetType()); 
    1151                     break; 
    1152                 } 
    1153             } 
     1136    // Find the tool whose wxWidgets event ID matches CE.GetId(). 
     1137    const ArrayT<ToolT*>& Tools=m_ToolManager->GetTools(); 
     1138 
     1139    for (unsigned long ToolNr=0; ToolNr<Tools.Size(); ToolNr++) 
     1140    { 
     1141        ToolT* Tool=Tools[ToolNr]; 
     1142 
     1143        if (Tool->GetWxEventID()==CE.GetId()) 
     1144        { 
     1145            m_ToolManager->SetActiveTool(Tool->GetType()); 
     1146            break; 
    11541147        } 
    11551148    } 
    … …  
    11591152void ChildFrameT::OnMenuToolsUpdate(wxUpdateUIEvent& UE) 
    11601153{ 
    1161     switch (UE.GetId()) 
    1162     { 
    1163         case 0: 
    1164         default: 
    1165         { 
    1166             // Find the tool whose wxWidgets event ID matches UE.GetId(). 
    1167             const ArrayT<ToolT*>& Tools=m_ToolManager->GetTools(); 
    1168  
    1169             for (unsigned long ToolNr=0; ToolNr<Tools.Size(); ToolNr++) 
    1170             { 
    1171                 ToolT* Tool=Tools[ToolNr]; 
    1172  
    1173                 if (Tool->GetWxEventID()==UE.GetId()) 
    1174                 { 
    1175                     UE.Check(Tool->IsActiveTool()); 
    1176                     break; 
    1177                 } 
    1178             } 
    1179         } 
    1180     } 
     1154    const ToolT* ActiveTool=m_ToolManager->GetActiveTool(); 
     1155 
     1156    if (ActiveTool && ActiveTool->GetWxEventID()==UE.GetId()) 
     1157        UE.Check(true); 
    11811158} 
    11821159 

  • cafu/trunk/CaWE/Tool.hpp

    r36 r118  
    7777 
    7878    /// Returns the ID of the wxWidgets event (menu selection or toolbar button click) that is associated with activating this tool. 
    79     virtual int GetWxEventID()=0; 
     79    virtual int GetWxEventID() const=0; 
    8080 
    8181    /// Returns the options bar window associated with this tool. NULL if no options bar has been assigned. 

  • cafu/trunk/CaWE/ToolCamera.hpp

    r36 r118  
    5050 
    5151    // Implementations/overrides of ToolT methods. 
    52     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_CAMERA; } 
     52    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_CAMERA; } 
    5353    wxWindow* GetOptionsBar(); 
    5454 

  • cafu/trunk/CaWE/ToolClip.hpp

    r36 r118  
    5050 
    5151    // Implementations/overrides of ToolT methods. 
    52     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_CLIP; } 
     52    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_CLIP; } 
    5353    wxWindow* GetOptionsBar(); 
    5454    void      OnActivate(ToolT* OldTool); 

  • cafu/trunk/CaWE/ToolEditSurface.hpp

    r36 r118  
    3939 
    4040    // Implementations/overrides of ToolT methods. 
    41     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_EDITSURFACEPROPERTIES; } 
     41    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_EDITSURFACEPROPERTIES; } 
    4242    wxWindow* GetOptionsBar(); 
    4343    void      OnActivate(ToolT* OldTool); 

  • cafu/trunk/CaWE/ToolMorph.hpp

    r36 r118  
    6161 
    6262    // Implementations/overrides of ToolT methods. 
    63     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_EDITVERTICES; } 
     63    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_EDITVERTICES; } 
    6464    wxWindow* GetOptionsBar(); 
    6565    void      OnActivate(ToolT* OldTool); 

  • cafu/trunk/CaWE/ToolNewBezierPatch.hpp

    r36 r118  
    4444 
    4545    // Implementations/overrides of ToolT methods. 
    46     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWBEZIERPATCH; } 
     46    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWBEZIERPATCH; } 
    4747    wxWindow* GetOptionsBar(); 
    4848 

  • cafu/trunk/CaWE/ToolNewBrush.hpp

    r36 r118  
    4343 
    4444    // Implementations/overrides of ToolT methods. 
    45     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWBRUSH; } 
     45    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWBRUSH; } 
    4646    wxWindow* GetOptionsBar(); 
    4747 

  • cafu/trunk/CaWE/ToolNewEntity.hpp

    r36 r118  
    4343 
    4444    // Implementations/overrides of ToolT methods. 
    45     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWENTITY; } 
     45    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWENTITY; } 
    4646    wxWindow* GetOptionsBar(); 
    4747 

  • cafu/trunk/CaWE/ToolNewLight.hpp

    r36 r118  
    4040 
    4141    // Implementations/overrides of ToolT methods. 
    42     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWLIGHT; } 
     42    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWLIGHT; } 
    4343    wxWindow* GetOptionsBar(); 
    4444 

  • cafu/trunk/CaWE/ToolNewTerrain.hpp

    r36 r118  
    4343 
    4444    // Implementations/overrides of ToolT methods. 
    45     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWTERRAIN; } 
     45    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_NEWTERRAIN; } 
    4646    wxWindow* GetOptionsBar(); 
    4747 

  • cafu/trunk/CaWE/ToolSelection.hpp

    r67 r118  
    5959 
    6060    // Implementations/overrides of ToolT methods. 
    61     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_SELECTION; } 
     61    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_SELECTION; } 
    6262    wxWindow* GetOptionsBar(); 
    6363    void      OnActivate(ToolT* OldTool); 

  • cafu/trunk/CaWE/ToolTerrainEdit.hpp

    r36 r118  
    111111 
    112112    // Implementations/overrides of ToolT methods. 
    113     int       GetWxEventID() { return ChildFrameT::ID_MENU_TOOLS_TOOL_TERRAINEDITOR; } 
     113    int       GetWxEventID() const { return ChildFrameT::ID_MENU_TOOLS_TOOL_TERRAINEDITOR; } 
    114114    wxWindow* GetOptionsBar(); 
    115115    void      OnActivate(ToolT* OldTool);