From c2de996fe11daa514046c3f80f9fd08585617b31 Mon Sep 17 00:00:00 2001
From: "yaoh.wu" <yaoh.wu@fanruan.com>
Date: Tue, 24 Jul 2018 09:13:49 +0800
Subject: [PATCH] =?UTF-8?q?=E5=A4=8D=E7=94=A8=E4=BB=A3=E7=A0=81?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../remote/ui/list/AddedMemberList.java       | 16 ----------------
 .../remote/ui/list/AddingMemberList.java      | 15 ---------------
 .../fr/design/remote/ui/list/MemberList.java  | 19 ++++++++++++++++---
 3 files changed, 16 insertions(+), 34 deletions(-)

diff --git a/designer-base/src/main/java/com/fr/design/remote/ui/list/AddedMemberList.java b/designer-base/src/main/java/com/fr/design/remote/ui/list/AddedMemberList.java
index ad4d880d9..2d95dce4b 100644
--- a/designer-base/src/main/java/com/fr/design/remote/ui/list/AddedMemberList.java
+++ b/designer-base/src/main/java/com/fr/design/remote/ui/list/AddedMemberList.java
@@ -3,8 +3,6 @@ package com.fr.design.remote.ui.list;
 import com.fr.workspace.server.authority.RemoteDesignMember;
 
 import javax.swing.DefaultListModel;
-import java.awt.Point;
-import java.awt.event.MouseEvent;
 import java.util.Vector;
 
 public class AddedMemberList extends MemberList {
@@ -25,20 +23,6 @@ public class AddedMemberList extends MemberList {
         super(listData);
     }
 
-
-    @Override
-    protected boolean shouldDisplaySelected(MouseEvent e) {
-        Point point = e.getPoint();
-        int rX = point.x;
-        int rY = point.y;
-        int index = this.getSelectedIndex();
-        int x = 270;
-        int y = 25 * index;
-        int width = 20;
-        int height = 25;
-        return x <= rX && rX <= x + width && y <= rY && rY <= y + height;
-    }
-
     @Override
     protected void displaySelected() {
         RemoteDesignMember member = getSelectedValue();
diff --git a/designer-base/src/main/java/com/fr/design/remote/ui/list/AddingMemberList.java b/designer-base/src/main/java/com/fr/design/remote/ui/list/AddingMemberList.java
index bcd42d5c9..649907d05 100644
--- a/designer-base/src/main/java/com/fr/design/remote/ui/list/AddingMemberList.java
+++ b/designer-base/src/main/java/com/fr/design/remote/ui/list/AddingMemberList.java
@@ -3,8 +3,6 @@ package com.fr.design.remote.ui.list;
 import com.fr.workspace.server.authority.RemoteDesignMember;
 
 import javax.swing.DefaultListModel;
-import java.awt.Point;
-import java.awt.event.MouseEvent;
 import java.util.Vector;
 
 public class AddingMemberList extends MemberList {
@@ -36,19 +34,6 @@ public class AddingMemberList extends MemberList {
         repaint();
         fireSelectedChange();
     }
-
-    @Override
-    protected boolean shouldDisplaySelected(MouseEvent e) {
-        Point point = e.getPoint();
-        int rX = point.x;
-        int rY = point.y;
-        int index = this.getSelectedIndex();
-        int x = 270;
-        int y = 25 * index;
-        int width = 25;
-        int height = 25;
-        return x <= rX && rX <= x + width && y <= rY && rY <= y + height;
-    }
 }
 
 
diff --git a/designer-base/src/main/java/com/fr/design/remote/ui/list/MemberList.java b/designer-base/src/main/java/com/fr/design/remote/ui/list/MemberList.java
index 60e44af99..10e99f0e3 100644
--- a/designer-base/src/main/java/com/fr/design/remote/ui/list/MemberList.java
+++ b/designer-base/src/main/java/com/fr/design/remote/ui/list/MemberList.java
@@ -7,6 +7,7 @@ import javax.swing.DefaultListModel;
 import javax.swing.JList;
 import javax.swing.ListSelectionModel;
 import java.awt.Color;
+import java.awt.Point;
 import java.awt.event.KeyAdapter;
 import java.awt.event.KeyEvent;
 import java.awt.event.MouseAdapter;
@@ -14,6 +15,12 @@ import java.awt.event.MouseEvent;
 import java.util.Vector;
 
 public abstract class MemberList extends JList<RemoteDesignMember> {
+    static final int TRIGGER_AREA_X = 270;
+    static final int TRIGGER_AREA_Y_EACH = 25;
+
+    static final int TRIGGER_AREA_WIDTH = 25;
+    static final int TRIGGER_AREA_HEIGHT = 25;
+
 
     public MemberList(DefaultListModel<RemoteDesignMember> dataModel) {
         super(dataModel);
@@ -79,7 +86,13 @@ public abstract class MemberList extends JList<RemoteDesignMember> {
 
     abstract protected void displaySelected();
 
-    abstract protected boolean shouldDisplaySelected(MouseEvent e);
-
-
+    protected boolean shouldDisplaySelected(MouseEvent e) {
+        Point point = e.getPoint();
+        int rX = point.x;
+        int rY = point.y;
+        int index = this.getSelectedIndex();
+        int x = TRIGGER_AREA_X;
+        int y = TRIGGER_AREA_Y_EACH * index;
+        return x <= rX && rX <= x + TRIGGER_AREA_WIDTH && y <= rY && rY <= y + TRIGGER_AREA_HEIGHT;
+    }
 }