From 88c7039ebe0862c483a5d2ed6dfb1970746f8252 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Thu, 5 Feb 2015 16:56:51 +0100 Subject: [PATCH] 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 file changed, 9 deletions(-) diff --git a/js/ui/dateMenu.js b/js/ui/dateMenu.js index b71aad8ac..60d29ceb4 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(); } });