implemented action layout without callbacks

Author: Alexander Dadukin

lib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/ActionParamsResolver.kt

@@ -17,6 +17,15 @@ internal class ActionParamsResolver {
         val shouldTruncateText: Boolean = false
     )
 
+    companion object {
+        val EMPTY_INFO = ActionInfo(
+            textSize = 14F,
+            actionHeight = 0,
+            actionWidth = 0,
+            maxLines = 1
+        )
+    }
+
     private val textPaint = TextPaint(TextPaint.ANTI_ALIAS_FLAG)
 
     fun obtainActionInfoFor(

lib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/ActionViewObserver.kt

@@ -0,0 +1,9 @@
+package github.com.st235.lib_swipetoactionlayout
+
+import android.view.View
+
+internal interface ActionViewObserver {
+    fun onInit(width: Int, height: Int, actions: List<SwipeAction>) { }
+    fun onMeasure(width: Int, height: Int, actions: List<SwipeAction>, views: List<View>, gravity: SwipeActionActionViewFactory.Gravity)
+    fun onLayout(width: Int, height: Int, actions: List<SwipeAction>, views: List<View>, gravity: SwipeActionActionViewFactory.Gravity)
+}

lib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/HudViewController.kt

@@ -13,34 +13,29 @@ import android.widget.FrameLayout
 
 internal class HudViewController(
     context: Context,
-    private val swipeActionViewFactory: SwipeActionViewFactory
-) {
+    private val swipeActionViewFactory: SwipeActionActionViewFactory
+): ActionViewObserver {
 
     private val hudView: ViewGroup = FrameLayout(context)
 
     private lateinit var leftActionView: View
     private lateinit var rightActionView: View
 
-    fun attachToParent(parent: ViewGroup, firstAction: SwipeAction) {
+    fun attachToParent(parent: ViewGroup, action: SwipeAction) {
         val lp = FrameLayout.LayoutParams(FrameLayout.LayoutParams.MATCH_PARENT, FrameLayout.LayoutParams.MATCH_PARENT)
         hudView.setBackgroundColor(Color.TRANSPARENT)
         parent.addView(hudView, lp)
 
-
-        val desiredSize = Math.min(parent.width, parent.height)
-
         leftActionView = swipeActionViewFactory.create(
-            action = firstAction,
-            actionInfo = swipeActionViewFactory.lastKnownActionInfo,
-            gravity = SwipeActionViewFactory.Gravity.LEFT
+            action = action,
+            gravity = SwipeActionActionViewFactory.Gravity.LEFT
         )
 
         leftActionView.visibility = View.INVISIBLE
 
         rightActionView = swipeActionViewFactory.create(
-            action = firstAction,
-            actionInfo = swipeActionViewFactory.lastKnownActionInfo,
-            gravity = SwipeActionViewFactory.Gravity.RIGHT
+            action = action,
+            gravity = SwipeActionActionViewFactory.Gravity.RIGHT
         )
 
         rightActionView.visibility = View.INVISIBLE
@@ -53,6 +48,29 @@ internal class HudViewController(
         parent.removeView(hudView)
     }
 
+    override fun onMeasure(
+        width: Int,
+        height: Int,
+        actions: List<SwipeAction>,
+        views: List<View>,
+        gravity: SwipeActionActionViewFactory.Gravity
+    ) {
+        hudView.measure(
+            View.MeasureSpec.makeMeasureSpec(width, View.MeasureSpec.EXACTLY),
+            View.MeasureSpec.makeMeasureSpec(height, View.MeasureSpec.EXACTLY)
+        )
+    }
+
+    override fun onLayout(
+        width: Int,
+        height: Int,
+        actions: List<SwipeAction>,
+        views: List<View>,
+        gravity: SwipeActionActionViewFactory.Gravity
+    ) {
+        hudView.layout(0, 0, width, height)
+    }
+
     fun hideIfNeeded() {
         if (hudView.visibility != View.INVISIBLE) {
             hudView.visibility = View.INVISIBLE

…toactionlayout/SwipeActionViewFactory.kt …onlayout/SwipeActionActionViewFactory.ktlib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/SwipeActionViewFactory.kt renamed to lib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/SwipeActionActionViewFactory.kt lib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/SwipeActionViewFactory.kt renamed to lib-swipetoactionlayout/src/main/java/github/com/st235/lib_swipetoactionlayout/SwipeActionActionViewFactory.kt

@@ -2,21 +2,23 @@ package github.com.st235.lib_swipetoactionlayout
 
 import android.annotation.SuppressLint
 import android.graphics.PorterDuff
-import android.graphics.Rect
 import android.text.TextUtils
 import android.view.View
 import android.widget.FrameLayout
 import android.widget.LinearLayout
 import android.widget.TextView
 import androidx.appcompat.widget.AppCompatImageView
+import github.com.st235.lib_swipetoactionlayout.ActionParamsResolver.Companion.EMPTY_INFO
 import github.com.st235.lib_swipetoactionlayout.utils.toPx
 
-internal class SwipeActionViewFactory(
+internal class SwipeActionActionViewFactory(
     private val swipeToActionLayout: SwipeToActionLayout,
     private val onActionClickListener: OnActionClickListener
-) {
+) : ActionViewObserver {
 
-    val HORIZONTAL_MARGIN_IN_DP = 8F
+    companion object {
+        const val HORIZONTAL_MARGIN_IN_DP = 8F
+    }
 
     @SuppressLint("RtlHardcoded")
     enum class Gravity(internal var androidGravity: Int) {
@@ -25,71 +27,107 @@ internal class SwipeActionViewFactory(
     }
 
     private val context = swipeToActionLayout.context
-    private val parentBounds = Rect()
     private val actionParamsResolver = ActionParamsResolver()
-    internal lateinit var lastKnownActionInfo: ActionParamsResolver.ActionInfo
-
-    fun onOwnerBoundsChanged(newWidth: Int, newHeight: Int) {
-        parentBounds.set(0, 0, newWidth, newHeight)
-    }
+    internal var lastKnownActionInfo = EMPTY_INFO
 
     fun createLayout(actions: List<SwipeAction>, gravity: Gravity): List<View> {
         val views = mutableListOf<View>()
-        val desiredSize = Math.min(parentBounds.width(), parentBounds.height())
-        lastKnownActionInfo = actionParamsResolver.obtainActionInfoFor(
-            (parentBounds.width() * 0.9F).toInt(),
-            Math.min(desiredSize, parentBounds.width() / actions.size), desiredSize, HORIZONTAL_MARGIN_IN_DP, actions
-        )
-
-        var margin = (actions.size - 1) * desiredSize
 
         if (gravity == Gravity.LEFT) {
             for (i in (actions.size - 1) downTo 0) {
-                val action = actions[i]
-
-                val actionView = create(action, lastKnownActionInfo, margin, 0, gravity)
+                val actionView = create(actions[i], gravity)
                 swipeToActionLayout.addView(actionView, actionView.layoutParams)
 
                 views.add(0, actionView)
-
-                margin -= desiredSize
             }
         }
 
-        margin = (actions.size - 1) * desiredSize
-
         if (gravity == Gravity.RIGHT) {
             for (i in 0 until actions.size) {
-                val action = actions[i]
-
-                val actionView = create(action, lastKnownActionInfo, 0, margin, gravity)
+                val actionView = create(actions[i], gravity)
                 swipeToActionLayout.addView(actionView, actionView.layoutParams)
 
                 views.add(actionView)
-                actionView.isClickable = true
-                actionView.isFocusable = true
-
-                margin -= desiredSize
             }
         }
 
         return views
     }
 
+    override fun onInit(width: Int, height: Int, actions: List<SwipeAction>) {
+        val desiredSize = Math.min(width, height)
+        lastKnownActionInfo = actionParamsResolver.obtainActionInfoFor(
+            (width * 0.9F).toInt(),
+            Math.min(desiredSize, height / actions.size), desiredSize, HORIZONTAL_MARGIN_IN_DP, actions
+        )
+    }
+
+    override fun onMeasure(
+        width: Int,
+        height: Int,
+        actions: List<SwipeAction>,
+        views: List<View>,
+        gravity: Gravity
+    ) {
+        if (gravity == Gravity.LEFT) {
+            for (i in (actions.size - 1) downTo 0) {
+                val actionView = views[i]
+                actionView.measure(
+                    View.MeasureSpec.makeMeasureSpec(lastKnownActionInfo.actionWidth, View.MeasureSpec.EXACTLY),
+                    View.MeasureSpec.makeMeasureSpec(lastKnownActionInfo.actionHeight, View.MeasureSpec.EXACTLY)
+                )
+            }
+        }
+
+        if (gravity == Gravity.RIGHT) {
+            for (i in 0 until actions.size) {
+                val actionView = views[i]
+                actionView.measure(
+                    View.MeasureSpec.makeMeasureSpec(lastKnownActionInfo.actionWidth, View.MeasureSpec.EXACTLY),
+                    View.MeasureSpec.makeMeasureSpec(lastKnownActionInfo.actionHeight, View.MeasureSpec.EXACTLY)
+                )
+            }
+        }
+    }
+
+    override fun onLayout(
+        width: Int,
+        height: Int,
+        actions: List<SwipeAction>,
+        views: List<View>,
+        gravity: Gravity
+    ) {
+        var margin = (actions.size - 1) * lastKnownActionInfo.actionWidth
+        if (gravity == Gravity.LEFT) {
+            for (i in (actions.size - 1) downTo 0) {
+                val actionView = views[i]
+                actionView.layout(margin, 0, margin + lastKnownActionInfo.actionWidth, height)
+                margin -= lastKnownActionInfo.actionWidth
+            }
+        }
+
+        margin = (actions.size - 1) * lastKnownActionInfo.actionWidth
+
+        if (gravity == Gravity.RIGHT) {
+            for (i in 0 until actions.size) {
+                val actionView = views[i]
+                actionView.layout(width - margin - lastKnownActionInfo.actionWidth, 0, width - margin, height)
+                margin -= lastKnownActionInfo.actionWidth
+            }
+        }
+    }
+
     internal fun create(
         action: SwipeAction,
-        actionInfo: ActionParamsResolver.ActionInfo,
-        marginLeft: Int = 0,
-        marginRight: Int = 0,
         gravity: Gravity
     ): View {
         val actionView = LinearLayout(context)
         actionView.orientation = LinearLayout.VERTICAL
         actionView.setVerticalGravity(android.view.Gravity.CENTER_VERTICAL)
         actionView.setHorizontalGravity(android.view.Gravity.CENTER_HORIZONTAL)
-        actionView.setBackgroundColor(action.color)
         actionView.isFocusable = true
         actionView.isClickable = true
+        actionView.setBackgroundColor(action.color)
         actionView.setOnClickListener { onActionClickListener(actionView, action) }
 
         val iconLayoutParams = LinearLayout.LayoutParams(
@@ -107,11 +145,12 @@ internal class SwipeActionViewFactory(
         textLayoutParams.rightMargin = HORIZONTAL_MARGIN_IN_DP.toPx().toInt()
 
         actionView.addView(createIconView(action), iconLayoutParams)
-        actionView.addView(createTextView(action, actionInfo), textLayoutParams)
+        actionView.addView(createTextView(action, lastKnownActionInfo), textLayoutParams)
 
-        val actionLayoutParams = FrameLayout.LayoutParams(actionInfo.actionWidth, actionInfo.actionHeight)
-        actionLayoutParams.leftMargin = marginLeft
-        actionLayoutParams.rightMargin = marginRight
+        val actionLayoutParams =
+            FrameLayout.LayoutParams(LinearLayout.LayoutParams.WRAP_CONTENT, LinearLayout.LayoutParams.MATCH_PARENT)
+        actionLayoutParams.leftMargin = 0
+        actionLayoutParams.rightMargin = 0
         actionLayoutParams.gravity = gravity.androidGravity
 
         actionView.layoutParams = actionLayoutParams
@@ -126,13 +165,16 @@ internal class SwipeActionViewFactory(
         return iconView
     }
 
-    private fun createTextView(action: SwipeAction, actionInfo: ActionParamsResolver.ActionInfo): View {
+    private fun createTextView(
+        action: SwipeAction,
+        actionInfo: ActionParamsResolver.ActionInfo
+    ): View {
         val textView = TextView(context)
         textView.text = action.text
         textView.setTextColor(action.textColor)
         textView.textSize = actionInfo.textSize
-        textView.gravity = android.view.Gravity.CENTER
         textView.maxLines = actionInfo.maxLines
+        textView.gravity = android.view.Gravity.CENTER
         textView.setLineSpacing(0F, 0.9F)
 
         if (actionInfo.shouldTruncateText) {