← Back to team overview

kicad-developers team mailing list archive

Re: Default Canvas for v5

 

Hey Jon,

Did you forget to add a file to this patch?  I'm getting the following
build error:

CMakeFiles/gerbview_kiface.dir/gerbview_frame.cpp.o: In function
`GERBVIEW_FRAME::GERBVIEW_FRAME(KIWAY*, wxWindow*)':
/home/wayne/src/kicad/kicad-trunk/gerbview/gerbview_frame.cpp:218:
undefined reference to `EDA_DRAW_FRAME::ShowFirstRunDialog(EDA_DRAW_FRAME*)'

I don't see a ShowFirstRunDialog() in the patch.

Wayne


On 01/03/2018 10:27 PM, Jon Evans wrote:
> Hi Wayne, Nick,
> 
> Updated patch attached with your suggestions.
> 
> -Jon
> 
> On Wed, Jan 3, 2018 at 3:36 PM, Nick Østergaard <oe.nick@xxxxxxxxx
> <mailto:oe.nick@xxxxxxxxx>> wrote:
> 
>     Hmm, maybe it is ok to use the technical terms OpenGL and Cairo in
>     the help text for the menu item. Afterall those terms have been used
>     a lot in help forums and launchpad, so it may make it easier for the
>     curious to discover.
> 
>     2018-01-03 21:15 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx
>     <mailto:stambaughw@xxxxxxxxx>>:
> 
>         There is also wxRichMessageDialog[1] which has an optional
>         checkbox for
>         show once features.
> 
>         [1]:
>         http://docs.wxwidgets.org/3.0/classwx_rich_message_dialog.html
>         <http://docs.wxwidgets.org/3.0/classwx_rich_message_dialog.html>
> 
>         On 1/3/2018 3:13 PM, Jon Evans wrote:
>         > I figured we might want to re-use this in the future if we ever have
>         > additional "show once" things that are more complicated than yes/no
>         > (although I didn't make it with formbuilder).
>         > It would be fine to switch to a messagedialog.  I can probably do that
>         > tonight.
>         >
>         > -Jon
>         >
>         > On Wed, Jan 3, 2018 at 3:09 PM, Wayne Stambaugh <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>
>         > <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>> wrote:
>         >
>         >     John,
>         >
>         >     I took a look at your patch.  Is there any reason why you didn't use
>         >     wxMessageDialog instead of creating wxFormbuilder project for such a
>         >     simple dialog?  It seems to me that wxMessageDialog would do the trick
>         >     as it allows you to change the strings for the Ok and Cancel buttons.
>         >     Other than that, everything else seems fine.
>         >
>         >     Cheers,
>         >
>         >     Wayne
>         >
>         >     On 12/31/2017 4:28 PM, Jon Evans wrote:
>         >     > For your consideration, a patch is attached that implements the above.
>         >     >
>         >     >
>         >     > On Sun, Dec 31, 2017 at 2:24 PM, Jon Evans <jon@xxxxxxxxxxxxx <mailto:jon@xxxxxxxxxxxxx>
>         <mailto:jon@xxxxxxxxxxxxx <mailto:jon@xxxxxxxxxxxxx>>
>         >     > <mailto:jon@xxxxxxxxxxxxx <mailto:jon@xxxxxxxxxxxxx>
>         <mailto:jon@xxxxxxxxxxxxx <mailto:jon@xxxxxxxxxxxxx>>>> wrote:
>         >     >
>         >     >     I think that getting automatic OpenGL detection and recovery working
>         >     >     for 5.0 might be ambitious.
>         >     >     Maybe Chris can detail what the next steps are for his approach
>         >     >     there, but in case we want to push that to a later release, here's
>         >     >     what I propose:
>         >     >
>         >     >     1) Rename the View menu options based on my proposal "Legacy",
>         >     >     "Modern (Accelerated)", "Modern (Fallback)"
>         >     >     2) Change PCB_BASE_FRAME::SwitchCanvas to save the canvas config
>         >     >     value after calling UseGalCanvas() rather than before.
>         >     >     3) Show a first-run dialog like the mockup in my earlier email, IF
>         >     >     canvas is not OpenGL and the dialog has not been shown before
>         >     >     44) If user clicks "yes" in the first run dialog, then:
>         >     >         (a) set the config value to prevent the dialog from showing again,
>         >     >         (b) Set the canvas config setting to Cairo (but don't switch the
>         >     >     GAL to Cairo), then
>         >     >         (c) switch to OpenGL canvas using SwitchCanvas()
>         >     >
>         >     >     That way, if the app crashes in step 4c, it should come back up as
>         >     >     Cairo on the next launch (assuming we crash right away when trying
>         >     >     to set up the canvas)
>         >     >
>         >     >     Any concerns with the above?
>         >     >
>         >     >     -Jon
>         >     >
>         >     >
>         >     >
>         >     >     On Sun, Dec 31, 2017 at 1:55 PM, Jeff Young <jeff@xxxxxxxxx <mailto:jeff@xxxxxxxxx> <mailto:jeff@xxxxxxxxx
>         <mailto:jeff@xxxxxxxxx>>
>         >     >     <mailto:jeff@xxxxxxxxx <mailto:jeff@xxxxxxxxx> <mailto:jeff@xxxxxxxxx
>         <mailto:jeff@xxxxxxxxx>>>> wrote:
>         >     >
>         >     >         +1
>         >     >
>         >     >         No menu items.  Just a checkbox for Enable Hardware Acceleration
>         >     >         in Preferences.
>         >     >
>         >     >>         On 31 Dec 2017, at 18:36, Andy Peters <devel@xxxxxxxxx <mailto:devel@xxxxxxxxx> <mailto:devel@xxxxxxxxx
>         <mailto:devel@xxxxxxxxx>>
>         >     >>         <mailto:devel@xxxxxxxxx <mailto:devel@xxxxxxxxx> <mailto:devel@xxxxxxxxx
>         <mailto:devel@xxxxxxxxx>>>> wrote:
>         >     >>
>         >     >>>         - In the case of graphics glitches, inform the users in the
>         >     >>>         FAQ/Manual
>         >     >>>         that they can fall back to software renderer in the View menu.
>         >     >>>         - Don't use the term 'canvas' or 'view'. Just 'Enable HW
>         >     >>>         acceleration’.
>         >     >>
>         >     >>         I like that. The fewer technical/programmer terms, the better.
>         >     >
>         >     >
>         >     >         _______________________________________________
>         >     >         Mailing list: https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>
>         >     >         <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>>
>         >     >         Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>         >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>         >     >         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>         >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>>
>         >     >         Unsubscribe :
>         https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>
>         >     >         <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>>
>         >     >         More help   :
>         https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>
>         >     <https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>>
>         >     >         <https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>
>         >     <https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>>>
>         >     >
>         >     >
>         >     >
>         >     >
>         >     >
>         >     > _______________________________________________
>         >     > Mailing list: https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>
>         >     > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>         >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>         >     > Unsubscribe : https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>
>         >     > More help   : https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>
>         >     <https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>>
>         >     >
>         >
>         >     _______________________________________________
>         >     Mailing list: https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>
>         >     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>         >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
>         >     Unsubscribe : https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>
>         >     <https://launchpad.net/~kicad-developers
>         <https://launchpad.net/~kicad-developers>>
>         >     More help   : https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>
>         >     <https://help.launchpad.net/ListHelp
>         <https://help.launchpad.net/ListHelp>>
>         >
>         >
> 
>         _______________________________________________
>         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>
> 
> 
> 


Follow ups

References