[gnome-shell] overviewControls: Make getSlide/updateSlide private



commit dbf3bb112ce7185d502d26ac9a6e58805c680164
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Wed Sep 18 19:22:54 2013 -0400

    overviewControls: Make getSlide/updateSlide private

 js/ui/overviewControls.js |   24 ++++++++++++------------
 1 files changed, 12 insertions(+), 12 deletions(-)
---
diff --git a/js/ui/overviewControls.js b/js/ui/overviewControls.js
index d388340..7e1f6a2 100644
--- a/js/ui/overviewControls.js
+++ b/js/ui/overviewControls.js
@@ -120,12 +120,12 @@ const SlidingControl = new Lang.Class({
         Main.overview.connect('window-drag-end', Lang.bind(this, this._onWindowDragEnd));
     },
 
-    getSlide: function() {
+    _getSlide: function() {
         throw new Error('getSlide() must be overridden');
     },
 
-    updateSlide: function() {
-        Tweener.addTween(this.layout, { slideX: this.getSlide(),
+    _updateSlide: function() {
+        Tweener.addTween(this.layout, { slideX: this._getSlide(),
                                         time: SIDE_CONTROLS_ANIMATION_TIME,
                                         transition: 'easeOutQuad' });
     },
@@ -170,7 +170,7 @@ const SlidingControl = new Lang.Class({
 
     _onOverviewShowing: function() {
         this._visible = true;
-        this.layout.slideX = this.getSlide();
+        this.layout.slideX = this._getSlide();
         this.actor.translation_x = this._getTranslation();
         this.slideIn();
     },
@@ -190,12 +190,12 @@ const SlidingControl = new Lang.Class({
     _onDragBegin: function() {
         this._inDrag = true;
         this.actor.translation_x = 0;
-        this.updateSlide();
+        this._updateSlide();
     },
 
     _onDragEnd: function() {
         this._inDrag = false;
-        this.updateSlide();
+        this._updateSlide();
     },
 
     fadeIn: function() {
@@ -229,7 +229,7 @@ const SlidingControl = new Lang.Class({
         // selector; this means we can now safely set the full slide for
         // the next page, since slideIn or slideOut might have been called,
         // changing the visiblity
-        this.layout.slideX = this.getSlide();
+        this.layout.slideX = this._getSlide();
         this._updateTranslation();
     }
 });
@@ -248,8 +248,8 @@ const ThumbnailsSlider = new Lang.Class({
         this.actor.track_hover = true;
         this.actor.add_actor(this._thumbnailsBox.actor);
 
-        Main.layoutManager.connect('monitors-changed', Lang.bind(this, this.updateSlide));
-        this.actor.connect('notify::hover', Lang.bind(this, this.updateSlide));
+        Main.layoutManager.connect('monitors-changed', Lang.bind(this, this._updateSlide));
+        this.actor.connect('notify::hover', Lang.bind(this, this._updateSlide));
         this._thumbnailsBox.actor.bind_property('visible', this.actor, 'visible', 
GObject.BindingFlags.SYNC_CREATE);
     },
 
@@ -281,7 +281,7 @@ const ThumbnailsSlider = new Lang.Class({
         return child.get_theme_node().get_length('visible-width');
     },
 
-    getSlide: function() {
+    _getSlide: function() {
         if (!this._visible)
             return 0;
 
@@ -325,10 +325,10 @@ const DashSlider = new Lang.Class({
 
         this.actor.add_actor(this._dash.actor);
 
-        this._dash.connect('icon-size-changed', Lang.bind(this, this.updateSlide));
+        this._dash.connect('icon-size-changed', Lang.bind(this, this._updateSlide));
     },
 
-    getSlide: function() {
+    _getSlide: function() {
         if (this._visible || this._inDrag)
             return 1;
         else


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