ICEfaces
  1. ICEfaces
  2. ICE-10976

icecore:idleMonitor - Modal background shown when using modal ace:dialog

          Details

          • Support Case References:
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Workaround is to use the "ice.onUserInactivity(interval, idleCallback, activeCallback)" JS callback instead of the <icecore:idleMonitor> component. Using the JS callback allows the application to display an ace:dialog correctly via client-side JS when the idle event fires.
            Show
            Workaround is to use the "ice.onUserInactivity(interval, idleCallback, activeCallback)" JS callback instead of the <icecore:idleMonitor> component. Using the JS callback allows the application to display an ace:dialog correctly via client-side JS when the idle event fires.

            Description

            An ace:dialog is used in an icecore:idleMonitor to be shown when the idle interval is reached. The issue is if modal=true on the ace:dialog, the modal background is rendered even without the idle interval being reached.

            Sample code:
                        <icecore:idleMonitor interval="50" userActiveEvents="click">
                            <ace:dialog visible="true" modal="true"
                                        closable="false">
                                <h5 id="idleMonitor" style="color: red;">Session Will Expire in 10 Seconds!</h5>
                                <h:commandButton value="OK"/>
                            </ace:dialog>
                        </icecore:idleMonitor>

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached sample source code.

              Show
              Arran Mccullough added a comment - Attached sample source code.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case13589Example.zip [ 22095 ]
              Arran Mccullough made changes -
              Attachment Case13589Example.war [ 22096 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Will consider as an improvement for icecore:idleMonitor, but for now the recommendation is to use the workaround approach.

              Show
              Ken Fyten added a comment - Will consider as an improvement for icecore:idleMonitor, but for now the recommendation is to use the workaround approach.
              Ken Fyten made changes -
              Workaround Description Workaround is to use the "ice.onUserInactivity(interval, idleCallback, activeCallback)" JS callback instead of the <icecore:idleMonitor> component. Using the JS callback allows the application to display an ace:dialog correctly via client-side JS when the idle event fires.
              Fix Version/s 4.2 [ 12870 ]
              Priority Major [ 3 ] Minor [ 4 ]
              Workaround Exists Yes [ 10007 ]
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Hide
              Permalink
              Mircea Toma added a comment -

              Render dialog overlay as sibling of the dialog component root element. Then whenever the dialog is hidden its overlay will be hidden as well.

              Show
              Mircea Toma added a comment - Render dialog overlay as sibling of the dialog component root element. Then whenever the dialog is hidden its overlay will be hidden as well.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #49100 Tue Aug 09 06:42:44 MDT 2016 mircea.toma ICE-10976 Render dialog overlay as sibling of the dialog component root element.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/dialog/dialog.css
              Hide
              Permalink
              Carmen Cristurean added a comment - - edited

              Verified ICEfaces4 trunk r.49101 using attached test case, on IE8, IE11, MS Edge, FF41/47, Chrome52; ran icecore:idleMonitor regressions on FF47.
              Test case has been added to idleMonitor QA test app:
              http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/idleMonitor

              This issue is still reproducible with EE-3.3.0-maintenance branch r.49101, so perhaps the fix must be ported to the maintenance branch?

              Show
              Carmen Cristurean added a comment - - edited Verified ICEfaces4 trunk r.49101 using attached test case, on IE8, IE11, MS Edge, FF41/47, Chrome52; ran icecore:idleMonitor regressions on FF47. Test case has been added to idleMonitor QA test app: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/idleMonitor This issue is still reproducible with EE-3.3.0-maintenance branch r.49101, so perhaps the fix must be ported to the maintenance branch?
              Hide
              Permalink
              Mircea Toma added a comment -

              It can be ported back, yes. It 's just that the issue wasn't targeted for 3.3.*.

              Show
              Mircea Toma added a comment - It can be ported back, yes. It 's just that the issue wasn't targeted for 3.3.*.
              Mircea Toma made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Arran Mccullough made changes -
              Fix Version/s EE-3.3.0.GA_P05 [ 13082 ]
              Hide
              Permalink
              Mircea Toma added a comment -

              Ported back fix to maintenance branch.

              Show
              Mircea Toma added a comment - Ported back fix to maintenance branch.
              Mircea Toma made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified EE-3.3.0-maintenance branch r.49104 on IE11, IE8, MS Edge, FF47, Chrome52.

              Show
              Carmen Cristurean added a comment - Verified EE-3.3.0-maintenance branch r.49104 on IE11, IE8, MS Edge, FF47, Chrome52.
              Ken Fyten made changes -
              Fix Version/s 4.2.BETA [ 13091 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: