Calling open(), close(), or toggle() before SlideHolder is layouted now delays execution of this methods (it crashed before)

Author: dmitry-zaitsev

example/src/com/agimind/widget/SlideHolder.java

@@ -16,6 +16,9 @@
 
 package com.agimind.widget;
 
+import java.util.LinkedList;
+import java.util.Queue;
+
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
@@ -55,6 +58,8 @@ public class SlideHolder extends FrameLayout {
 	private boolean mInterceptTouch = true;
 	private boolean mAlwaysOpened = false;
 
+	private Queue<Runnable> mWhenReady = new LinkedList<Runnable>();
+	
 	private OnSlideListener mListener;
 
 	public SlideHolder(Context context) {
@@ -150,6 +155,18 @@ public boolean open() {
 			return false;
 		}
 
+		if(!isReadyForSlide()) {
+			mWhenReady.add(new Runnable() {
+				
+				@Override
+				public void run() {
+					open();
+				}
+			});
+			
+			return true;
+		}
+		
 		initSlideMode();
 
 		Animation anim = new SlideAnimation(mOffset, mEndOffset);
@@ -182,6 +199,18 @@ public boolean close() {
 			return false;
 		}
 
+		if(!isReadyForSlide()) {
+			mWhenReady.add(new Runnable() {
+				
+				@Override
+				public void run() {
+					close();
+				}
+			});
+			
+			return true;
+		}
+		
 		initSlideMode();
 
 		Animation anim = new SlideAnimation(mOffset, mEndOffset);
@@ -246,6 +275,15 @@ protected void onLayout(boolean changed, int l, int t, int r, int b) {
 				);
 
 		invalidate();
+		
+		Runnable rn;
+        while((rn = mWhenReady.poll()) != null) {
+        	rn.run();
+        }
+	}
+	
+	private boolean isReadyForSlide() {
+		return (getWidth() > 0 && getHeight() > 0);
 	}
 
 	@Override

sidemenu.jar

@@ -16,6 +16,9 @@
 
 package com.agimind.widget;
 
+import java.util.LinkedList;
+import java.util.Queue;
+
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.Canvas;
@@ -55,6 +58,8 @@ public class SlideHolder extends FrameLayout {
 	private boolean mInterceptTouch = true;
 	private boolean mAlwaysOpened = false;
 
+	private Queue<Runnable> mWhenReady = new LinkedList<Runnable>();
+	
 	private OnSlideListener mListener;
 
 	public SlideHolder(Context context) {
@@ -150,6 +155,18 @@ public boolean open() {
 			return false;
 		}
 
+		if(!isReadyForSlide()) {
+			mWhenReady.add(new Runnable() {
+				
+				@Override
+				public void run() {
+					open();
+				}
+			});
+			
+			return true;
+		}
+		
 		initSlideMode();
 
 		Animation anim = new SlideAnimation(mOffset, mEndOffset);
@@ -182,6 +199,18 @@ public boolean close() {
 			return false;
 		}
 
+		if(!isReadyForSlide()) {
+			mWhenReady.add(new Runnable() {
+				
+				@Override
+				public void run() {
+					close();
+				}
+			});
+			
+			return true;
+		}
+		
 		initSlideMode();
 
 		Animation anim = new SlideAnimation(mOffset, mEndOffset);
@@ -246,6 +275,15 @@ protected void onLayout(boolean changed, int l, int t, int r, int b) {
 				);
 
 		invalidate();
+		
+		Runnable rn;
+        while((rn = mWhenReady.poll()) != null) {
+        	rn.run();
+        }
+	}
+	
+	private boolean isReadyForSlide() {
+		return (getWidth() > 0 && getHeight() > 0);
 	}
 
 	@Override