From 8f0332b019d5010f9c1a6a38a69b30503c1e5c29 Mon Sep 17 00:00:00 2001
From: Dave Greenberg <dave@civicrm.org>
Date: Fri, 20 Jun 2014 16:20:45 -0700
Subject: [PATCH] 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
---
 CRM/Admin/Page/ScheduleReminders.php           | 1 +
 templates/CRM/Admin/Page/ScheduleReminders.tpl | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

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}
-- 
2.25.1