changeset 3985:d46d9d93305a

diff: drop "confirm to show potentially huge diff" click-through It is obvious from the context that the diff was cut off to use resources efficiently. The user clicked "Show full diff" because he wanted the diff anyway. Asking twice is just annoying and serves no purpose.
author Mads Kiilerich <madski@unity3d.com>
date Wed, 12 Jun 2013 01:36:35 +0200
parents 425b9eb67d3a
children 6aa83fecf669
files rhodecode/templates/compare/compare_diff.html
diffstat 1 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/rhodecode/templates/compare/compare_diff.html	Tue Jun 04 14:01:46 2013 +0200
+++ b/rhodecode/templates/compare/compare_diff.html	Wed Jun 12 01:36:35 2013 +0200
@@ -55,7 +55,7 @@
               %endfor
             </div>
             % if c.limited_diff:
-              <h5>${_('Changeset was too big and was cut off...')} <a href="${h.url.current(fulldiff=1, **request.GET.mixed())}" onclick="return confirm('${_("confirm to show potentially huge diff")}')">${_('Show full diff')}</a></h5>
+              <h5>${_('Changeset was too big and was cut off...')} <a href="${h.url.current(fulldiff=1, **request.GET.mixed())}">${_('Show full diff')}</a></h5>
             % endif
         </div>
     </div>
@@ -66,7 +66,7 @@
       ${diff_block.diff_block_simple([c.changes[fid]])}
     %endfor
     % if c.limited_diff:
-      <h4>${_('Changeset was too big and was cut off...')} <a href="${h.url.current(fulldiff=1, **request.GET.mixed())}" onclick="return confirm('${_("confirm to show potentially huge diff")}')">${_('Show full diff')}</a></h4>
+      <h4>${_('Changeset was too big and was cut off...')} <a href="${h.url.current(fulldiff=1, **request.GET.mixed())}">${_('Show full diff')}</a></h4>
     % endif
      <script type="text/javascript">