ICEfaces
  1. ICEfaces
  2. ICE-6316

Regression: ICE-2368 Autoposition panel not working correctly

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0.0
          • Fix Version/s: 2.0.0
          • Component/s: None
          • Labels:
            None
          • Environment:
            Icefaces2 trunk revision# 23251
            Server: tomcat6
            Browser: IE7 & Opera10.10
          • Assignee Priority:
            P1

            Description

            The autoposition popup panel position cannot be changed dynamically on IE & Opera browsers only.
            Testcase available at "C:\repo\qa\trunk\Regression-Icefaces2\Nightly\ICE-2368"

              Activity

              Ascending order - Click to sort in descending order
              Mandeep Hayher created issue -
              Mandeep Hayher made changes -
              Field Original Value New Value
              Assignee Ken Fyten [ ken.fyten ]
              Mandeep Hayher made changes -
              Salesforce Case []
              Description The autoposition popup panel position cannot be changed dynamically on IE & Opera browsers only The autoposition popup panel position cannot be changed dynamically on IE & Opera browsers only.
              Testcase available at "C:\repo\qa\trunk\Regression-Icefaces2\Nightly\ICE-2368"
              Hide
              Permalink
              Ken Fyten added a comment -

              Another possible regression from recent logic changes to popup panel JS.

              I've asked Mandeep to add specific instructions to the JIRA, but they might already be in the test case app.

              Show
              Ken Fyten added a comment - Another possible regression from recent logic changes to popup panel JS. I've asked Mandeep to add specific instructions to the JIRA, but they might already be in the test case app.
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 2.0.0 [ 10230 ]
              Assignee Priority P2
              Assignee Ken Fyten [ ken.fyten ] Mircea Toma [ mircea.toma ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P2 P1
              Hide
              Permalink
              Mandeep Hayher added a comment -

              Icefaces2 revision# 23501
              Server: Tomca6
              With deltaSubmit=true

              AutoCenter panel does not work on any browser, on FF & Opera it works only once but on IE it never works.

              Steps:
              1)check the autocenterPanel checkbox
              2)A popup panle should be displayed
              3) drag and drop the popup panel a couple times to reproduce the issue

              Show
              Mandeep Hayher added a comment - Icefaces2 revision# 23501 Server: Tomca6 With deltaSubmit=true AutoCenter panel does not work on any browser, on FF & Opera it works only once but on IE it never works. Steps: 1)check the autocenterPanel checkbox 2)A popup panle should be displayed 3) drag and drop the popup panel a couple times to reproduce the issue
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #23579 Thu Dec 16 14:19:51 MST 2010 mircea.toma ICE-6316 Add direct parameter to the parameter map only when it does not participate also in parameter diffing.
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/DeltaSubmitPhaseListener.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #23583 Thu Dec 16 14:48:45 MST 2010 mircea.toma ICE-6316 Introduce constants.
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/DeltaSubmitPhaseListener.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #23584 Thu Dec 16 14:49:44 MST 2010 mircea.toma ICE-6316 Fix typo.
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/DeltaSubmitPhaseListener.java
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              Drag and drop form submit always sends an additional icefacesCSSUpdate='' parameter. Due to ICE-5913 fixes al forms, including the hidden form used for delta submit, gets this hidden input field automatically.

              The solution does not remove the source of this issue since that would mean reverting ICE-5913 fixes. Instead a test was added in DeltaSubmitPhaseListener to make sure that the direct parameters are added to the parameter map only when they do not participate also in parameter diffing.

              Show
              Mircea Toma added a comment - - edited Drag and drop form submit always sends an additional icefacesCSSUpdate='' parameter. Due to ICE-5913 fixes al forms, including the hidden form used for delta submit, gets this hidden input field automatically. The solution does not remove the source of this issue since that would mean reverting ICE-5913 fixes. Instead a test was added in DeltaSubmitPhaseListener to make sure that the direct parameters are added to the parameter map only when they do not participate also in parameter diffing.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Mandeep Hayher added a comment - - edited

              Icefaces2 revision# 23643
              Server: Tomcat6
              Browser: Opera10.60 & IE8

              AutoCenterPanel now works fine on all browser but AutoPositionPanel stil has problem.

              The autoposition popup panel position cannot be changed dynamically on IE & Opera browsers only.
              1) load application, check AutoPositionPanel to display is on screen
              2) change autoPositionInput value and press enter.
              Notice the position of autoPositionPanel is not updated
              3) now drag and drop the autoPositionPanel to anywhere
              Notice the autoPositionPanel position is adjusted to match the autoPositionInput values.

              New jira created 'ICE-6410'

              Show
              Mandeep Hayher added a comment - - edited Icefaces2 revision# 23643 Server: Tomcat6 Browser: Opera10.60 & IE8 AutoCenterPanel now works fine on all browser but AutoPositionPanel stil has problem. The autoposition popup panel position cannot be changed dynamically on IE & Opera browsers only. 1) load application, check AutoPositionPanel to display is on screen 2) change autoPositionInput value and press enter. Notice the position of autoPositionPanel is not updated 3) now drag and drop the autoPositionPanel to anywhere Notice the autoPositionPanel position is adjusted to match the autoPositionInput values. New jira created ' ICE-6410 '
              Mandeep Hayher made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Mandeep Hayher made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Mandeep Hayher made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Mandeep Hayher
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: