From a07e8bbf37959a7387683808a7018859a1b6d80a Mon Sep 17 00:00:00 2001 From: "Jasper St. Pierre" Date: Wed, 4 Dec 2013 22:56:59 -0500 Subject: [PATCH] messageTray: Remove addBody as a public API As it's unused. --- js/ui/messageTray.js | 35 ++++++++++------------------------- 1 file changed, 10 insertions(+), 25 deletions(-) diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js index 52067361b..a4442f592 100644 --- a/js/ui/messageTray.js +++ b/js/ui/messageTray.js @@ -444,15 +444,14 @@ const NotificationApplicationPolicy = new Lang.Class({ // unless 'customContent' parameter with the value 'true' is specified // in @params. // -// Additional notification content can be added with addActor() and -// addBody() methods. The notification content is put inside a -// scrollview, so if it gets too tall, the notification will scroll -// rather than continue to grow. In addition to this main content -// area, there is also a single-row action area, which is not -// scrolled and can contain a single actor. The action area can -// be set by calling setActionArea() method. There is also a -// convenience method addButton() for adding a button to the action -// area. +// Additional notification content can be added with addActor(). The +// notification content is put inside a scrollview, so if it gets too +// tall, the notification will scroll rather than continue to grow. +// In addition to this main content area, there is also a single-row +// action area, which is not scrolled and can contain a single actor. +// The action area can be set by calling setActionArea() method. There +// is also a convenience method addButton() for adding a button to the +// action area. // // If @params contains a 'customContent' parameter with the value %true, // then @banner will not be shown in the body of the notification when the @@ -710,25 +709,11 @@ const Notification = new Lang.Class({ this.updated(); }, - // addBody: - // @text: the text - // @markup: %true if @text contains pango markup - // @style: style to use when adding the actor containing the text - // - // Adds a multi-line label containing @text to the notification. - // - // Return value: the newly-added label - addBody: function(text, markup, style) { - let label = new URLHighlighter(text, true, markup); - - this.addActor(label.actor, style); - return label.actor; - }, - _addBannerBody: function() { if (this.bannerBodyText && !this._bannerBodyAdded) { + let label = new URLHighlighter(this.bannerBodyText, true, this.bannerBodyMarkup); + this.addActor(label.actor); this._bannerBodyAdded = true; - this.addBody(this.bannerBodyText, this.bannerBodyMarkup); } },