toolbars_lib_view.cpp and toolbars_libedit.cpp: do not
recreate comboboxes when clearing toolbars. Clearing toolbars delete only
tools, not other widgets. recreating comboboxes creates strange cosmetic issues.
From master, commit 598ed8b39
.
This commit is contained in:
parent
c7f02d5e51
commit
2395c6fdaa
|
@ -201,6 +201,7 @@ LIB_EDIT_FRAME::LIB_EDIT_FRAME( KIWAY* aKiway, wxWindow* aParent ) :
|
||||||
SCH_BASE_FRAME( aKiway, aParent, FRAME_SCH_LIB_EDITOR, _( "Library Editor" ),
|
SCH_BASE_FRAME( aKiway, aParent, FRAME_SCH_LIB_EDITOR, _( "Library Editor" ),
|
||||||
wxDefaultPosition, wxDefaultSize, KICAD_DEFAULT_DRAWFRAME_STYLE, LIB_EDIT_FRAME_NAME )
|
wxDefaultPosition, wxDefaultSize, KICAD_DEFAULT_DRAWFRAME_STYLE, LIB_EDIT_FRAME_NAME )
|
||||||
{
|
{
|
||||||
|
m_partSelectBox = nullptr;
|
||||||
m_showAxis = true; // true to draw axis
|
m_showAxis = true; // true to draw axis
|
||||||
SetShowDeMorgan( false );
|
SetShowDeMorgan( false );
|
||||||
m_drawSpecificConvert = true;
|
m_drawSpecificConvert = true;
|
||||||
|
|
|
@ -177,12 +177,12 @@ void LIB_EDIT_FRAME::ReCreateHToolbar()
|
||||||
|
|
||||||
KiScaledSeparator( m_mainToolBar, this );
|
KiScaledSeparator( m_mainToolBar, this );
|
||||||
|
|
||||||
m_partSelectBox = new wxComboBox( m_mainToolBar,
|
if( m_partSelectBox == nullptr )
|
||||||
ID_LIBEDIT_SELECT_PART_NUMBER,
|
m_partSelectBox = new wxComboBox( m_mainToolBar, ID_LIBEDIT_SELECT_PART_NUMBER,
|
||||||
wxEmptyString,
|
wxEmptyString,
|
||||||
wxDefaultPosition,
|
wxDefaultPosition, wxSize( LISTBOX_WIDTH, -1 ),
|
||||||
wxSize( LISTBOX_WIDTH, -1 ),
|
0, nullptr, wxCB_READONLY );
|
||||||
0, nullptr, wxCB_READONLY );
|
|
||||||
m_mainToolBar->AddControl( m_partSelectBox );
|
m_mainToolBar->AddControl( m_partSelectBox );
|
||||||
|
|
||||||
KiScaledSeparator( m_mainToolBar, this );
|
KiScaledSeparator( m_mainToolBar, this );
|
||||||
|
|
|
@ -95,8 +95,10 @@ void LIB_VIEW_FRAME::ReCreateHToolbar()
|
||||||
|
|
||||||
m_mainToolBar->AddSeparator();
|
m_mainToolBar->AddSeparator();
|
||||||
|
|
||||||
m_unitChoice = new wxChoice( m_mainToolBar, ID_LIBVIEW_SELECT_PART_NUMBER,
|
if( m_unitChoice == nullptr )
|
||||||
wxDefaultPosition, wxSize( 150, -1 ) );
|
m_unitChoice = new wxChoice( m_mainToolBar, ID_LIBVIEW_SELECT_PART_NUMBER,
|
||||||
|
wxDefaultPosition, wxSize( 150, -1 ) );
|
||||||
|
|
||||||
m_mainToolBar->AddControl( m_unitChoice );
|
m_mainToolBar->AddControl( m_unitChoice );
|
||||||
|
|
||||||
m_mainToolBar->AddSeparator();
|
m_mainToolBar->AddSeparator();
|
||||||
|
|
|
@ -107,6 +107,7 @@ LIB_VIEW_FRAME::LIB_VIEW_FRAME( KIWAY* aKiway, wxWindow* aParent, FRAME_T aFrame
|
||||||
wxDefaultPosition, wxDefaultSize,
|
wxDefaultPosition, wxDefaultSize,
|
||||||
aFrameType == FRAME_SCH_VIEWER_MODAL ? LIB_VIEW_STYLE_MODAL : LIB_VIEW_STYLE,
|
aFrameType == FRAME_SCH_VIEWER_MODAL ? LIB_VIEW_STYLE_MODAL : LIB_VIEW_STYLE,
|
||||||
aFrameType == FRAME_SCH_VIEWER_MODAL ? LIB_VIEW_NAME_MODAL : LIB_VIEW_NAME ),
|
aFrameType == FRAME_SCH_VIEWER_MODAL ? LIB_VIEW_NAME_MODAL : LIB_VIEW_NAME ),
|
||||||
|
m_unitChoice( nullptr ),
|
||||||
m_libList( nullptr ),
|
m_libList( nullptr ),
|
||||||
m_cmpList( nullptr ),
|
m_cmpList( nullptr ),
|
||||||
m_previewItem( nullptr )
|
m_previewItem( nullptr )
|
||||||
|
|
Loading…
Reference in New Issue