← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH 1/7] Clear comp highlighting before highlight new one.

 

I don't see e65fb87a4 in master branch.
But I have just pulled master branch, have built it and component
highlighting works fine now.
So this patch is not necessary.

ср, 16 янв. 2019 г. в 15:50, Wayne Stambaugh <stambaughw@xxxxxxxxx>:

> Is this patch necessary?  I'm not seeing this with e65fb87a4.  Clicking
> on another reference in the field editor clears the previously
> highlighted symbol.
>
> On 1/16/2019 8:16 AM, Baranovskiy Konstantin wrote:
> > On clicking a reference cell in the field editor of Eeschema a
> > corresponding component is highlighted in Eeschema. But previously
> > highlighted components are staying highlighted.
> >
> > With current patch all highlightings are cleared before highlighting
> > selected component.
> > ---
> >  eeschema/dialogs/dialog_fields_editor_global.cpp | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/eeschema/dialogs/dialog_fields_editor_global.cpp
> b/eeschema/dialogs/dialog_fields_editor_global.cpp
> > index d2616ea95..603ebf649 100644
> > --- a/eeschema/dialogs/dialog_fields_editor_global.cpp
> > +++ b/eeschema/dialogs/dialog_fields_editor_global.cpp
> > @@ -979,6 +979,7 @@ void DIALOG_FIELDS_EDITOR_GLOBAL::OnTableCellClick(
> wxGridEvent& event )
> >      {
> >          // Clear highligted symbols, if any
> >          m_parent->GetCanvas()->GetView()->HighlightItem( nullptr,
> nullptr );
> > +        m_parent->GetCanvas()->Refresh();
> >
> >          m_dataModel->ExpandCollapseRow( event.GetRow() );
> >          std::vector<SCH_REFERENCE> refs =
> m_dataModel->GetRowReferences( event.GetRow() );
> >
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
>

References