From de284ec8834c24a63257c23bfd7926cd0639a54d Mon Sep 17 00:00:00 2001
From: Xelio <xelio.cheong@gmail.com>
Date: Fri, 15 Mar 2013 21:41:09 +0800
Subject: [PATCH] Plugin AP List: New Features Clicking the current "Enemy" or
 "Friendly" label again will scroll the sidebar to top.

---
 plugins/ap-list.user.js | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/plugins/ap-list.user.js b/plugins/ap-list.user.js
index d9de91ec..a2d36a53 100644
--- a/plugins/ap-list.user.js
+++ b/plugins/ap-list.user.js
@@ -1,7 +1,7 @@
 // ==UserScript==
 // @id             iitc-plugin-ap-list@xelio
 // @name           IITC plugin: AP List
-// @version        0.4.1.@@DATETIMEVERSION@@
+// @version        0.4.2.@@DATETIMEVERSION@@
 // @namespace      https://github.com/jonatkins/ingress-intel-total-conversion
 // @updateURL      @@UPDATEURL@@
 // @downloadURL    @@DOWNLOADURL@@
@@ -408,22 +408,24 @@ window.plugin.apList.getAttackApGain = function(d) {
 
 // Change display table to friendly portals
 window.plugin.apList.displayFriendly = function() {
-  plugin.apList.displaySide = plugin.apList.SIDE_FRIENDLY;
-  plugin.apList.changeDisplaySide(plugin.apList.displaySide);
+  plugin.apList.changeDisplaySide(plugin.apList.SIDE_FRIENDLY);
 }
 
 // Change display table to enemy portals
 window.plugin.apList.displayEnemy = function() {
-  plugin.apList.displaySide = plugin.apList.SIDE_ENEMY;
-  plugin.apList.changeDisplaySide(plugin.apList.displaySide);
+  plugin.apList.changeDisplaySide(plugin.apList.SIDE_ENEMY);
 }
 
 window.plugin.apList.changeDisplaySide = function(side) {
-  plugin.apList.updatePortalTable(side);
-  plugin.apList.toggleSideLabel(side);
-
-  var scrollTo = $("#ap-list").position().top + $("#ap-list").outerHeight() 
-                - $("#sidebar").height() + $("#sidebar").scrollTop()
+  var isChange = (plugin.apList.displaySide !== side);
+  var scrollTo = 0;
+  if(isChange) {
+    plugin.apList.displaySide = side;
+    plugin.apList.updatePortalTable(side);
+    plugin.apList.toggleSideLabel(side);
+    scrollTo = $("#ap-list").position().top + $("#ap-list").outerHeight()
+            - $("#sidebar").height() + $("#sidebar").scrollTop();
+  }
   $('#sidebar').scrollTop(scrollTo);
 }