[vtkusers] Handling "holding down" a key

Bill Lorensen bill.lorensen at gmail.com
Sun Sep 12 14:23:19 EDT 2010


David G,

I don't see style changes. I see comment addition...

On Sun, Sep 12, 2010 at 12:53 PM, David Gobbi <david.gobbi at gmail.com> wrote:
> Didn't David Cole just tell you how hard it is to read commits that
> mix style changes with functional changes?  And I'm not the best
> person to review this... I don't use VTK's built-in InteractorStyles.
>
>  David
>
>
> On Sun, Sep 12, 2010 at 10:38 AM, David Doria <daviddoria+vtk at gmail.com> wrote:
>>>> Hi David,
>>>>
>>>> Clicking the 'x' key to change the state is preferable.  You just need
>>>> to add a visual cue to inform the user of the current state, e.g. you
>>>> can change the shape of the mouse cursor or draw a horizontal arrow
>>>> somewhere.
>>>>
>>>>  David
>>
>> @David G.
>>
>> I made branch here:
>>
>> repo: git at github.com:daviddoria/daviddoria-vtk.git
>> branch: VTK-TrackballActor
>>
>> Would you mind taking a look?
>>
>> (These two commits:
>> http://github.com/daviddoria/daviddoria-vtk/commit/687ed4819ae61e169ca016e3442e9f778b0ef5d9
>> and http://github.com/daviddoria/daviddoria-vtk/commit/1671de6acf35a3d1cb4cd44f1fdb481b50e17e3e)
>>
>> As for "change the shape of the mouse cursor or draw a horizontal
>> arrow somewhere." - it may be unnecessary overhead to force a text
>> actor or some other orientation marker into the scene. It really isn't
>> that hard to keep track of mentally :)
>>
>> @Jim - just as above, I didn't want to force an orientationWidget to
>> be present to allow for generic use of this.
>>
>> Thanks,
>>
>> David
>>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the VTK FAQ at: http://www.vtk.org/Wiki/VTK_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtkusers
>



More information about the vtkusers mailing list