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

Make symbol highlighting from Symbol Fields Table working again. 

Fixes #13003
https://gitlab.com/kicad/code/kicad/issues/13003
This commit is contained in:
jean-pierre charras 2022-11-25 16:17:20 +01:00
parent 7e0ba42f0e
commit f2efb9c601
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
eeschema/dialogs/dialog_symbol_fields_table.cpp
View File

@ -1120,7 +1120,7 @@ void DIALOG_SYMBOL_FIELDS_TABLE::OnRemoveField( wxCommandEvent& event )
wxString confirm_msg = wxString::Format( _( "Are you sure you want to remove the field '%s'?" ), wxString confirm_msg = wxString::Format( _( "Are you sure you want to remove the field '%s'?" ),
fieldName ); fieldName );
if( !IsOK( this, confirm_msg ) ) if( !IsOK( this, confirm_msg ) )
return; return;
@ -1142,7 +1142,7 @@ void DIALOG_SYMBOL_FIELDS_TABLE::OnRemoveField( wxCommandEvent& event )
wxGridTableMessage msg( m_dataModel, wxGRIDTABLE_NOTIFY_COLS_DELETED, wxGridTableMessage msg( m_dataModel, wxGRIDTABLE_NOTIFY_COLS_DELETED,
m_fieldsCtrl->GetItemCount(), 1 ); m_fieldsCtrl->GetItemCount(), 1 );
m_grid->ProcessTableMessage( msg ); m_grid->ProcessTableMessage( msg );
// set up attributes on the new quantities column // set up attributes on the new quantities column
@ -1311,10 +1311,9 @@ void DIALOG_SYMBOL_FIELDS_TABLE::OnTableCellClick( wxGridEvent& event )
{ {
SCH_EDITOR_CONTROL* editor = m_parent->GetToolManager()->GetTool<SCH_EDITOR_CONTROL>(); SCH_EDITOR_CONTROL* editor = m_parent->GetToolManager()->GetTool<SCH_EDITOR_CONTROL>();
wxString path = refs[0].GetPath(); // search and highlight the symbol found by reference
wxString reference = refs[0].GetRef() + refs[0].GetRefNumber(); wxString reference = refs[0].GetRef() + refs[0].GetRefNumber();
editor->FindSymbolAndItem( nullptr, &reference, true, HIGHLIGHT_SYMBOL, wxEmptyString );
editor->FindSymbolAndItem( &path, &reference, true, HIGHLIGHT_SYMBOL, wxEmptyString );
} }
} }
else else