From a71a74233f8f38d7e7f51171b96cdd9c7bdfff62 Mon Sep 17 00:00:00 2001 From: Lisa Wray Date: Sun, 14 Aug 2016 15:48:37 -0400 Subject: [PATCH] Make sure to use correct bounds for padding and non-zero getLeft() --- .../src/main/java/com/xwray/passwordview/PasswordView.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/library/src/main/java/com/xwray/passwordview/PasswordView.java b/library/src/main/java/com/xwray/passwordview/PasswordView.java index 6d609ec..41c3b38 100644 --- a/library/src/main/java/com/xwray/passwordview/PasswordView.java +++ b/library/src/main/java/com/xwray/passwordview/PasswordView.java @@ -91,8 +91,11 @@ protected void setup() { } @Override public boolean onTouchEvent(MotionEvent event) { + int drawableRightX = getWidth() - getPaddingRight(); + int drawableLeftX = drawableRightX - getCompoundDrawables()[2].getBounds().width(); + if (event.getAction() == MotionEvent.ACTION_DOWN - && event.getX() >= (getRight() - getCompoundDrawables()[2].getBounds().width())) { + && event.getX() >= drawableLeftX && event.getX() <= drawableRightX) { visible = !visible; setup(); invalidate();