← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Add KICAD_TEMPLATE_DIR to Configure Paths... help

 

John,

I merged your patch.

Thanks,

Wayne

On 10/2/2018 5:42 PM, John Beard wrote:
> Hi Wayne,
> 
> Here's the updated patch. The only changes are comments and var names.
> 
> I'll keep looking at wxGrid cell tooltips. If anyone knows if we
> already can do that (or WX has an easy way that's doesn't need to deal
> with mouse events manually, please let me know!)
> 
> Cheers,
> 
> John
> On Tue, Oct 2, 2018 at 10:04 PM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>
>> Hey John,
>>
>> No problem.  I'll hold off until you send the revised patch.  I'm fine
>> with the current patch the way that it is.  It was just the tool tip
>> code that threw me off.  I thought maybe there was something missing.
>>
>> Cheers,
>>
>> Wayne
>>
>> On 10/02/2018 04:45 PM, John Beard wrote:
>>> Hi Wayne,
>>>
>>> Sorry about that, I actually started with the intention of providing
>>> tooltips for the wxGrids in the library manager and the paths
>>> configuration dialog. However, I found adding a tooltip to a wxGrid
>>> cells is not very easy, so I retreated to just updating the help text
>>> and centralising logic. I evidently neglected to update the comments
>>> adequately.
>>>
>>> I haven't given up on the tooltips, but it'll need more work to do it
>>> nicely in wx. On yt bright side, hopefully if I can do it, it will be
>>> helpful for decorating other wxGrids with per cell tooltips.
>>>
>>> I'll update the commentary on this patch.
>>>
>>> Cheers,
>>>
>>> John
>>>
>>> On 1 October 2018 18:58:17 BST, Wayne Stambaugh <stambaughw@xxxxxxxxx>
>>> wrote:
>>>
>>>     Hey John,
>>>
>>>     I like this solution to the translation issue better than some of the
>>>     other solutions that have been proposed.  I did notice that you added a
>>>     function to provide the tooltip for defined env variable but you did not
>>>     hook up the tooltip logic in the dialog.  Are you planning to do that at
>>>     some point in the future?
>>>
>>>     I'm going to merge this patch if there are no objections.
>>>
>>>     Cheers,
>>>
>>>     Wayne
>>>
>>>     On 10/1/2018 5:35 AM, John Beard wrote:
>>>
>>>         Hi,
>>>
>>>         Small patch that:
>>>
>>>         * Adds KICAD_TEMPLATE_DIR and KICAD_USER_TEMPLATE_DIR to the help
>>>         dialog in Configure Paths...
>>>         * Documents KICAD_PTEMPLATES as deprecated.
>>>         * Moves some generic functions to do with env variables to a
>>>         separate
>>>         file. This means env var help text is no longer embedded in a
>>>         block of
>>>         HTML and also we could use it to provide tool-tips or similar to
>>>         other
>>>         dialogs.
>>>
>>>         Cheers,
>>>
>>>         John
>>>         ------------------------------------------------------------------------
>>>         Mailing list: https://launchpad.net/~kicad-developers
>>>         <https://launchpad.net/%7Ekicad-developers>
>>>         Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>         Unsubscribe : https://launchpad.net/~kicad-developers
>>>         <https://launchpad.net/%7Ekicad-developers>
>>>         More help : https://help.launchpad.net/ListHelp
>>>
>>>     ------------------------------------------------------------------------
>>>     Mailing list: https://launchpad.net/~kicad-developers
>>>     <https://launchpad.net/%7Ekicad-developers>
>>>     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>     Unsubscribe : https://launchpad.net/~kicad-developers
>>>     <https://launchpad.net/%7Ekicad-developers>
>>>     More help   : https://help.launchpad.net/ListHelp
>>>


References