From ef6001bfe6c469f74488bf9b037ab123766f7831 Mon Sep 17 00:00:00 2001 From: Jeff Young Date: Wed, 28 Dec 2022 10:20:05 +0000 Subject: [PATCH] Use consistent terminology: opposite of Add is Remove. Note: violating string freeze because this change should ease translation, not make it harder. --- eeschema/sim/sim_plot_frame.cpp | 10 ++++++---- eeschema/sim/sim_plot_frame.h | 2 +- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/eeschema/sim/sim_plot_frame.cpp b/eeschema/sim/sim_plot_frame.cpp index e9c849699d..1c6dbc7b3a 100644 --- a/eeschema/sim/sim_plot_frame.cpp +++ b/eeschema/sim/sim_plot_frame.cpp @@ -1922,11 +1922,13 @@ SIM_PLOT_FRAME::SIGNAL_CONTEXT_MENU::SIGNAL_CONTEXT_MENU( const wxString& aSigna { SIM_PLOT_PANEL* plot = m_plotFrame->GetCurrentPlot(); - AddMenuItem( this, HIDE_SIGNAL, _( "Hide Signal" ), _( "Erase the signal from plot screen" ), + AddMenuItem( this, REMOVE_SIGNAL, _( "Remove Signal" ), _( "Remove the signal from the plot" ), KiBitmap( BITMAPS::trash ) ); TRACE* trace = plot->GetTrace( m_signal ); + AppendSeparator(); + if( trace->HasCursor() ) AddMenuItem( this, HIDE_CURSOR, _( "Hide Cursor" ), KiBitmap( BITMAPS::pcb_target ) ); else @@ -1943,9 +1945,9 @@ void SIM_PLOT_FRAME::SIGNAL_CONTEXT_MENU::onMenuEvent( wxMenuEvent& aEvent ) switch( aEvent.GetId() ) { - case HIDE_SIGNAL: m_plotFrame->removePlot( m_signal ); break; - case SHOW_CURSOR: plot->EnableCursor( m_signal, true ); break; - case HIDE_CURSOR: plot->EnableCursor( m_signal, false ); break; + case REMOVE_SIGNAL: m_plotFrame->removePlot( m_signal ); break; + case SHOW_CURSOR: plot->EnableCursor( m_signal, true ); break; + case HIDE_CURSOR: plot->EnableCursor( m_signal, false ); break; } } diff --git a/eeschema/sim/sim_plot_frame.h b/eeschema/sim/sim_plot_frame.h index a77040add1..00e623a24c 100644 --- a/eeschema/sim/sim_plot_frame.h +++ b/eeschema/sim/sim_plot_frame.h @@ -340,7 +340,7 @@ private: enum SIGNAL_CONTEXT_MENU_EVENTS { - HIDE_SIGNAL = 944, + REMOVE_SIGNAL = 944, SHOW_CURSOR, HIDE_CURSOR };