Refactored setDate function to update selected date based on the units passed as a param.

Author: g1eb

src/datetime-range.html

@@ -19,46 +19,47 @@
     </div>
     <div class="calendar" ng-show="!!calendar_active">
       <div class="calendar-header">
-        <div class="arrow arrow-left" ng-click="setDate(selected.clone().subtract(1, 'months'))"></div>
+        <div class="arrow arrow-left" ng-click="updateDate('month', selected.clone().subtract(1, 'months').month())"></div>
         {{ selected.format('MMMM') }}
-        <div class="arrow arrow-right" ng-click="setDate(selected.clone().add(1, 'months'))"></div>
+        <div class="arrow arrow-right" ng-click="updateDate('month', selected.clone().add(1, 'months').month())"></div>
       </div>
       <div class="calendar-body">
         <div class="weekdays">
           <span class="weekday" ng-repeat="weekday in 'weeeeek' track by $index">
             {{ selected.clone().startOf('week').add($index, 'days').format('ddd') }}
           </span>
         </div>
-        <div class="week" ng-repeat="week in 'months' | limitTo: (selected.clone().endOf('month').week() - selected.clone().startOf('month').week()) + 1 track by $index">
+        <div class="week" ng-repeat="week in 'months' | limitTo: ((selected.clone().endOf('month').week() - selected.clone().startOf('month').week()) + 1) track by $index">
           <span class="date" ng-repeat="date in 'weeeeek' track by $index"
+            ng-init="_date = selected.clone().startOf('month').add($parent.$index, 'weeks').weekday($index).startOf('day')"
             ng-class="{
-              'current': selected.clone().startOf('month').add($parent.$index, 'weeks').weekday($index).startOf('day').isSame(current.clone().startOf('day')),
-              'active': selected.clone().startOf('month').add($parent.$index, 'weeks').weekday($index).startOf('day').isSame(selected.clone().startOf('day')),
-              'inactive': selected.clone().startOf('month').add($parent.$index, 'weeks').weekday($index).month() !== selected.month()
+              'current': _date.isSame(current.clone().startOf('day')),
+              'active': _date.isSame(selected.clone().startOf('day')),
+              'inactive': _date.month() !== selected.month()
             }"
-            ng-click="setDate(selected.clone().date(selected.clone().startOf('month').add($parent.$index, 'weeks').weekday($index).date()))">
-            {{ selected.clone().startOf('month').add($parent.$index, 'weeks').weekday($index).date() }}
+            ng-click="updateDate('month', _date.month()) && updateDate('date', _date.date()) && updateDate('month', _date.month())">
+            {{ _date.date() }}
           </span>
         </div>
       </div>
     </div>
     <div class="timer">
       <div class="timer-hours">
-        <div class="arrow arrow-up" ng-click="setDate(selected.clone().add(1, 'hours'))"></div>
+        <div class="arrow arrow-up" ng-click="updateDate('hours', selected.clone().add(1, 'hours').hours())"></div>
         {{ selected.format('HH') }}
-        <div class="arrow arrow-down" ng-click="setDate(selected.clone().subtract(1, 'hours'))"></div>
+        <div class="arrow arrow-down" ng-click="updateDate('hours', selected.clone().subtract(1, 'hours').hours())"></div>
       </div>
       <div class="timer-divider">:</div>
       <div class="timer-minutes">
-        <div class="arrow arrow-up" ng-click="setDate(selected.clone().add(1, 'minutes'))"></div>
+        <div class="arrow arrow-up" ng-click="updateDate('minutes', selected.clone().add(1, 'minutes').minutes())"></div>
         {{ selected.format('mm') }}
-        <div class="arrow arrow-down" ng-click="setDate(selected.clone().subtract(1, 'minutes'))"></div>
+        <div class="arrow arrow-down" ng-click="updateDate('minutes', selected.clone().subtract(1, 'minutes').minutes())"></div>
       </div>
       <div class="timer-divider">:</div>
       <div class="timer-seconds">
-        <div class="arrow arrow-up" ng-click="setDate(selected.clone().add(1, 'seconds'))"></div>
+        <div class="arrow arrow-up" ng-click="updateDate('seconds', selected.clone().add(1, 'seconds').seconds())"></div>
         {{ selected.format('ss') }}
-        <div class="arrow arrow-down" ng-click="setDate(selected.clone().subtract(1, 'seconds'))"></div>
+        <div class="arrow arrow-down" ng-click="updateDate('seconds', selected.clone().subtract(1, 'seconds').seconds())"></div>
       </div>
     </div>
   </div>

src/datetime-range.js

@@ -21,10 +21,11 @@ angular.module('g1b.datetime-range', []).
           scope.current = moment();
 
           // Set selected date
-          scope.setDate = function (date) {
-            if ( (scope.selected === scope.start && date < scope.end) ||
-              ( scope.selected === scope.end && date > scope.start ) ) {
-              scope.selected.seconds(date.get('seconds')).minute(date.get('minutes')).hours(date.get('hours')).date(date.get('date')).month(date.get('month')).year(date.get('year'));
+          scope.updateDate = function (unit, value) {
+            var _date = scope.selected.clone()[unit](value);
+            if ( ( scope.selected === scope.start && _date < scope.end) ||
+              ( scope.selected === scope.end && _date > scope.start ) ) {
+              scope.selected[unit](_date[unit]());
               scope.handler();
             }
           };