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
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached sample source code.

              Show
              Arran Mccullough added a comment - Attached sample source code.
              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.
              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.
              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.*.
              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.
              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.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: