Mark various scrolling attributes as deprecated (alamkanak#148)

Author: thellmund

core/src/main/java/com/alamkanak/weekview/WeekView.kt

@@ -1057,6 +1057,7 @@ class WeekView<T : Any> @JvmOverloads constructor(
      * Returns the scrolling speed factor in horizontal direction.
      */
     @PublicApi
+    @Deprecated("This value is no longer being taken into account.")
     var xScrollingSpeed: Float
         get() = configWrapper.xScrollingSpeed
         set(value) {
@@ -1067,6 +1068,10 @@ class WeekView<T : Any> @JvmOverloads constructor(
      * Returns whether WeekView can fling horizontally.
      */
     @PublicApi
+    @Deprecated(
+        message = "Use isHorizontalScrollingEnabled instead.",
+        replaceWith = ReplaceWith("isHorizontalScrollingEnabled")
+    )
     var isHorizontalFlingEnabled: Boolean
         get() = configWrapper.horizontalFlingEnabled
         set(value) {
@@ -1086,6 +1091,7 @@ class WeekView<T : Any> @JvmOverloads constructor(
     /**
      * Returns whether WeekView can fling vertically.
      */
+    @Deprecated("This value is no longer being taken into account.")
     @PublicApi
     var isVerticalFlingEnabled: Boolean
         get() = configWrapper.verticalFlingEnabled
@@ -1094,6 +1100,7 @@ class WeekView<T : Any> @JvmOverloads constructor(
         }
 
     @PublicApi
+    @Deprecated("This value is no longer being taken into account.")
     var scrollDuration: Int
         get() = configWrapper.scrollDuration
         set(value) {

core/src/main/java/com/alamkanak/weekview/WeekViewConfig.kt

@@ -113,10 +113,14 @@ internal class WeekViewConfig(
     var daySeparatorStrokeWidth: Int = 0
 
     // Scrolling
+    @Deprecated("No longer used")
     var xScrollingSpeed: Float = 0f
+    @Deprecated("No longer used")
     var verticalFlingEnabled: Boolean = false
+    @Deprecated("No longer used")
     var horizontalFlingEnabled: Boolean = false
     var horizontalScrollingEnabled: Boolean = false
+    @Deprecated("No longer used")
     var scrollDuration: Int = 0
 
     // Time range

core/src/main/java/com/alamkanak/weekview/WeekViewConfigWrapper.kt

@@ -252,12 +252,14 @@ internal class WeekViewConfigWrapper(
             config.maxHour = value
         }
 
+    @Deprecated("No longer used")
     var xScrollingSpeed: Float
         get() = config.xScrollingSpeed
         set(value) {
             config.xScrollingSpeed = value
         }
 
+    @Deprecated("No longer used")
     var horizontalFlingEnabled: Boolean
         get() = config.horizontalFlingEnabled
         set(value) {
@@ -270,12 +272,14 @@ internal class WeekViewConfigWrapper(
             config.horizontalScrollingEnabled = value
         }
 
+    @Deprecated("No longer used")
     var verticalFlingEnabled: Boolean
         get() = config.verticalFlingEnabled
         set(value) {
             config.verticalFlingEnabled = value
         }
 
+    @Deprecated("No longer used")
     var scrollDuration: Int
         get() = config.scrollDuration
         set(value) {

core/src/main/java/com/alamkanak/weekview/WeekViewGestureHandler.kt

@@ -92,9 +92,11 @@ internal class WeekViewGestureHandler<T : Any>(
         // Calculate the new origin after scroll.
         when (currentScrollDirection) {
             Left, Right -> {
-                config.currentOrigin.x -= distanceX * config.xScrollingSpeed
-                config.currentOrigin.x = min(config.currentOrigin.x, config.maxX)
-                config.currentOrigin.x = max(config.currentOrigin.x, config.minX)
+                config.currentOrigin.x -= distanceX
+                config.currentOrigin.x = config.currentOrigin.x.limit(
+                    minValue = config.minX,
+                    maxValue = config.maxX
+                )
                 onInvalidation()
             }
             Vertical -> {
@@ -113,11 +115,7 @@ internal class WeekViewGestureHandler<T : Any>(
         velocityX: Float,
         velocityY: Float
     ): Boolean {
-        if (currentFlingDirection.isHorizontal && !config.horizontalFlingEnabled) {
-            return true
-        }
-
-        if (currentFlingDirection.isVertical && !config.verticalFlingEnabled) {
+        if (currentFlingDirection.isHorizontal && !config.horizontalScrollingEnabled) {
             return true
         }