You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by de...@apache.org on 2017/09/17 12:30:06 UTC

[myfaces-trinidad] 22/33: TRINIDAD-2289 - function _pprControlCapture() causes an error in IE8 when it tries to focus on a PPR'd element

This is an automated email from the ASF dual-hosted git repository.

deki pushed a commit to branch 2.0.0.x-branch
in repository https://gitbox.apache.org/repos/asf/myfaces-trinidad.git

commit bcbf53fbf2b3fbac96f67a4589c39ecdc976a7b7
Author: Scott Bryan <so...@apache.org>
AuthorDate: Tue Jul 31 21:06:37 2012 +0000

    TRINIDAD-2289 - function _pprControlCapture() causes an error in IE8 when it tries to focus on a PPR'd element
    
    * Added these changes to a one-off of 2.0.0.
    * Thanks for the patch Kentaro
---
 .../src/main/javascript/META-INF/adf/jsLibs/Core.js         | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js b/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js
index 2d6dab7..6fc6ee4 100644
--- a/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js
+++ b/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js
@@ -3717,7 +3717,18 @@ function _pprControlCapture(win, set)
 
         // return focus to the post-PPR target element
         if (win._pprEventElement)
-          win._pprEventElement.focus();
+        {
+          // focus() should be in a try/catch block because if the element
+          // was PPR'd then IE8 will throw an error
+          try
+          {
+            win._pprEventElement.focus();
+          }
+          catch(e) 
+          {
+            // do nothing
+          }
+        }
         body.style.cursor = win._pprSavedCursor;
         win._pprSavedCursor = null;
         win._pprSavedCursorFlag = false;

-- 
To stop receiving notification emails like this one, please contact
"commits@myfaces.apache.org" <co...@myfaces.apache.org>.