Merge pull request #415 from wlonk/say-rust-on-crate-page

Show the word "rust" on crate detail page

Author: alexcrichton

app/routes/crate.js

@@ -7,5 +7,22 @@ export default Ember.Route.extend({
                 return this.controllerFor('application').set('flashError', `Crate '${params.crate_id}' does not exist`);
             }
         });
+    },
+
+    afterModel(model) {
+        this.setHeadTags(model);
+    },
+
+    setHeadTags(model) {
+        var headTags = [{
+            type: 'meta',
+            tagId: 'meta-description-tag',
+            attrs: {
+                name: 'description',
+                content: model.get('description') || 'A package for Rust.'
+            }
+        }];
+
+        this.set('headTags', headTags);
     }
 });

app/routes/index.js

@@ -9,6 +9,7 @@ export default Ember.Route.extend({
             content: 'cargo is the package manager and crate host for rust'
         }
     }],
+
     model() {
         function addCrates(store, crates) {
             for (var i = 0; i < crates.length; i++) {

app/styles/app.scss

@@ -288,6 +288,13 @@ ul.dropdown {
     h1 { font-size: 1.5em; }
 }
 
+h1 {
+  span.subtitle {
+    display: block;
+    font-size: small;
+  }
+}
+
 #p404 {
     font-size: 120%;
     line-height: 3em;

app/templates/application.hbs

@@ -1,4 +1,4 @@
-{{title "Cargo" separator=' - ' prepend=true}}
+{{title "Cargo: packages for Rust" separator=' - ' prepend=true}}
 
 <a href='https://github.com/rust-lang/crates.io' class='fork-me'>
   <img src='/assets/forkme.png'/>
@@ -10,7 +10,10 @@
              height=100 width=100 />
     {{/link-to}}
     {{#link-to "index"}}
-        <h1>CARGO</h1>
+      <h1>
+        CARGO
+        <span class="subtitle">packages for Rust</span>
+      </h1>
     {{/link-to}}
 
     <form class='search' action='/search' {{ action "search" on="submit" }} >

tests/acceptance/front-page-test.js

@@ -8,7 +8,7 @@ test('visiting /', function(assert) {
 
     andThen(function() {
         assert.equal(currentURL(), '/');
-        assert.equal(document.title, 'Cargo');
+        assert.equal(document.title, 'Cargo: packages for Rust');
 
         findWithAssert('a[href="/install"]');
         findWithAssert('a[href="/crates"]');

tests/acceptance/search-test.js

@@ -13,7 +13,7 @@ test('searching for "rust"', function(assert) {
 
     andThen(function() {
         assert.equal(currentURL(), '/search?q=rust');
-        assert.equal(document.title, 'Search Results for \'rust\' - Cargo');
+        assert.equal(document.title, 'Search Results for \'rust\' - Cargo: packages for Rust');
 
         findWithAssert('a[href="/search?page=2&q=rust"]');
         assert.notOk(find('a[href="/search?page=3&q=rust"]')[0]);
@@ -29,7 +29,7 @@ test('searching for "rust"', function(assert) {
 
     andThen(function() {
         assert.equal(currentURL(), '/search?page=2&q=rust');
-        assert.equal(document.title, 'Search Results for \'rust\' - Cargo');
+        assert.equal(document.title, 'Search Results for \'rust\' - Cargo: packages for Rust');
 
         findWithAssert('a[href="/search?q=rust"]');
         assert.notOk(find('a[href="/search?page=3&q=rust"]')[0]);