Fix menu sub_items and sidebar hide_position crashes

Fixed #467

Sidebar component:
  - Add default: nil with values: ["left", "right", nil] to hide_position attr
  - Handle nil position in show_sidebar/hide_sidebar to return a no-op instead of crashing

Author: shahryarjb

priv/components/sidebar.eex

@@ -60,8 +60,9 @@ defmodule <%= @module %> do
   attr :position, :string, default: "start", doc: "Determines the element position"
 
   attr :hide_position, :string,
-    values: ["left", "right"],
-    doc: "Determines what position should be hidden"
+    default: nil,
+    values: ["left", "right", nil],
+    doc: "Determines what position should be hidden, accepts \"left\" or \"right\""
 
   attr :class, :any, default: nil, doc: "Custom CSS class for additional styling"
   attr :hide_button_class, :string, default: nil, doc: "Custom CSS class for additional styling"
@@ -191,8 +192,10 @@ defmodule <%= @module %> do
   """
 
   def show_sidebar(js \\ %JS{}, id, position) when is_binary(id) do
-    JS.remove_class(js, hide_position(position), to: "##{id}")
-    |> JS.add_class("transform-none", to: "##{id}")
+    case hide_position(position) do
+      nil -> js
+      classes -> JS.remove_class(js, classes, to: "##{id}") |> JS.add_class("transform-none", to: "##{id}")
+    end
   end
 
   @doc """
@@ -218,8 +221,10 @@ defmodule <%= @module %> do
   """
 
   def hide_sidebar(js \\ %JS{}, id, position) do
-    JS.remove_class(js, "transform-none", to: "##{id}")
-    |> JS.add_class(hide_position(position), to: "##{id}")
+    case hide_position(position) do
+      nil -> js
+      classes -> JS.remove_class(js, "transform-none", to: "##{id}") |> JS.add_class(classes, to: "##{id}")
+    end
   end
 
   defp hide_position("left"), do: "-translate-x-full md:translate-x-0"