Activate the footprint wizard correctly

Starting the footprint wizard should cause the actual wizard to activate
This commit is contained in:
Seth Hillbrand 2018-12-10 05:43:44 -07:00
parent 4b3bfb0115
commit 48aad33314
2 changed files with 9 additions and 2 deletions

View File

@ -122,7 +122,7 @@ FOOTPRINT_WIZARD_FRAME::FOOTPRINT_WIZARD_FRAME( KIWAY* aKiway,
wxDefaultPosition, wxDefaultSize, wxDefaultPosition, wxDefaultSize,
aParent ? KICAD_DEFAULT_DRAWFRAME_STYLE | MODAL_MODE_EXTRASTYLE aParent ? KICAD_DEFAULT_DRAWFRAME_STYLE | MODAL_MODE_EXTRASTYLE
: KICAD_DEFAULT_DRAWFRAME_STYLE | wxSTAY_ON_TOP, : KICAD_DEFAULT_DRAWFRAME_STYLE | wxSTAY_ON_TOP,
FOOTPRINT_WIZARD_FRAME_NAME ) FOOTPRINT_WIZARD_FRAME_NAME ), m_wizardListShown( false )
{ {
wxASSERT( aFrameType == FRAME_PCB_FOOTPRINT_WIZARD ); wxASSERT( aFrameType == FRAME_PCB_FOOTPRINT_WIZARD );
@ -271,7 +271,7 @@ FOOTPRINT_WIZARD_FRAME::FOOTPRINT_WIZARD_FRAME( KIWAY* aKiway,
// It means the call to FOOTPRINT_WIZARD_FRAME::ShowModal will change the // It means the call to FOOTPRINT_WIZARD_FRAME::ShowModal will change the
// Event Loop Manager, and stop the one created by the dialog. // Event Loop Manager, and stop the one created by the dialog.
// It does not happen on all W.M., perhaps due to the way the order events are called // It does not happen on all W.M., perhaps due to the way the order events are called
// SelectFootprintWizard(); // See the call in onActivate instead
} }
@ -597,6 +597,11 @@ void FOOTPRINT_WIZARD_FRAME::OnActivate( wxActivateEvent& event )
if( !event.GetActive() ) if( !event.GetActive() )
return; return;
if( !m_wizardListShown )
{
m_wizardListShown = true;
SelectFootprintWizard();
}
#if 0 #if 0
// Currently, we do not have a way to see if a Python wizard has changed, // Currently, we do not have a way to see if a Python wizard has changed,
// therefore the lists of parameters and option has to be rebuilt // therefore the lists of parameters and option has to be rebuilt

View File

@ -63,6 +63,8 @@ private:
wxString m_auiPerspective; ///< Encoded string describing the AUI layout wxString m_auiPerspective; ///< Encoded string describing the AUI layout
bool m_wizardListShown; ///< A show-once flag for the wizard list
protected: protected:
wxString m_wizardName; ///< name of the current wizard wxString m_wizardName; ///< name of the current wizard
wxString m_wizardDescription; ///< description of the wizard wxString m_wizardDescription; ///< description of the wizard