ICEfaces
  1. ICEfaces
  2. ICE-8665

ace:dateTimeEntry text input key-events causing slowdown with large DOMs in IE7/8. (was: Add "onblur" client event to ace:dateTimeEntry)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.BETA1
    • Fix Version/s: 3.2
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      -

      Description

      The ace:dateTimeEntry registers some key event listeners on the text input element (when in popup mode). These listeners are causing noticeable slowdown in browser responsiveness when used with a very large DOM in IE 7/8.

        Activity

        Migration created issue -
        Hide
        Tyler Johnson added a comment - - edited

        sf-110694.war attached with source. Ready for deployment on Tomcat 7.

        Show
        Tyler Johnson added a comment - - edited sf-110694.war attached with source. Ready for deployment on Tomcat 7.
        Hide
        yip.ng added a comment - - edited

        SEVERE: Critical error during deployment:
        java.lang.NoSuchMethodError: org.icefaces.util.EnvUtils.isIncludeScrollOffsets(Ljavax/faces/context/FacesContext;)Z
        at com.icesoft.faces.application.ExtrasSetup.<init>(ExtrasSetup.java:62)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39)
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
        at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
        at java.lang.Class.newInstance0(Class.java:355)
        at java.lang.Class.newInstance(Class.java:308)
        at com.sun.faces.config.processor.AbstractConfigProcessor.createInstance(AbstractConfigProcessor.java:268)
        at com.sun.faces.config.processor.ApplicationConfigProcessor.addSystemEventListener(ApplicationConfigProcessor.java:804)
        at com.sun.faces.config.processor.ApplicationConfigProcessor.process(ApplicationConfigProcessor.java:315)
        at com.sun.faces.config.processor.AbstractConfigProcessor.invokeNext(AbstractConfigProcessor.java:114)
        at com.sun.faces.config.processor.LifecycleConfigProcessor.process(LifecycleConfigProcessor.java:116)
        at com.sun.faces.config.processor.AbstractConfigProcessor.invokeNext(AbstractConfigProcessor.java:114)
        at com.sun.faces.config.processor.FactoryConfigProcessor.process(FactoryConfigProcessor.java:222)
        at com.sun.faces.config.ConfigManager.initialize(ConfigManager.java:360)
        at com.sun.faces.config.ConfigureListener.contextInitialized(ConfigureListener.java:225)
        at org.apache.catalina.core.StandardContext.listenerStart(StandardContext.java:4723)
        at org.apache.catalina.core.StandardContext$1.call(StandardContext.java:5226)
        at org.apache.catalina.core.StandardContext$1.call(StandardContext.java:5221)
        at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
        at java.util.concurrent.FutureTask.run(FutureTask.java:138)
        at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
        at java.lang.Thread.run(Thread.java:662)

        Show
        yip.ng added a comment - - edited SEVERE: Critical error during deployment: java.lang.NoSuchMethodError: org.icefaces.util.EnvUtils.isIncludeScrollOffsets(Ljavax/faces/context/FacesContext;)Z at com.icesoft.faces.application.ExtrasSetup.<init>(ExtrasSetup.java:62) at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39) at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27) at java.lang.reflect.Constructor.newInstance(Constructor.java:513) at java.lang.Class.newInstance0(Class.java:355) at java.lang.Class.newInstance(Class.java:308) at com.sun.faces.config.processor.AbstractConfigProcessor.createInstance(AbstractConfigProcessor.java:268) at com.sun.faces.config.processor.ApplicationConfigProcessor.addSystemEventListener(ApplicationConfigProcessor.java:804) at com.sun.faces.config.processor.ApplicationConfigProcessor.process(ApplicationConfigProcessor.java:315) at com.sun.faces.config.processor.AbstractConfigProcessor.invokeNext(AbstractConfigProcessor.java:114) at com.sun.faces.config.processor.LifecycleConfigProcessor.process(LifecycleConfigProcessor.java:116) at com.sun.faces.config.processor.AbstractConfigProcessor.invokeNext(AbstractConfigProcessor.java:114) at com.sun.faces.config.processor.FactoryConfigProcessor.process(FactoryConfigProcessor.java:222) at com.sun.faces.config.ConfigManager.initialize(ConfigManager.java:360) at com.sun.faces.config.ConfigureListener.contextInitialized(ConfigureListener.java:225) at org.apache.catalina.core.StandardContext.listenerStart(StandardContext.java:4723) at org.apache.catalina.core.StandardContext$1.call(StandardContext.java:5226) at org.apache.catalina.core.StandardContext$1.call(StandardContext.java:5221) at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303) at java.util.concurrent.FutureTask.run(FutureTask.java:138) at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908) at java.lang.Thread.run(Thread.java:662)
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #31578 Thu Oct 18 10:24:23 MDT 2012 yip.ng ICE-8665: Improve performance of keyboard input on text field. (Mainly for IE.)
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/readme.txt
        Hide
        Ken Fyten added a comment - - edited

        We've seen this type of thing before with IE7/8 and large DOMs, the browser cannot process DOM changes/layout updates in an efficient manner when the DOM is large.

        was:The ace:dateTimeEntry should have the onblur event available.

        Show
        Ken Fyten added a comment - - edited We've seen this type of thing before with IE7/8 and large DOMs, the browser cannot process DOM changes/layout updates in an efficient manner when the DOM is large. was:The ace:dateTimeEntry should have the onblur event available.
        Hide
        Ken Fyten added a comment - - edited

        New Regression:
        Showcase (Firefox 16/Tomcat 7):
        Manual Test Results:
        dateTimeEntry - Ajax Submit, Label Position, Indicator Text: after selecting new date in the popup calendar, the calendar closes and opens up again (not a PartialStateSavings specific issue).

        Show
        Ken Fyten added a comment - - edited New Regression: Showcase (Firefox 16/Tomcat 7): Manual Test Results: dateTimeEntry - Ajax Submit, Label Position, Indicator Text: after selecting new date in the popup calendar, the calendar closes and opens up again (not a PartialStateSavings specific issue).
        Hide
        yip.ng added a comment - - edited

        Same even if reverting the changes. Therefore a different issue.

        Show
        yip.ng added a comment - - edited Same even if reverting the changes. Therefore a different issue.
        Hide
        Ken Fyten added a comment - - edited

        Regression is due to change in ICE-8677. Marking this fixed again.

        Show
        Ken Fyten added a comment - - edited Regression is due to change in ICE-8677 . Marking this fixed again.
        Migration made changes -
        Field Original Value New Value
        Reporter Migration [ remote ] Tyler Johnson [ tyler.johnson ]
        Migration made changes -
        Fix Version/s 3.2 [ 10338 ]
        Migration made changes -
        Attachment sf-110694.war [ 14903 ]
        Migration made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Migration made changes -
        Summary Add "onblur" client event to ace:dateTimeEntry ace:dateTimeEntry text input key-events causing slowdown with large DOMs in IE7/8. (was: Add "onblur" client event to ace:dateTimeEntry)
        Description
        The ace:dateTimeEntry should have the onblur event available.
        The ace:dateTimeEntry registers some key event listeners on the text input element (when in popup mode). These listeners are causing noticeable slowdown in browser responsiveness when used with a very large DOM in IE 7/8.
        Migration made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Icefaces Administrator made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: