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

Werner Punz commented on MYFACES-3653:
--------------------------------------

Ok I also now added some fallback code for the universal case, which keeps the 
focus and caret position in place. The question arises now, why doing the copy 
properties method at all, the universal fallback code seems to work 
sufficiently for me for all modern browsers (and old ies down to ie7 quirks 
mode == ie6)
Mircea can you shed a light on this why you went that route instead of simply 
using document.activeElement and trying to retain the caret position? Maybe I 
have overlooked something.
For now I will keep both cases in the codebase, but I need further input on the 
patch why you went with copyproperties and retaining the old element.

                
> Partial update for input elements require special case
> ------------------------------------------------------
>
>                 Key: MYFACES-3653
>                 URL: https://issues.apache.org/jira/browse/MYFACES-3653
>             Project: MyFaces Core
>          Issue Type: Bug
>         Environment: any browser
>            Reporter: Mircea Toma
>            Assignee: Werner Punz
>         Attachments: ICE-8367.patch
>
>
> When focus is own by an input element and an incoming partial update will 
> update the element, the focus is removed from the element. Most often this 
> occurs when validation is run for each input field and input is cleared if 
> corresponding validation failed.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to