ADFA-2128: Handle IllegalArgumentException when previewing NavigationView

layouteditor/src/main/assets/palette/layouts.json

@@ -89,5 +89,15 @@
             "android:layout_width": "match_parent",
             "android:padding": "8dp"
         }
+    },
+    {
+        "name": "DrawerLayout",
+        "className": "org.appdevforall.codeonthego.layouteditor.editor.palette.layouts.DrawerLayoutDesign",
+        "iconName": "ic_palette_drawer_layout",
+        "defaultAttributes": {
+            "android:layout_width": "match_parent",
+            "android:layout_height": "match_parent",
+            "android:padding": "8dp"
+        }
     }
 ]

layouteditor/src/main/assets/widgetclasses.json

@@ -31,6 +31,7 @@
     "FrameLayout": "org.appdevforall.codeonthego.layouteditor.editor.palette.layouts.FrameLayoutDesign",
     "TableLayout": "org.appdevforall.codeonthego.layouteditor.editor.palette.layouts.TableLayoutDesign",
     "TableRow": "org.appdevforall.codeonthego.layouteditor.editor.palette.layouts.TableRowDesign",
+    "DrawerLayout":"org.appdevforall.codeonthego.layouteditor.editor.palette.layouts.DrawerLayoutDesign",
     "Space": "android.widget.Space",
     "GridLayout": "org.appdevforall.codeonthego.layouteditor.editor.palette.legacy.GridLayoutDesign",
     "TabHost": "org.appdevforall.codeonthego.layouteditor.editor.palette.legacy.TabHostDesign",

layouteditor/src/main/java/org/appdevforall/codeonthego/layouteditor/editor/palette/containers/NavigationViewDesign.java

@@ -1,42 +1,57 @@
 package org.appdevforall.codeonthego.layouteditor.editor.palette.containers;
 
-import com.google.android.material.navigation.NavigationView;
 import android.content.Context;
 import android.graphics.Canvas;
+import com.google.android.material.navigation.NavigationView;
 import org.appdevforall.codeonthego.layouteditor.utils.Constants;
 import org.appdevforall.codeonthego.layouteditor.utils.Utils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class NavigationViewDesign extends NavigationView {
-
-  private boolean drawStrokeEnabled;
-  private boolean isBlueprint;
-
-  public NavigationViewDesign(Context context) {
-    super(context);
-  }
-
-  @Override
-  protected void dispatchDraw(Canvas canvas) {
-    super.dispatchDraw(canvas);
-
-    if (drawStrokeEnabled)
-      Utils.drawDashPathStroke(
-          this, canvas, isBlueprint ? Constants.BLUEPRINT_DASH_COLOR : Constants.DESIGN_DASH_COLOR);
-  }
-
-  public void setStrokeEnabled(boolean enabled) {
-    drawStrokeEnabled = enabled;
-    invalidate();
-  }
-
-  @Override
-  public void draw(Canvas canvas) {
-    if (isBlueprint) Utils.drawDashPathStroke(this, canvas, Constants.BLUEPRINT_DASH_COLOR);
-    else super.draw(canvas);
-  }
-
-  public void setBlueprint(boolean isBlueprint) {
-    this.isBlueprint = isBlueprint;
-    invalidate();
-  }
+
+	private boolean drawStrokeEnabled;
+	private boolean isBlueprint;
+
+	private final Logger logger = LoggerFactory.getLogger(NavigationViewDesign.class);
+
+	public NavigationViewDesign(Context context) {
+		super(context);
+	}
+
+	@Override
+	public void draw(Canvas canvas) {
+		if (isBlueprint)
+			Utils.drawDashPathStroke(this, canvas, Constants.BLUEPRINT_DASH_COLOR);
+		else
+			super.draw(canvas);
+	}
+
+	public void setBlueprint(boolean isBlueprint) {
+		this.isBlueprint = isBlueprint;
+		invalidate();
+	}
+
+	public void setStrokeEnabled(boolean enabled) {
+		drawStrokeEnabled = enabled;
+		invalidate();
+	}
+
+	@Override
+	protected void dispatchDraw(Canvas canvas) {
+		super.dispatchDraw(canvas);
+
+		if (drawStrokeEnabled)
+			Utils.drawDashPathStroke(
+					this, canvas, isBlueprint ? Constants.BLUEPRINT_DASH_COLOR : Constants.DESIGN_DASH_COLOR);
+	}
+
+	@Override
+	protected void onAttachedToWindow() {
+		try {
+			super.onAttachedToWindow();
+		} catch (IllegalArgumentException e) {
+			logger.error("NavigationView should be placed in a DrawerLayout", e);
+		}
+	}
 }

layouteditor/src/main/java/org/appdevforall/codeonthego/layouteditor/editor/palette/layouts/DrawerLayoutDesign.kt

@@ -0,0 +1,56 @@
+package org.appdevforall.codeonthego.layouteditor.editor.palette.layouts
+
+import android.content.Context
+import android.graphics.Canvas
+import androidx.drawerlayout.widget.DrawerLayout
+import org.appdevforall.codeonthego.layouteditor.utils.Constants
+import org.appdevforall.codeonthego.layouteditor.utils.Utils
+import org.slf4j.Logger
+import org.slf4j.LoggerFactory
+
+class DrawerLayoutDesign(
+	context: Context,
+) : DrawerLayout(context) {
+	private var drawStrokeEnabled = false
+	private var isBlueprint = false
+
+	private val logger: Logger = LoggerFactory.getLogger(DrawerLayoutDesign::class.java)
+
+	override fun dispatchDraw(canvas: Canvas) {
+		super.dispatchDraw(canvas)
+
+		if (drawStrokeEnabled) {
+			Utils.drawDashPathStroke(
+				this,
+				canvas,
+				if (isBlueprint) Constants.BLUEPRINT_DASH_COLOR else Constants.DESIGN_DASH_COLOR,
+			)
+		}
+	}
+
+	override fun draw(canvas: Canvas) {
+		if (isBlueprint) {
+			Utils.drawDashPathStroke(this, canvas, Constants.BLUEPRINT_DASH_COLOR)
+		} else {
+			super.draw(canvas)
+		}
+	}
+
+	fun setStrokeEnabled(enabled: Boolean) {
+		drawStrokeEnabled = enabled
+		invalidate()
+	}
+
+	fun setBlueprint(isBlueprint: Boolean) {
+		this.isBlueprint = isBlueprint
+		invalidate()
+	}
+
+	override fun onAttachedToWindow() {
+		try {
+			super.onAttachedToWindow()
+		} catch (e: Exception) {
+			logger.error("Error in previewing DrawerLayoutDesign", e)
+		}
+	}
+}