Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-11742

Apply FormNavigation across all (or almost all) forms

    Details

    • Type: Improvement
    • Status: Done/Fixed
    • Priority: Minor
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.3.0
    • Fix Version/s: 4.5
    • Component/s: Core CiviCRM
    • Labels:
      None
    • Documentation Required?:
      None

      Description

      We include a jQuery plugin called "formNavigate" which warns users about abandoning unsaved changes. Currently, this is included on an adhoc basis on ~36 tpl files. This seems like something that we should use on all (or almost all) our forms.

      Proposal:

      • Remove the 36+ references to formNavigate.tpl
      • In CRM_Core_Resources::addCoreResources(), add formNavigate.tpl to the page-footer of every page (with possible exception for public facing pages)
      • Modify formNavigate.tpl to bind to any ".crm-container" instead of the current
      • Modify jquery.formNavigate.js to ignore elements with a CSS class (like "crm-navigate-ignore")
      • Apply "crm-navigate-ignore" to the 3-4 spots which shouldn't use it

      Ticket initially assigned to Kurund for feedback. You can do it if you want or feel free to assign back to me.

        Attachments

          Activity

            People

            • Assignee:
              kurund Kurund Jalmi
              Reporter:
              timotten Tim Otten
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: