fix page title for apps detail page (bug 738088)

mozilla · Mar 22, 2012 · 8bfe003 · 8bfe003 · potch · Mar 22, 2012
1 parent a436ae4
commit 8bfe003
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/media/js/mkt/fragments.js b/media/js/mkt/fragments.js
@@ -21,6 +21,12 @@
             $.get(href, function(d) {
                 clearTimeout(timeout);
                 z.page.html(d).trigger('fragmentloaded');
+
+                // We so sneaky.
+                var $title = z.page.find('title');
+                document.title = $title.text();
+                $title.remove();
+
                 _.delay(function() { $loading.removeClass('active'); }, 400);
                 $('html, body').animate({scrollTop: 0}, 200);
             });

diff --git a/mkt/detail/templates/detail/app.html b/mkt/detail/templates/detail/app.html
@@ -1,5 +1,7 @@
 {% extends 'mkt/base.html' %}
 
+{% block title %}{{ mkt_page_title(product.name) }}{% endblock %}
+
 {% block content %}
   <section class="product-details">
     <div class="visual">

diff --git a/mkt/templates/mkt/fragment.html b/mkt/templates/mkt/fragment.html
@@ -1,2 +1,4 @@
+<title>{% block title %}{% endblock %}</title>
+
 {% block content %}
-{% endblock %}
+{% endblock %}