From 86e0e1cccbe094dcd75954bcf61d9113cc9f6a94 Mon Sep 17 00:00:00 2001 From: Mike Williams Date: Tue, 20 Jun 2023 12:21:50 -0400 Subject: [PATCH] PCB, Change Footprint: actually update text attributes --- pcbnew/pcb_edit_frame.cpp | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/pcbnew/pcb_edit_frame.cpp b/pcbnew/pcb_edit_frame.cpp index 2752ebb1e0..a64f2f3974 100644 --- a/pcbnew/pcb_edit_frame.cpp +++ b/pcbnew/pcb_edit_frame.cpp @@ -2078,17 +2078,17 @@ void processTextItem( const PCB_TEXT& aSrc, PCB_TEXT& aDest, bool* aUpdated ) { if( resetText ) + { *aUpdated |= aSrc.GetText() != aDest.GetText(); - else + aDest.SetText( aSrc.GetText() ); + } if( resetTextLayers ) { *aUpdated |= aSrc.GetLayer() != aDest.GetLayer(); *aUpdated |= aSrc.IsVisible() != aDest.IsVisible(); - } - else - { + aDest.SetLayer( aSrc.GetLayer() ); aDest.SetVisible( aSrc.IsVisible() ); } @@ -2101,9 +2101,7 @@ void processTextItem( const PCB_TEXT& aSrc, PCB_TEXT& aDest, *aUpdated |= aSrc.GetTextThickness() != aDest.GetTextThickness(); *aUpdated |= aSrc.GetTextAngle() != aDest.GetTextAngle(); *aUpdated |= aSrc.GetFPRelativePosition() != aDest.GetFPRelativePosition(); - } - else - { + // Careful: the visible bit and position are also set by SetAttributes() bool visible = aDest.IsVisible(); aDest.SetAttributes( aSrc );