← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Hotkey edit dialog filter + fixes

 

Hey John,

Given that 5.1 is pretty far along and the delta between the 5.0 and
development branches, I would rather you didn't spent a lot of time
trying to back port the dialog fixes which are low priority.  I want to
get 5.0.1 tagged as soon as possible and merge the eeschema-gal stuff so
we can get some wider testing on that.  Having all hands on deck to fix
any issue we find in the eeschema-gal code will be more beneficial.

Cheers,

Wayne

On 09/28/2018 10:23 AM, John Beard wrote:
> Hi Wayne,
> 
> Do you have any preference on the hotkey list dialog? I'm happy to go
> either way, but I'd like to know the preferred direction before I
> spend too much time on it!
> 
> Cheers,
> 
> John
> On Fri, Sep 28, 2018 at 3:11 PM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>> Hey John,
>>
>> Nice work!  I merged your patch into the development branch.  I do have
>> one favor to ask if you have the time.  Patch 6 does not apply to the
>> 5.0 branch which also suffers from the crash bug.  If you could merge it
>> into the 5.0 branch and send me a separate patch I would appreciate it.
>> If you don't have the time please let me know because this needs to be
>> fixed for the 5.0.1 release.
>>
>> Cheers,
>>
>> Wayne
>>
>> On 09/28/2018 07:53 AM, John Beard wrote:
>>> Hi,
>>>
>>> Here is a patch set for adding a filter control to the hotkey editor
>>> dialog. Preview video: https://sendvid.com/je0cyg87
>>>
>>> Most of the work in the first commit is separating out the hotkey data
>>> from the UI widget code.
>>>
>>> This also fixes a couple of other bugs (one crashing, and one able to
>>> get the HK configs into a mess with conflicts) in that dialog, and
>>> adds some mock objects to the common QA tests, which will make it
>>> easier to get more of libcommon under test.
>>>
>>> A 5.0 series fix for the crashing bug will follow.
>>>
>>> This is also useful as groundwork for
>>> https://bugs.launchpad.net/kicad/+bug/1778374 (a 5.1 milestone), but
>>> that will need a further decision to choose between:
>>>
>>> * Use a read-only variant of the hotkey editor TreeListView for the
>>> hotkey list dialog (needs more code: a new dialog and adding a
>>> read-only mode to the current widget), or
>>> * Remove the hotkey list dialog altogether and bind the "show hotkey
>>> list" command to simply opening the prefs dialog to the hotkey panel
>>>
>>> Cheers,
>>>
>>> John
>>>
>>>
>>>
>>> _______________________________________________
>>> 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
>>>
>>
>> _______________________________________________
>> 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