haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Enable/Disable the apply/discard buttons 

When there are no pending actions, we should not show the user the
ability to apply/discard the pending actions.  This uses wxUpdateUI in
the correct manner, limiting it to only the elements being updated and
using the event actions to ensure we don't have a cascade

Fixes https://gitlab.com/kicad/code/kicad/issues/10761

(cherry picked from commit 09a2d50e09)
This commit is contained in:
Seth Hillbrand 2022-05-11 13:47:50 -07:00
parent 40e0d06381
commit dedbcac3ae
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
kicad/pcm/dialogs/dialog_pcm.cpp
View File

@ -131,7 +131,8 @@ DIALOG_PCM::DIALOG_PCM( wxWindow* parent ) : DIALOG_PCM_BASE( parent )
{ wxID_CANCEL, _( "Discard Pending Changes" ) } } ); { wxID_CANCEL, _( "Discard Pending Changes" ) } } );
Bind( wxEVT_CLOSE_WINDOW, &DIALOG_PCM::OnCloseWindow, this ); Bind( wxEVT_CLOSE_WINDOW, &DIALOG_PCM::OnCloseWindow, this );
m_sdbSizer1Cancel->Bind( wxEVT_UPDATE_UI, &DIALOG_PCM::OnUpdateEventButtons, this );
m_sdbSizer1Apply->Bind( wxEVT_UPDATE_UI, &DIALOG_PCM::OnUpdateEventButtons, this );
SETTINGS_MANAGER& mgr = Pgm().GetSettingsManager(); SETTINGS_MANAGER& mgr = Pgm().GetSettingsManager();
KICAD_SETTINGS* app_settings = mgr.GetAppSettings<KICAD_SETTINGS>(); KICAD_SETTINGS* app_settings = mgr.GetAppSettings<KICAD_SETTINGS>();
@ -160,6 +161,12 @@ DIALOG_PCM::~DIALOG_PCM()
} }
void DIALOG_PCM::OnUpdateEventButtons( wxUpdateUIEvent& event )
{
event.Enable( !m_pendingActions.empty() );
}
void DIALOG_PCM::OnCloseClicked( wxCommandEvent& event ) void DIALOG_PCM::OnCloseClicked( wxCommandEvent& event )
{ {
if( m_pendingActions.size() == 0 if( m_pendingActions.size() == 0

3
kicad/pcm/dialogs/dialog_pcm.h
View File

@ -67,6 +67,9 @@ public:
///< Discards selected pending actions ///< Discards selected pending actions
void OnDiscardActionClicked( wxCommandEvent& event ) override; void OnDiscardActionClicked( wxCommandEvent& event ) override;
///< Handles modification of the buttons' status
void OnUpdateEventButtons( wxUpdateUIEvent& event );
private: private:
/** /**
* @brief Gets package data from PCM and displays it on repository tab * @brief Gets package data from PCM and displays it on repository tab