[ 
https://issues.apache.org/jira/browse/PIVOT-959?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14167527#comment-14167527
 ] 

Roger Whitcomb commented on PIVOT-959:
--------------------------------------

In looking through the code, it seems like TerraTextInputSkin is the only class 
that does this (call "super.keyPressed" twice).  The "consumed" value from the 
first call is never used, and in fact, is set by each of the "if" cases, and 
"else" sets it by calling "super.keyPressed" again.  So, I agree with 
Alessandro's analysis, and believe that we should take out the first call.

If there would be problems it would be with the keys that are processed in this 
skin method itself (which would be DELETE, BACKSPACE, HOME, END, arrow keys, 
etc.).  The only problem I could envision would be with TAB (which should 
traverse fields in a container) but which is never processed by TextInput.  So, 
I believe this fix would be completely safe.

> TerraTextInputSkin may process the same key press twice
> -------------------------------------------------------
>
>                 Key: PIVOT-959
>                 URL: https://issues.apache.org/jira/browse/PIVOT-959
>             Project: Pivot
>          Issue Type: Bug
>          Components: wtk-terra
>    Affects Versions: 2.0.4
>            Reporter: Alessandro Pira
>            Priority: Minor
>             Fix For: 2.1, 2.0.5
>
>
> TerraTextInputSkin's keyPress method may call super.keyPress twice (once as 
> its first instruction, and then again in the "else" branch).
> I suggest to remove the first ancestor call.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to