ICEfaces
  1. ICEfaces
  2. ICE-10313

Support "order of precedence" when multiple ace:submitMonitors active concurrently.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 4, ace:submitMonitor

      Description

      There is an opportunity to improve the sophistication of the ace:submitMonitor in cases where multiple monitors are defined on the same page with overlapping scopes.

      For example, an application may want to use a general monitor that activates for any submit on the page, blocks the entire page, and presents a general busy indicator while the submit is in progress. In addition, the application may also want to provide a more specific monitor on say, a particular form on the page ("ClaimForm"). In that case, if the "ClaimForm" submits, it's monitor would block only the form and present a custom message (e.g. "Submitting Claim...").

      Currently, both monitors would be triggered by the submit of the "ClaimForm", which may be undesirable. A more sophisticated approach would have a submitMonitor with a more specific scope override any others with more general scopes. That way, multiple monitors could be setup on a page and only one of them would be activated by a given submit.

        Activity

        Ken Fyten created issue -
        Ken Fyten made changes -
        Field Original Value New Value
        Fix Version/s 4.1 [ 11375 ]
        Affects Documentation (User Guide, Ref. Guide, etc.),Sample App./Tutorial,Compatibility/Configuration [ 10003, 10001, 10002 ]
        Arran Mccullough made changes -
        Support Case References Support Case #12955 - https://icesoft.my.salesforce.com/5007000000edYKn
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA_P03 [ 11572 ]
        Ken Fyten made changes -
        Assignee Mircea Toma [ mircea.toma ]
        Hide
        Mircea Toma added a comment -

        After the fixes made for ICE-10284 I believe the component can be already setup to work as described above. See comment: http://jira.icesoft.org/browse/ICE-10284?focusedCommentId=55341&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-55341

        Show
        Mircea Toma added a comment - After the fixes made for ICE-10284 I believe the component can be already setup to work as described above. See comment: http://jira.icesoft.org/browse/ICE-10284?focusedCommentId=55341&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-55341
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]
        Hide
        Ken Fyten added a comment - - edited

        Re-opened.

        It would still be a useful improvement to have the ace:submitMonitor be able support multiple overlapping monitors with different levels of specificity in the page, while avoiding the possibility of having multiple monitors being displayed simultaneously.

        For example, if the entire body of the document had a default submit monitor defined that presented a "Please Wait..." message, you could also define additional submit monitors on specific components in the page with more specific messages, such as on a form in a panel to add a new record to a database, which might present a message that says "Adding new Employee...", etc. So, it would be preferable if the submitMonitor was able to determine which instance of itself was most local to the component triggering the submit in the component tree and only present that monitor for any given submit.

        Show
        Ken Fyten added a comment - - edited Re-opened. It would still be a useful improvement to have the ace:submitMonitor be able support multiple overlapping monitors with different levels of specificity in the page, while avoiding the possibility of having multiple monitors being displayed simultaneously. For example, if the entire body of the document had a default submit monitor defined that presented a "Please Wait..." message, you could also define additional submit monitors on specific components in the page with more specific messages, such as on a form in a panel to add a new record to a database, which might present a message that says "Adding new Employee...", etc. So, it would be preferable if the submitMonitor was able to determine which instance of itself was most local to the component triggering the submit in the component tree and only present that monitor for any given submit.
        Ken Fyten made changes -
        Resolution Invalid [ 6 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43903 Tue Jan 13 16:54:49 MST 2015 mircea.toma ICE-10313 Implemented order of precedence for the submit monitors.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/submitmonitor/submitmonitor.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43904 Tue Jan 13 16:56:54 MST 2015 mircea.toma ICE-10313 Fix comment.
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/submitmonitor/submitmonitor.js
        Hide
        Mircea Toma added a comment -

        Implemented the order of precedence for the monitors by executing only the monitor that is the closest (in DOM hierarchy) to the element that triggered the submit.

        Show
        Mircea Toma added a comment - Implemented the order of precedence for the monitors by executing only the monitor that is the closest (in DOM hierarchy) to the element that triggered the submit.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: