kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #16187
Re: OS X File Open Dialog
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Mon, 05 Jan 2015 08:23:31 -0500
-
Cc:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
In-reply-to:
<CAGu=xg3_zohAHM2RoSc0k4HDC2i0UWQB1z72LA8mtVOUeGJNUQ@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0
Would one of the OSX devs please validate this patch to see if it
resolves the OSX file open dialog issue and let me know so I can commit
this patch?
Thanks,
Wayne
On 1/3/2015 3:06 PM, Paulo Henrique Silva wrote:
> Hi Bernard,
>
> Different from the patch I have added the code to
> bool PGM_BASE::initPgm()
> because this seems to be one of the more global spots to initialize
> things.
> I am not sure about that… in kiface_i.cpp
> bool KIFACE_I::start_common( int aCtlBits )
> would also maybe appropriate?
>
> But… even if it brings back the filter drop down, it doesn’t seem to
> work with wxWidgets 3.0.2.
> The
> <<<
> wxFileDialog dlg( aParent, _( "Open Board File" ), path, name,
> fileFilters,
> wxFD_OPEN | wxFD_FILE_MUST_EXIST );
>
> if( dlg.ShowModal() != wxID_CANCEL )
> {
> int chosenFilter = dlg.GetFilterIndex();
> >>>
> dlg.GetFilterIndex() always return 0 regardless from what I choose.
> Moreover, changing the filter does not affect the file list at all
> (from the wxWidgets code it should…).
> I guess files not matching the current filter should get greyed out,
> but always all supported files are enabled.
>
> So, this doesn’t really seem to fix the original problem - at least
> on both my 10.9 and 10.10 setup.
> Can somebody else confirm this?
>
>
> You are right.
>
> I dig a little and the real issue seems to be on wxWidgets. I filled a
> bug on wxWidgets (still being moderated). The wxWidgets patch is
> attached here together with my original change now applied at
> PGM_BASE::initPgm as you did.
>
> I tested it on OX 10.10 and it works as expected now. Patch cleanly
> applied on wxWidgets 3.0.2 and wxPython 3.0.2.0.
>
> Sorry to miss the wxWdgets issue.
>
> Cheers,
>
> -- Paulo Henrique
> -- Curta Circuitos
>
>
Follow ups
References