replace public_send with tag

Author: lsouoliveira

lib/ruby_ui/sidebar/sidebar_group_action.rb

@@ -2,13 +2,13 @@
 
 module RubyUI
   class SidebarGroupAction < Base
-    def initialize(as: "button", **attrs)
+    def initialize(as: :button, **attrs)
       @as = as
       super(**attrs)
     end
 
     def view_template(&)
-      public_send(@as, **attrs, &)
+      tag(@as, **attrs, &)
     end
 
     private

lib/ruby_ui/sidebar/sidebar_menu_action.rb

@@ -2,13 +2,13 @@
 
 module RubyUI
   class SidebarMenuAction < Base
-    def initialize(as: "button", show_on_hover: false, **attrs)
+    def initialize(as: :button, show_on_hover: false, **attrs)
       @as = as
       super(**attrs)
     end
 
     def view_template(&)
-      public_send(@as, **attrs, &)
+      tag(@as, **attrs, &)
     end
 
     private

lib/ruby_ui/sidebar/sidebar_menu_button.rb

@@ -3,18 +3,18 @@
 module RubyUI
   class SidebarMenuButton < Base
     VARIANT_CLASSES = {
-      default: "hover:bg-sidebar-accent hover:text-sidebar-accent-foreground",
+      default: 'hover:bg-sidebar-accent hover:text-sidebar-accent-foreground',
       outline:
-        "bg-background shadow-[0_0_0_1px_hsl(var(--sidebar-border))] hover:bg-sidebar-accent hover:text-sidebar-accent-foreground hover:shadow-[0_0_0_1px_hsl(var(--sidebar-accent))]"
+        'bg-background shadow-[0_0_0_1px_hsl(var(--sidebar-border))] hover:bg-sidebar-accent hover:text-sidebar-accent-foreground hover:shadow-[0_0_0_1px_hsl(var(--sidebar-accent))]'
     }.freeze
 
     SIZE_CLASSES = {
-      default: "h-8 text-sm",
-      sm: "h-7 text-xs",
-      lg: "h-12 text-sm group-data-[collapsible=icon]:!p-0"
+      default: 'h-8 text-sm',
+      sm: 'h-7 text-xs',
+      lg: 'h-12 text-sm group-data-[collapsible=icon]:!p-0'
     }.freeze
 
-    def initialize(as: "button", variant: :default, size: :default, active: false, **attrs)
+    def initialize(as: :button, variant: :default, size: :default, active: false, **attrs)
       raise ArgumentError, "Invalid variant: #{variant}" unless VARIANT_CLASSES.key?(variant)
       raise ArgumentError, "Invalid size: #{size}" unless SIZE_CLASSES.key?(size)
 
@@ -26,20 +26,34 @@ def initialize(as: "button", variant: :default, size: :default, active: false, *
     end
 
     def view_template(&)
-      public_send(@as, **attrs, &)
+      tag(@as, **attrs, &)
     end
 
     private
 
     def default_attrs
       {
         class: [
-          "peer/menu-button flex w-full items-center gap-2 overflow-hidden rounded-md p-2 text-left text-sm outline-none ring-sidebar-ring transition-[width,height,padding] hover:bg-sidebar-accent hover:text-sidebar-accent-foreground focus-visible:ring-2 active:bg-sidebar-accent active:text-sidebar-accent-foreground disabled:pointer-events-none disabled:opacity-50 group-has-[[data-sidebar=menu-action]]/menu-item:pr-8 aria-disabled:pointer-events-none aria-disabled:opacity-50 data-[active=true]:bg-sidebar-accent data-[active=true]:font-medium data-[active=true]:text-sidebar-accent-foreground data-[state=open]:hover:bg-sidebar-accent data-[state=open]:hover:text-sidebar-accent-foreground group-data-[collapsible=icon]:!size-8 group-data-[collapsible=icon]:!p-2 [&>span:last-child]:truncate [&>svg]:size-4 [&>svg]:shrink-0",
+          'peer/menu-button flex w-full items-center gap-2 overflow-hidden',
+          'rounded-md p-2 text-left text-sm outline-none ring-sidebar-ring',
+          'transition-[width,height,padding] hover:bg-sidebar-accent',
+          'hover:text-sidebar-accent-foreground focus-visible:ring-2',
+          'active:bg-sidebar-accent active:text-sidebar-accent-foreground',
+          'disabled:pointer-events-none disabled:opacity-50',
+          'group-has-[[data-sidebar=menu-action]]/menu-item:pr-8',
+          'aria-disabled:pointer-events-none aria-disabled:opacity-50',
+          'data-[active=true]:bg-sidebar-accent data-[active=true]:font-medium',
+          'data-[active=true]:text-sidebar-accent-foreground',
+          'data-[state=open]:hover:bg-sidebar-accent',
+          'data-[state=open]:hover:text-sidebar-accent-foreground',
+          'group-data-[collapsible=icon]:!size-8',
+          'group-data-[collapsible=icon]:!p-2 [&>span:last-child]:truncate',
+          '[&>svg]:size-4 [&>svg]:shrink-0',
           VARIANT_CLASSES[@variant],
           SIZE_CLASSES[@size]
         ],
         data: {
-          sidebar: "menu-button",
+          sidebar: 'menu-button',
           size: @size,
           active: @active.to_s
         }

lib/ruby_ui/sidebar/sidebar_menu_sub_button.rb

@@ -7,7 +7,7 @@ class SidebarMenuSubButton < Base
       md: "text-sm"
     }.freeze
 
-    def initialize(as: "button", size: :md, active: false, **attrs)
+    def initialize(as: :button, size: :md, active: false, **attrs)
       raise ArgumentError, "Invalid size: #{size}" unless SIZE_CLASSES.key?(size)
 
       @as = as
@@ -17,7 +17,7 @@ def initialize(as: "button", size: :md, active: false, **attrs)
     end
 
     def view_template(&)
-      public_send(@as, **attrs, &)
+      tag(@as, **attrs, &)
     end
 
     private

test/ruby_ui/sidebar_test.rb

@@ -23,12 +23,12 @@ def test_render_with_all_items
                   RubyUI.SidebarMenuItem do
                     RubyUI.SidebarMenuSub do
                       RubyUI.SidebarMenuSubItem do
-                        RubyUI.SidebarMenuSubButton(as: "a", href: "#") { "Sub Item 1" }
+                        RubyUI.SidebarMenuSubButton(as: :a, href: "#") { "Sub Item 1" }
                       end
                     end
                   end
                   RubyUI.SidebarMenuItem do
-                    RubyUI.SidebarMenuButton(as: "a", href: "#") { "Settings" }
+                    RubyUI.SidebarMenuButton(as: :a, href: "#") { "Settings" }
                     RubyUI.SidebarMenuAction { "Settings" }
                   end
                   RubyUI.SidebarMenuItem do