kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #31260
Re: [PATCH] Add live footprint filtering in modview window
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Mon, 30 Oct 2017 19:40:22 -0400
-
In-reply-to:
<CAMfgvU_iU-M-TaEQvWc-ZcSq7zS_fmEUCtJ9ES6qta+A3Q=LKA@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0
On 10/30/2017 5:23 PM, Oliver Walters wrote:
> Thanks for the suggestions on fixing the text. I have that sorted.
>
> I will look into different ways of caching footprint data so it is quicker.
>
> Wayne, I didn't know about FOOTPRINT_FILTER I will switch to using that
> instead (and provide regex search).
Thanks Oliver!
>
> On 31 Oct 2017 06:55, "Seth Hillbrand" <seth.hillbrand@xxxxxxxxx
> <mailto:seth.hillbrand@xxxxxxxxx>> wrote:
>
> On Mon, Oct 30, 2017 at 11:42 AM, Wayne Stambaugh
> <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>wrote:
>
> On 10/30/2017 1:16 PM, Seth Hillbrand wrote:
> > Oliver, this is neat and very helpful.
> >
> > The greyed-out thing is a wx2.8 bug. You can work around it by setting
> > the foreground color when updating the filter like this:
> >
> > void FOOTPRINT_VIEWER_FRAME::OnFilterUpdated( wxCommandEvent& event )
> > {
> > + // Workaround wx2.8 bug showing greyed color
> > + if( m_searchBox->GetValue() != m_searchBox->GetDescriptiveText() )
> > + m_searchBox->SetForegroundColour(
> > m_searchBox->GetDefaultAttributes().colFg );
> > +
> > // Filter is non case sensitive
> > wxString filter = m_searchBox->GetValue().Lower();
> >
> > The searchbox handles resetting it to grey on idle() when the text is empty.
>
> Don't you mean wx 3.0? CMake should not even generate the build
> configuration files without wx 3.0 or greater.
>
>
> Hmm... This was an issue back in 2.8 that appears to be only partly
> fixed. The workaround I suggest above is functional but, for this,
> we can also execute a cleaner fix by setting the descriptive text in
> the declaration:
>
> @@ -67,9 +67,10 @@ void FOOTPRINT_VIEWER_FRAME::ReCreateHToolbar()
> KiBitmap( module_xpm ),
> _( "Select footprint to browse" ) );
>
> - m_searchBox = new wxSearchCtrl( m_mainToolBar,
> ID_MODVIEW_SEARCH_TEXT );
> + m_searchBox = new wxSearchCtrl( m_mainToolBar,
> ID_MODVIEW_SEARCH_TEXT,
> + _( "Enter filter string" ) );
> m_searchBox->SetMinSize( wxSize( 250, 30 ) );
> - m_searchBox->SetDescriptiveText( _( "Enter filter string" ) );
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> More help : https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
>
>
>
> _______________________________________________
> 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
>
Follow ups
References