Fix ugly avatar loading in presence screen

This fixes #18
initLab · Jul 21, 2014 · d61b756 · d61b756
1 parent 4dc964a
commit d61b756
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/app/assets/javascripts/presence.js.coffee b/app/assets/javascripts/presence.js.coffee
@@ -0,0 +1,6 @@
+$(document).on "page:change", ->
+  $('#presence .avatar img').on 'load', ->
+    $(this).parent().fadeIn()
+  .each ->
+    if $(this).complete
+      $(this).trigger 'load'
diff --git a/app/assets/stylesheets/base/_base.css.scss b/app/assets/stylesheets/base/_base.css.scss
@@ -154,4 +154,6 @@ a.button, a:focus.button, a:hover.button, a:visited.button {
     z-index: 1;
     display: none;
   }
-}
+}
+
+#presence .avatar { display: none; }
diff --git a/app/views/users/_user.html.erb b/app/views/users/_user.html.erb
@@ -1,6 +1,6 @@
 <li>
   <span class="avatar">
-    <%= image_tag "//www.gravatar.com/avatar/#{user.email_md5}?s=50&d=retro" %>
+    <%= image_tag "//www.gravatar.com/avatar/#{user.email_md5}?s=50&d=retro", alt: user.name %>
   </span>
   <span class="info">
     <% if user.url.present?  %>