ICEfaces
  1. ICEfaces
  2. ICE-3120

Modal popups visible property is reset after being dragged

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.7
          • Fix Version/s: 1.7.1
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            A modal popup is set to draggable and is displayed on load of a page. On the popup there is a command button that will hide the popup when clicked by setting its visible property to false. This works fine unless the popup is dragged. This seems to be setting the visible property to true on all submits. If for example the popup is part of a panelTab navigating to another tab and then back to the first tab will show the popup when it should of been set to false from closing the popup. This behavior is also replicated in navigating to another page when the popup is part of a facelets template. In this case if the popup is not dragged then the popup does not re-appear, but if the popup is dragged it re-appears on every page that is navigated to. I have been able to reproduce this issue in a facelets example and a non-facelets example.
            1. File
              main.jspx
              3 kB
              Arran McCullough
            2. Java Source File
              TestBean.java
              0.6 kB
              Arran McCullough

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arran McCullough added a comment -

              Debugging of the setter/getter for the visible object:

              Without being dragged:
              Page load:
              isPopupVisible:true
              isPopupVisible:true
              isPopupVisible:true
              Click OK button (hide popup)
              isPopupVisible:false
              isPopupVisible:false
              isPopupVisible:false
              Click on new tab:
              isPopupVisible:false
              isPopupVisible:false
              isPopupVisible:false

              With dragging popup:
              Page load:
              isPopupVisible:true
              isPopupVisible:true
              isPopupVisible:true
              Dragged popup:
              setPopupVisible:true
              isPopupVisible:true
              isPopupVisible:true
              isPopupVisible:true
              Click OK button (hide popup):
              setPopupVisible:true
              isPopupVisible:false
              isPopupVisible:false
              isPopupVisible:false
              Click on new tab:
              setPopupVisible:true
              isPopupVisible:false
              isPopupVisible:false
              isPopupVisible:false
              Click back to first tab:
              setPopupVisible:true
              isPopupVisible:true
              isPopupVisible:true
              isPopupVisible:true

              Show
              Arran McCullough added a comment - Debugging of the setter/getter for the visible object: Without being dragged: Page load: isPopupVisible:true isPopupVisible:true isPopupVisible:true Click OK button (hide popup) isPopupVisible:false isPopupVisible:false isPopupVisible:false Click on new tab: isPopupVisible:false isPopupVisible:false isPopupVisible:false With dragging popup: Page load: isPopupVisible:true isPopupVisible:true isPopupVisible:true Dragged popup: setPopupVisible:true isPopupVisible:true isPopupVisible:true isPopupVisible:true Click OK button (hide popup): setPopupVisible:true isPopupVisible:false isPopupVisible:false isPopupVisible:false Click on new tab: setPopupVisible:true isPopupVisible:false isPopupVisible:false isPopupVisible:false Click back to first tab: setPopupVisible:true isPopupVisible:true isPopupVisible:true isPopupVisible:true
              Hide
              Permalink
              Arran McCullough added a comment -

              Sample code to duplicate issue

              Show
              Arran McCullough added a comment - Sample code to duplicate issue
              Hide
              Permalink
              Arran McCullough added a comment -

              Sample backing bean code to duplicate issue

              Show
              Arran McCullough added a comment - Sample backing bean code to duplicate issue
              Hide
              Permalink
              Adnan Durrani added a comment -

              trunk : revision 16790
              branch 1.7 : revision 16791

              Show
              Adnan Durrani added a comment - trunk : revision 16790 branch 1.7 : revision 16791

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Arran McCullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: