summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIvan Kohler <ivan@freeside.biz>2016-12-14 09:36:37 -0800
committerIvan Kohler <ivan@freeside.biz>2016-12-14 09:36:37 -0800
commit9d5f61a78f7cddeb65f6943f3e58326f47cc0c0b (patch)
tree077cc77dd4077c5de6760bb83aa72c70952eebf5
parent0af87aaed13a3e878946517f86ef11efb66c5c70 (diff)
add double-click prevention to discount popup/form, RT#73799, and clarify labels
-rwxr-xr-xhttemplate/edit/cust_pkg_discount.html10
-rw-r--r--httemplate/elements/tr-select-pkg-discount.html4
2 files changed, 10 insertions, 4 deletions
diff --git a/httemplate/edit/cust_pkg_discount.html b/httemplate/edit/cust_pkg_discount.html
index e1e3dae..79c3478 100755
--- a/httemplate/edit/cust_pkg_discount.html
+++ b/httemplate/edit/cust_pkg_discount.html
@@ -1,13 +1,17 @@
<& /elements/header-popup.html, "Discount Package" &>
<& /elements/error.html &>
-<FORM NAME="DiscountPkgForm" ACTION="<% $p %>edit/process/cust_pkg_discount.html" METHOD=POST>
+<FORM NAME = "DiscountPkgForm"
+ ACTION = "<% $p %>edit/process/cust_pkg_discount.html"
+ METHOD = POST
+ onSubmit = "document.DiscountPkgForm.submit.disabled=true;"
+>
<INPUT TYPE="hidden" NAME="pkgnum" VALUE="<% $pkgnum %>">
<% ntable('#cccccc') %>
<TR>
- <TH ALIGN="right">Current package&nbsp;</TH>
+ <TH ALIGN="right">Package&nbsp;</TH>
<TD COLSPAN=7>
<% $curuser->option('show_pkgnum') ? $cust_pkg->pkgnum.': ' : '' %><B><% $part_pkg->pkg |h %></B> - <% $part_pkg->comment |h %>
</TD>
@@ -18,6 +22,8 @@
curr_value_recur => $recur_discountnum,
disable_setup => $disable_setup,
disable_recur => $disable_recur,
+ setup_label => emt('Setup fee discount'),
+ recur_label => emt('Recurring fee discount'),
&>
</TABLE>
diff --git a/httemplate/elements/tr-select-pkg-discount.html b/httemplate/elements/tr-select-pkg-discount.html
index dc38cff..0c57fd8 100644
--- a/httemplate/elements/tr-select-pkg-discount.html
+++ b/httemplate/elements/tr-select-pkg-discount.html
@@ -34,7 +34,7 @@ description if curr_value_setup is set. Likewise "disable_recur".
% if ( $curuser->access_right('Waive setup fee') ) {
% push @$pre_options, -2 => 'Waive setup fee';
% }
-<& tr-td-label.html, label => emt('Setup fee') &>
+<& tr-td-label.html, label => $opt{setup_label} || emt('Setup fee') &>
<td>
<& select-discount.html,
field => 'setup_discountnum',
@@ -97,7 +97,7 @@ description if curr_value_setup is set. Likewise "disable_recur".
% if ( $curuser->access_right('Discount customer package')
% and !$opt{disable_recur} ) {
-<& tr-td-label.html, label => emt('Recurring fee') &>
+<& tr-td-label.html, label => $opt{recur_label} || emt('Recurring fee') &>
<td>
<& select-discount.html,
field => 'recur_discountnum',