Merge branch 'master' of github.com:travis-ci/travis-ember

assets/javascripts/app/models/artifact.coffee

@@ -2,6 +2,19 @@ require 'travis/model'
 
 @Travis.Artifact = Travis.Model.extend
   body: DS.attr('string')
+  init: ->
+    @_super.apply this, arguments
+    @set 'queue', Ember.A([])
 
   append: (body) ->
+    if @get('isLoaded')
       @set('body', @get('body') + body)
+    else
+      @get('queue').pushObject(body)
+
+  recordDidLoad: (->
+    if @get('isLoaded')
+      queue = @get('queue')
+      if queue.get('length') > 0
+        @append queue.toArray().join('')
+  ).observes('isLoaded')

assets/javascripts/app/models/build.coffee

@@ -28,11 +28,11 @@ require 'travis/model'
 
   requiredJobs: (->
     @get('jobs').filter (data) -> !data.get('allowFailure')
-  ).property()
+  ).property('jobs.@each.allowFailure')
 
   allowedFailureJobs: (->
     @get('jobs').filter (data) -> data.get('allowFailure')
-  ).property()
+  ).property('jobs.@each.allowFailure')
 
   configKeys: (->
     return [] unless config = @get('config')

assets/javascripts/app/templates/builds/list.hbs

@@ -13,7 +13,6 @@
     <tbody>
       {{#each build in builds}}
         {{#view Travis.BuildsItemView contextBinding="build"}}
-          <tr {{bindAttr class="view.color"}}>
           <td class="number">
             <span class="status"></span>
             {{#if id}}
@@ -36,7 +35,6 @@
           <td class="finished_at timeago" {{bindAttr title="finishedAt"}}>
             {{formatTime finishedAt}}
           </td>
-          </tr>
         {{/view}}
       {{/each}}
     </tbody>

assets/javascripts/app/templates/builds/show.hbs

@@ -1,6 +1,7 @@
 {{#with view}}
-  {{#if build.isLoaded}}
+  {{#if loading}}
-    <div id="build" {{bindAttr class="color"}}>
+    <span>Loading</span>
+  {{else}}
     <dl id="summary">
       <div class="left">
         <dt>{{t builds.name}}</dt>
@@ -18,18 +19,18 @@
 
       <div class="right">
         <dt>{{t builds.commit}}</dt>
-          <dd class="commit"><a {{bindAttr href="urlGithubCommit"}}>{{formatCommit build.commit}}</a></dd>
+        <dd class="commit"><a href="{{unbound urlGithubCommit}}">{{formatCommit build.commit}}</a></dd>
         {{#if commit.compareUrl}}
           <dt>{{t builds.compare}}</dt>
-            <dd class="compare"><a {{bindAttr href="commit.compareUrl"}}>{{pathFrom build.commit.compareUrl}}</a></dd>
+          <dd class="compare"><a href="{{unbound commit.compareUrl}}">{{pathFrom build.commit.compareUrl}}</a></dd>
         {{/if}}
         {{#if commit.authorName}}
           <dt>{{t builds.author}}</dt>
-            <dd class="author"><a {{bindAttr href="urlAuthor"}}>{{build.commit.authorName}}</a></dd>
+          <dd class="author"><a href="{{unbound urlAuthor}}">{{build.commit.authorName}}</a></dd>
         {{/if}}
         {{#if commit.committerName}}
           <dt>{{t builds.committer}}</dt>
-            <dd class="committer"><a {{bindAttr href="urlCommitter"}}>{{build.commit.committerName}}</a></dd>
+          <dd class="committer"><a href="{{unbound urlCommitter}}">{{build.commit.committerName}}</a></dd>
         {{/if}}
       </div>
 
@@ -41,7 +42,6 @@
         <dd class="config">{{formatConfig build.config}}</dd>
       {{/unless}}
     </dl>
-    </div>
 
     {{#if build.isMatrix}}
       {{view Travis.JobsView jobsBinding="build.requiredJobs" required="true"}}
@@ -49,9 +49,5 @@
     {{else}}
       {{view Travis.LogView contextBinding="build.jobs.firstObject"}}
     {{/if}}
-  {{else}}
-    <div id="build" class="loading">
-      <span>Loading</span>
-    </div>
   {{/if}}
 {{/with}}

assets/javascripts/app/views/build.coffee

@@ -9,6 +9,8 @@
       Travis.Build.olderThanNumber(id, number)
 
   BuildsItemView: Travis.View.extend
+    tagName: 'tr'
+    classNameBindings: ['color']
     repositoryBinding: 'controller.repository'
     buildBinding: 'context'
     commitBinding: 'build.commit'
@@ -27,11 +29,17 @@
 
   BuildView: Travis.View.extend
     templateName: 'builds/show'
+    elementId: 'build'
+    classNameBindings: ['color', 'loading']
 
     repositoryBinding: 'controller.repository'
     buildBinding: 'controller.build'
     commitBinding: 'build.commit'
 
+    loading: (->
+      !@get('build.isLoaded')
+    ).property('build.isLoaded')
+
     color: (->
       Travis.Helpers.colorForResult(@get('build.result'))
     ).property('build.result')