mozreview: remove the 'New Review Request' link (bug 1247595) r?mdoglio draft
authorbyron jones <glob@mozilla.com>
Thu, 11 Feb 2016 22:42:18 +0800
changeset 7225 952a9ebd92594845ed7b1ac31d8b5667925b8b1f
parent 7223 0ba15a1bf1678f759e3754c9670b59c82cbbce53
push id620
push userbjones@mozilla.com
push dateThu, 11 Feb 2016 14:44:33 +0000
reviewersmdoglio
bugs1247595
mozreview: remove the 'New Review Request' link (bug 1247595) r?mdoglio The 'New Review Request' link in the UI doesn't work, yet it's a very attractive target for new Review Board users. This hides the link. MozReview-Commit-ID: Ij5TD2AtIbf
pylib/mozreview/mozreview/extension.py
pylib/mozreview/mozreview/static/mozreview/js/default.js
pylib/mozreview/mozreview/static/mozreview/js/logout.js
--- a/pylib/mozreview/mozreview/extension.py
+++ b/pylib/mozreview/mozreview/extension.py
@@ -122,17 +122,17 @@ class MozReviewExtension(Extension):
             'source_filenames': ['mozreview/css/viewdiff.less'],
         },
         'admin': {
             'source_filenames': ['mozreview/css/admin.less'],
         },
     }
     js_bundles = {
         'default': {
-            'source_filenames': ['mozreview/js/logout.js'],
+            'source_filenames': ['mozreview/js/default.js'],
         },
         'filediffreviewer': {
             'source_filenames': ['mozreview/js/models/filediffreviewermodel.js',
                                  'mozreview/js/collections/filediffreviewercollection.js',
                                  'mozreview/js/init_filediffreviewer.js',
                                  'mozreview/js/diffviewer_customizations.js'],
             'apply_to': diffviewer_url_names,
         },
rename from pylib/mozreview/mozreview/static/mozreview/js/logout.js
rename to pylib/mozreview/mozreview/static/mozreview/js/default.js
--- a/pylib/mozreview/mozreview/static/mozreview/js/logout.js
+++ b/pylib/mozreview/mozreview/static/mozreview/js/default.js
@@ -1,4 +1,8 @@
 $(document).ready(function() {
+  // Hide "New Review Request" link
+  $('#navbar a[href="/r/new/"]').parent('li').hide();
+
+  // Fix logout url
   $("a[href='/account/logout/']").attr("href", "/mozreview/logout/");
   $("#accountnav li").css("visibility", "visible");
 });