GERBVIEW_SELECTION_TOOL, disambiguationMenu: do not hide an item before highlight it.

This is not useful, and creates an issue when closing the menu by ESC key.
Fixes #12636
https://gitlab.com/kicad/code/kicad/issues/12636
This commit is contained in:
jean-pierre charras 2022-10-14 09:33:32 +02:00
parent 292492bd01
commit 8ee73903c6
1 changed files with 1 additions and 4 deletions

View File

@ -442,7 +442,6 @@ EDA_ITEM* GERBVIEW_SELECTION_TOOL::disambiguationMenu( GERBER_COLLECTOR* aCollec
if( current )
{
current->ClearBrightened();
getView()->Hide( current, false );
highlightGroup.Remove( current );
getView()->MarkTargetDirty( KIGFX::TARGET_OVERLAY );
}
@ -454,7 +453,6 @@ EDA_ITEM* GERBVIEW_SELECTION_TOOL::disambiguationMenu( GERBER_COLLECTOR* aCollec
{
current = ( *aCollector )[id - 1];
current->SetBrightened();
getView()->Hide( current, true );
highlightGroup.Add( current );
getView()->MarkTargetDirty( KIGFX::TARGET_OVERLAY );
}
@ -477,10 +475,9 @@ EDA_ITEM* GERBVIEW_SELECTION_TOOL::disambiguationMenu( GERBER_COLLECTOR* aCollec
}
}
if( current && current->IsBrightened() )
if( current )
{
current->ClearBrightened();
getView()->Hide( current, false );
getView()->MarkTargetDirty( KIGFX::TARGET_OVERLAY );
}