Revert "Merge pull request #2304 from techAPJ/speech-bubble-update"

This reverts commit 6590ebbcbe, reversing
changes made to 3855ead62e.

app/assets/javascripts/discourse/controllers/header.js.es6

@@ -10,7 +10,6 @@ export default Discourse.Controller.extend({
   topic: null,
   showExtraInfo: null,
   notifications: null,
-  loading_notifications: null,
 
   showStarButton: function() {
     return Discourse.User.current() && !this.get('topic.isPrivateMessage');
@@ -26,17 +25,11 @@ export default Discourse.Controller.extend({
     showNotifications: function(headerView) {
       var self = this;
 
-      if (self.get('currentUser.unread_notifications') || self.get('currentUser.unread_private_messages') || !self.get('notifications')) {
-        self.set("loading_notifications", true);
-        Discourse.ajax("/notifications").then(function(result) {
-          self.setProperties({
-            notifications: result,
-            loading_notifications: false,
-            'currentUser.unread_notifications': 0
-          });
-        });
-      }
-      headerView.showDropdownBySelector("#user-notifications");
+      Discourse.ajax("/notifications").then(function(result) {
+        self.set("notifications", result);
+        self.set("currentUser.unread_notifications", 0);
+        headerView.showDropdownBySelector("#user-notifications");
+      });
     },
 
     jumpToTopPost: function () {
@@ -48,3 +41,5 @@ export default Discourse.Controller.extend({
   }
 
 });
+
+

app/assets/javascripts/discourse/controllers/notifications_controller.js

@@ -1,4 +1,3 @@
 Discourse.NotificationsController = Ember.ArrayController.extend(Discourse.HasCurrentUser, {
-  needs: ['header'],
   itemController: "notification"
 });

app/assets/javascripts/discourse/templates/notifications.js.handlebars

@@ -1,18 +1,14 @@
 <section class="d-dropdown" id="notifications-dropdown">
-  {{#unless controllers.header.loading_notifications}}
-    {{#if content}}
-      <ul>
-        {{#each}}
-          <li {{bind-attr class="read"}}>{{unbound boundI18n scope linkBinding="link" usernameBinding="username"}}</li>
-        {{/each}}
-        <li class="read last">
-          <a {{bind-attr href="currentUser.path"}}>{{i18n notifications.more}} &hellip;</a>
-        </li>
-      </ul>
-    {{else}}
-      <div class="none">{{i18n notifications.none}}</div>
-    {{/if}}
+  {{#if content}}
+    <ul>
+      {{#each}}
+        <li {{bind-attr class="read"}}>{{unbound boundI18n scope linkBinding="link" usernameBinding="username"}}</li>
+      {{/each}}
+      <li class="read last">
+        <a {{bind-attr href="currentUser.path"}}>{{i18n notifications.more}} &hellip;</a>
+      </li>
+    </ul>
   {{else}}
-    <div class='loading'><i class='fa fa-spinner fa-spin'></i></div>
-  {{/unless}}
+    <div class="none">{{i18n notifications.none}}</div>
+  {{/if}}
 </section>

app/assets/stylesheets/desktop/header.scss

@@ -204,13 +204,6 @@
     .none {
       padding: 5px;
     }
-    .loading {
-      padding: 10px;
-      display: block;
-      color: lighten($primary_text_color, 50%);
-      font-size: 24px;
-      text-align: center;
-    }
   }
 
   // Search

app/assets/stylesheets/mobile/header.scss

@@ -194,13 +194,6 @@
     .none {
       padding: 5px;
     }
-    .loading {
-      padding: 10px;
-      display: block;
-      color: #777;
-      font-size: 24px;
-      text-align: center;
-    }
   }
 
   // Search

test/javascripts/controllers/header_controller_test.js

@@ -23,7 +23,7 @@ test("showNotifications action", function() {
 
   equal(controller.get("notifications"), null, "notifications are null before data has finished loading");
   equal(Discourse.User.current().get("unread_notifications"), 1, "current user's unread notifications count is not zeroed before data has finished loading");
-  ok(viewSpy.showDropdownBySelector.calledWith("#user-notifications"), "dropdown with loading glyph is shown before data has finished loading");
+  ok(viewSpy.showDropdownBySelector.notCalled, "dropdown with notifications is not shown before data has finished loading");
 
 
   Ember.run(function() {

test/javascripts/controllers/notifications_controller_test.js

@@ -17,9 +17,7 @@ module("Discourse.NotificationsController", {
       return [scope, options.username, options.link].join(" ").trim();
     });
 
-    controller = Discourse.NotificationsController.create({
-      container: Discourse.__container__
-    });
+    controller = Discourse.NotificationsController.create();
 
     view = Ember.View.create({
       container: Discourse.__container__,