← Back to team overview

kicad-developers team mailing list archive

Re: PATCH: remove intermediate modal message in creating NET file that needs annotation first

 

Fix committed in r5812.  Thanks.

On 6/22/2015 10:05 PM, Henner Zeller wrote:
> On 22 June 2015 at 11:25, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>
>> On 6/22/2015 1:05 PM, Henner Zeller wrote:
>>> On 22 June 2015 at 09:39, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>>> Henner,
>>>>
>>>> This patch fails to apply cleanly against your
>>>> better-annotation-dialog-checkbox-messaging.diff patch.
>>>
>>> I think it conflicted with the "library rescue improvements" patch as
>>> there were changes in the vicinity.
>>>
>>> Updated the patch. Can you try again ?
>>>   https://github.com/hzeller/kicad/compare/remove-unnecessary-intermediate-step.diff
>>>
>>> -h
>>
>> One small fix I would like to see on this patch is to hide the user
>> message text control without the dead space at the top of the dialog
>> box.  I believe if you put the static text control in it's own sizer,
>> you can hide the sizer and the spacing will take care of itself.  Maybe
>> you can just hide the static text control.  You may have to call one of
>> the wxWindows fit functions to get this work properly but it would make
>> for a nicer looking dialog.
> 
> Just hiding did the trick. Updated patch on
>   https://github.com/hzeller/kicad/compare/remove-unnecessary-intermediate-step.diff
> 
> 
>>
>>>
>>>> Wayne
>>>>
>>>> On 6/20/2015 6:57 PM, Henner Zeller wrote:
>>>>> Hi,
>>>>> When writing a NET file, eeschema notices if not everything is
>>>>> annotated, so sends the user to annotate the schematic.
>>>>>
>>>>> However, it does so by asking this question in a modal dialog box
>>>>> first, and _then_ opens the annotation dialog, which is quite some
>>>>> annoying sequence.
>>>>> In particular if you go through it many times a day :)
>>>>>
>>>>> So I removed that superfluous message box, but instead added the
>>>>> message to the annotation dialog in that case. The user still can
>>>>> cancel there if they want.
>>>>>
>>>>> (See also my 'Possibly low-hanging fruit usability improvement' mail today)
>>>>>
>>>>> Patch is here:
>>>>>
>>>>> https://github.com/hzeller/kicad/compare/remove-unnecessary-intermediate-step.diff
>>>>>
>>>>> Suggested commit message
>>>>> -------
>>>>> Remove one unnecessary step in the workflow 'exporting NET file
>>>>> that requires annotation first'.
>>>>> -------
>>>>>
>>>>> -h
>>>>>
>>>>> _______________________________________________
>>>>> 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
>>
>> _______________________________________________
>> 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


References