Added a shiny new time click listener

Author: alamkanak

library/src/main/java/com/alamkanak/weekview/WeekView.java

@@ -33,7 +33,7 @@
 
 /**
  * Created by Raquib-ul-Alam Kanak on 7/21/2014.
- * Website: http://april-shower.com
+ * Website: http://alamkanak.me
  */
 public class WeekView extends View {
 
@@ -101,7 +101,7 @@ public class WeekView extends View {
     private EventClickListener mEventClickListener;
     private EventLongPressListener mEventLongPressListener;
     private MonthChangeListener mMonthChangeListener;
-    private HourClickListener mHourClickListener;
+    private TimeClickListener mTimeClickListener;
 
     private final GestureDetector.SimpleOnGestureListener mGestureListener = new GestureDetector.SimpleOnGestureListener() {
 
@@ -153,6 +153,7 @@ else if (mCurrentFlingDirection == Direction.VERTICAL){
 
         @Override
         public boolean onSingleTapConfirmed(MotionEvent e) {
+            // If the tap was on an event then trigger the callback.
             if (mEventRects != null && mEventClickListener != null) {
                 List<EventRect> reversedEventRects = mEventRects;
                 Collections.reverse(reversedEventRects);
@@ -164,12 +165,14 @@ public boolean onSingleTapConfirmed(MotionEvent e) {
                     }
                 }
             }
-            if(mHourClickListener != null) {
+
+            // If the tap was on in an empty space, then trigger the callback.
+            if (mTimeClickListener != null && e.getX() > mHeaderColumnWidth && e.getY() > (mHeaderTextHeight + mHeaderRowPadding * 2 + mHeaderMarginBottom)) {
                 Calendar selectedTime = getTimeFromPoint(e.getX(), e.getY());
-                mHourClickListener.onHourClicked(selectedTime);
-            }
-            else{
-                throw new IllegalStateException("You must provide a HourClickListener");
+                if (selectedTime != null) {
+                    playSoundEffect(SoundEffectConstants.CLICK);
+                    mTimeClickListener.onTimeClicked(selectedTime);
+                }
             }
 
             return super.onSingleTapConfirmed(e);
@@ -396,7 +399,7 @@ private void drawHeaderRowAndEvents(Canvas canvas) {
         lineCount = (lineCount) * (mNumberOfVisibleDays+1);
         float[] hourLines = new float[lineCount * 4];
 
-        // Clear the cache for events rectangles.
+        // Clear the cache for event rectangles.
         if (mEventRects != null) {
             for (EventRect eventRect: mEventRects) {
                 eventRect.rectF = null;
@@ -471,11 +474,16 @@ private void drawHeaderRowAndEvents(Canvas canvas) {
 
     }
 
+    /**
+     * Get the time and date where the user clicked on.
+     * @param x The x position of the touch event.
+     * @param y The y position of the touch event.
+     * @return The time and date at the clicked position.
+     */
     private Calendar getTimeFromPoint(float x, float y){
         int leftDaysWithGaps = (int) -(Math.ceil(mCurrentOrigin.x / (mWidthPerDay + mColumnGap)));
-        float startFromPixel = mCurrentOrigin.x + (mWidthPerDay + mColumnGap) * leftDaysWithGaps +
+        float startPixel = mCurrentOrigin.x + (mWidthPerDay + mColumnGap) * leftDaysWithGaps +
                 mHeaderColumnWidth;
-        float startPixel = startFromPixel;
         for (int dayNumber = leftDaysWithGaps + 1;
              dayNumber <= leftDaysWithGaps + mNumberOfVisibleDays + 1;
              dayNumber++) {
@@ -487,7 +495,9 @@ private Calendar getTimeFromPoint(float x, float y){
                 float pixelsFromZero = y - mCurrentOrigin.y - mHeaderTextHeight
                         - mHeaderRowPadding * 2 - mTimeTextHeight/2 - mHeaderMarginBottom;
                 int hour = (int)(pixelsFromZero / mHourHeight);
+                int minute = (int) (60 * (pixelsFromZero - hour * mHourHeight) / mHourHeight);
                 day.add(Calendar.HOUR, hour);
+                day.set(Calendar.MINUTE, minute);
                 return day;
             }
             startPixel += mWidthPerDay + mColumnGap;
@@ -705,13 +715,17 @@ private void getMoreEvents(Calendar day) {
         }
     }
 
+    /**
+     * Cache the event for smooth scrolling functionality.
+     * @param event The event to cache.
+     */
     private void cacheEvent(WeekViewEvent event) {
         if (!isSameDay(event.getStartTime(), event.getEndTime())) {
             Calendar endTime = (Calendar) event.getStartTime().clone();
             endTime.set(Calendar.HOUR_OF_DAY, 23);
             endTime.set(Calendar.MINUTE, 59);
             Calendar startTime = (Calendar) event.getEndTime().clone();
-            startTime.set(Calendar.HOUR_OF_DAY, 00);
+            startTime.set(Calendar.HOUR_OF_DAY, 0);
             startTime.set(Calendar.MINUTE, 0);
             WeekViewEvent event1 = new WeekViewEvent(event.getId(), event.getName(), event.getStartTime(), endTime);
             event1.setColor(event.getColor());
@@ -914,12 +928,12 @@ public void setEventLongPressListener(EventLongPressListener eventLongPressListe
         this.mEventLongPressListener = eventLongPressListener;
     }
 
-    public HourClickListener getHourClickListener(){
-        return mHourClickListener;
+    public TimeClickListener getHourClickListener(){
+        return mTimeClickListener;
     }
 
-    public void setHourClickListener(HourClickListener mHourClickListener){
-        this.mHourClickListener = mHourClickListener;
+    public void setHourClickListener(TimeClickListener mTimeClickListener){
+        this.mTimeClickListener = mTimeClickListener;
     }
 
 
@@ -1310,11 +1324,8 @@ public interface EventLongPressListener {
         public void onEventLongPress(WeekViewEvent event, RectF eventRect);
     }
 
-    /**
-     *   Added by Fan Chen on 12/19/2014
-     */
-    public interface HourClickListener{
-        public void onHourClicked(Calendar startTime);
+    public interface TimeClickListener {
+        public void onTimeClicked(Calendar time);
     }