[Ctk-developers] ShowStatusTextUponGoToStepSuccess workflow feature, please review

[Jean-Christophe Fillion-Robin]

+1

On Tue, Aug 16, 2011 at 8:03 PM, Julien Finet <julien.finet at kitware.com>wrote:

> I usually prefer XXXEnabled to XXXDisabled.
> Even if it is disabled by default.
>
> Julien.
>
>
> On Tue, Aug 16, 2011 at 3:39 PM, Jean-Christophe Fillion-Robin <
> jchris.fillionr at kitware.com> wrote:
>
>> Hi Daniel,
>>
>> Just looked at it.
>>
>> Please consider the following remarks:
>>  - Regarding the git commit msg. See
>> http://www.commontk.org/index.php/Contributing_to_CTK#Git_Commit_Style
>>
>>  - Regarding the patch:
>>      - May be you could just add a property named ErrorTextDisabled to
>> ctkWorkflowGroupBox ?
>>     - See line 110 of ctkWorkflowGroupBox.cpp
>>
>> Danielle> What do you think ?
>>
>> Thanks
>> Jc
>>
>>
>> On Tue, Aug 16, 2011 at 12:35 PM, Daniel Haehn <haehn at bwh.harvard.edu>wrote:
>>
>>> Hi,
>>>
>>> did anybody have a chance to look at it yet?
>>>
>>> Thanks,
>>> Daniel
>>>
>>> On Thu, Aug 11, 2011 at 1:15 PM, Daniel Haehn <haehn at bwh.harvard.edu>
>>> wrote:
>>> > Hi Devels,
>>> >
>>> > could anybody please review the following topic:
>>> >
>>> >
>>> https://github.com/haehn/CTK/commits/add-ShowStatusTextUponGoToStepSuccess-workflow-feature
>>> >
>>> > I added a property to be able to disable the Status Text update on the
>>> > origin step when a GoToStep(..)-call of a ctkWorkflow succeeds.
>>> >
>>> > Looking forward to your feedback..
>>> >
>>> > Daniel
>>> >
>>> _______________________________________________
>>> Ctk-developers mailing list
>>> Ctk-developers at commontk.org
>>> http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers
>>>
>>
>>
>>
>> --
>> +1 919 869 8849
>>
>>
>> _______________________________________________
>> Ctk-developers mailing list
>> Ctk-developers at commontk.org
>> http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers
>>
>>
>


-- 
+1 919 869 8849
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20110816/3b2bbddd/attachment.htm>