Made respondsToPointerEvents() more resilient to execution off-EDT by only calling isScrollable() if it is running on the EDT. This is because isScrollable may trigger some mutations from the UI while it determines if the component is scrollable. This likely fixes the issue that #3323 is meant to address.

shannah · shannah · commit 50d1ccc156fc · 2020-12-08T09:34:45.000-08:00
diff --git a/CodenameOne/src/com/codename1/ui/Component.java b/CodenameOne/src/com/codename1/ui/Component.java
@@ -4299,7 +4299,8 @@ public void addPullToRefresh(Runnable task){
      * @return True if the pointer responds to pointer events.
      */
     public boolean respondsToPointerEvents() {
-        return isVisible() && isEnabled() && (isScrollable() || isFocusable() || isGrabsPointerEvents() || isDraggable());
+        boolean isScrollable = CN.isEdt() ? isScrollable() : (scrollableXFlag() || scrollableYFlag());
+        return isVisible() && isEnabled() && (isScrollable || isFocusable() || isGrabsPointerEvents() || isDraggable());
     }
     
     private boolean pointerReleaseMaterialPullToRefresh() {

Original file line number	Diff line number	Diff line change
`@@ -4299,7 +4299,8 @@ public void addPullToRefresh(Runnable task){`
`4299`	`4299`	`* @return True if the pointer responds to pointer events.`
`4300`	`4300`	`*/`
`4301`	`4301`	`public boolean respondsToPointerEvents() {`
`4302`		`- return isVisible() && isEnabled() && (isScrollable() \|\| isFocusable() \|\| isGrabsPointerEvents() \|\| isDraggable());`
	`4302`	`+ boolean isScrollable = CN.isEdt() ? isScrollable() : (scrollableXFlag() \|\| scrollableYFlag());`
	`4303`	`+ return isVisible() && isEnabled() && (isScrollable \|\| isFocusable() \|\| isGrabsPointerEvents() \|\| isDraggable());`
`4303`	`4304`	`}`
`4304`	`4305`
`4305`	`4306`	`private boolean pointerReleaseMaterialPullToRefresh() {`