fix: add new method to get correct time indicator top position | fixes jquense#1396 (jquense#1447)

* fix: add new method to get correct time indicator top position

* fix(current time indicator): remove unnecessary fields

* fix(current time indicator): rename function

* fix(current time indicator): remove unnecessary code and arguments

Author: eightshone

src/DayColumn.js

@@ -97,7 +97,7 @@ class DayColumn extends React.Component {
     const current = getNow()
 
     if (current >= min && current <= max) {
-      const { top } = this.slotMetrics.getRange(current, current)
+      const top = this.slotMetrics.getCurrentTimePosition(current)
       this.setState({ timeIndicatorPosition: top })
     } else {
       this.clearTimeIndicatorInterval()

src/utils/TimeSlots.js

@@ -125,10 +125,8 @@ export function getSlotMetrics({ min: start, max: end, step, timeslots }) {
     },
 
     getRange(rangeStart, rangeEnd, ignoreMin, ignoreMax) {
-      if (!ignoreMin)
-        rangeStart = dates.min(end, dates.max(start, rangeStart))
-      if (!ignoreMax)
-        rangeEnd = dates.min(end, dates.max(start, rangeEnd))
+      if (!ignoreMin) rangeStart = dates.min(end, dates.max(start, rangeStart))
+      if (!ignoreMax) rangeEnd = dates.min(end, dates.max(start, rangeEnd))
 
       const rangeStartMin = positionFromDate(rangeStart)
       const rangeEndMin = positionFromDate(rangeEnd)
@@ -146,5 +144,12 @@ export function getSlotMetrics({ min: start, max: end, step, timeslots }) {
         endDate: rangeEnd,
       }
     },
+
+    getCurrentTimePosition(rangeStart) {
+      const rangeStartMin = positionFromDate(rangeStart)
+      const top = (rangeStartMin / (step * numSlots)) * 100
+
+      return top
+    },
   }
 }