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



commit 4b29c1f7c71ab1ad00c51f4caa63f8283029cc3a
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]