From: Dave Greenberg <dave@civicrm.org>
Date: Fri, 20 Jun 2014 23:20:45 +0000 (-0700)
Subject: CRM-14895 - Do not load Schedule Reminders form in pop-up to prevent scrollbar jumpin... 
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=8f0332b019d5010f9c1a6a38a69b30503c1e5c29;p=civicrm-core.git

CRM-14895 - Do not load Schedule Reminders form in pop-up to prevent scrollbar jumping and weird behavior in token inserts.

----------------------------------------
* CRM-14895:
  https://issues.civicrm.org/jira/browse/CRM-14895
---

diff --git a/CRM/Admin/Page/ScheduleReminders.php b/CRM/Admin/Page/ScheduleReminders.php
index 4f375464d2..d93f58954e 100644
--- a/CRM/Admin/Page/ScheduleReminders.php
+++ b/CRM/Admin/Page/ScheduleReminders.php
@@ -72,6 +72,7 @@ class CRM_Admin_Page_ScheduleReminders extends CRM_Core_Page_Basic {
           'url' => 'civicrm/admin/scheduleReminders',
           'qs' => 'action=update&id=%%id%%&reset=1',
           'title' => ts('Edit Schedule Reminders'),
+          'class' => 'no-popup',
         ),
         CRM_Core_Action::ENABLE => array(
           'name' => ts('Enable'),
diff --git a/templates/CRM/Admin/Page/ScheduleReminders.tpl b/templates/CRM/Admin/Page/ScheduleReminders.tpl
index d6e5e7a296..234011941d 100644
--- a/templates/CRM/Admin/Page/ScheduleReminders.tpl
+++ b/templates/CRM/Admin/Page/ScheduleReminders.tpl
@@ -39,7 +39,7 @@
     <div id="reminder">
       {include file="CRM/Admin/Page/Reminders.tpl"}
       <div class="action-link">
-        <a href="{crmURL q="action=add&reset=1"}" id="newScheduleReminder" class="button"><span><div class="icon add-icon"></div>{ts}Add Reminder{/ts}</span></a>
+        <a href="{crmURL q="action=add&reset=1"}" id="newScheduleReminder" class="button no-popup"><span><div class="icon add-icon"></div>{ts}Add Reminder{/ts}</span></a>
       </div>
 
     </div>
@@ -47,7 +47,7 @@
 {else}
     <div class="messages status no-popup">
       <div class="icon inform-icon"></div>
-        {capture assign=crmURL}href="{crmURL p='civicrm/admin/scheduleReminders' q="action=add&reset=1"}" class="action-item"{/capture}
+        {capture assign=crmURL}class="no-popup" href="{crmURL p='civicrm/admin/scheduleReminders' q="action=add&reset=1"}" class="action-item"{/capture}
         {ts 1=$crmURL}There are no Scheduled Reminders configured. You can <a %1>add one</a>.{/ts}
     </div>
 {/if}