make disabled install button and purchasing setup more obvious (bug 7…

…51710)
mozilla · May 4, 2012 · 97f0d3c · 97f0d3c
1 parent 52fa4a0
commit 97f0d3c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 14 deletions.
diff --git a/mkt/detail/templates/detail/app.html b/mkt/detail/templates/detail/app.html
@@ -72,17 +72,7 @@ <h3 class="hidden">{{ _('Categories') }}</h3>
               {{ _('Purchased') }}</span>
           {% elif product.can_purchase() %}
             {% if user_data(amo_user)['pre_auth'] %}
-              <span class="approval checkmark">{{ _('PayPal pre-approval') }}
-                </span>
-            {% else %}
-              <form method="post" id="preapproval-shortcut"
-                    action="{{ product.get_detail_url('purchase.preapproval') }}">
-                {{ csrf() }}
-                <input type="hidden" name="currency" value="USD">
-                <a class="approval-pitch" href="#">
-                  {{ _('Set up purchasing') }}
-                </a>
-              </form>
+              <span class="approval checkmark">{{ _('PayPal pre-approval') }}</span>
             {% endif %}
           {% endif %}
         {% endif %}

diff --git a/mkt/detail/tests/test_views.py b/mkt/detail/tests/test_views.py
@@ -276,7 +276,7 @@ def test_paid_no_preapproval_authenticated(self):
                                  password='password')
         self.make_premium(self.webapp)
         doc = self.get_pq()
-        eq_(doc('.approval-pitch').length, 1)
+        eq_(doc('.approval-pitch').length, 0)
         eq_(doc('.approval.checkmark').length, 0)
 
     def test_paid_preapproval_enabled(self):

diff --git a/mkt/templates/mkt/base.html b/mkt/templates/mkt/base.html
@@ -98,9 +98,9 @@
     {% include 'purchase/message.html' %}
 
     <script type="text/template" id="noApps">
-      <div class="noApps">You need a
+      <div class="noApps">You need to be using
       <a href="https://wiki.mozilla.org/Marketplace/Mozillian_Preview">
-      supported browser</a> to install this app.</div>
+      Firefox Nightly</a> to install this app.</div>
     </script>
 
     {% if not logged %}