[gnome-shell/wip/fmuellner/notification-redux+sass: 105/141] dateMenu: Stop hiding the events list



commit 4cf68b5307385581f53b726d05f21c325827457b
Author: Florian Müllner <fmuellner gnome org>
Date:   Thu Feb 5 16:56:51 2015 +0100

    dateMenu: Stop hiding the events list
    
    The message list is not only a replacement for the calendar events
    list, it will also take over the notification summary from the
    message tray. As we start drawing events from other sources than
    calendars, hiding it based on whether or not any calendars have
    been set up is no longer appropriate, so always include it in the
    calendar drop-down now.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=744817

 js/ui/dateMenu.js |    9 ---------
 1 files changed, 0 insertions(+), 9 deletions(-)
---
diff --git a/js/ui/dateMenu.js b/js/ui/dateMenu.js
index b71aad8..60d29ce 100644
--- a/js/ui/dateMenu.js
+++ b/js/ui/dateMenu.js
@@ -294,11 +294,6 @@ const DateMenuButton = new Lang.Class({
         this._sessionUpdated();
     },
 
-    _updateEventsVisibility: function() {
-        let visible = this._eventSource.hasCalendars;
-        this._messageList.actor.visible = visible;
-    },
-
     _getEventSource: function() {
         return new Calendar.DBusEventSource();
     },
@@ -311,9 +306,6 @@ const DateMenuButton = new Lang.Class({
         this._messageList.setEventSource(eventSource);
 
         this._eventSource = eventSource;
-        this._eventSource.connect('notify::has-calendars', Lang.bind(this, function() {
-            this._updateEventsVisibility();
-        }));
     },
 
     _sessionUpdated: function() {
@@ -325,6 +317,5 @@ const DateMenuButton = new Lang.Class({
             eventSource = new Calendar.EmptyEventSource();
         }
         this._setEventSource(eventSource);
-        this._updateEventsVisibility();
     }
 });


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]