Use kwargs within Navigation classes (#478)

lib/trestle/navigation/block.rb

@@ -28,12 +28,7 @@ def default_path
           @admin ? @admin.path : nil
         end
 
-        def item(name, path=nil, options={})
-          if path.is_a?(Hash)
-            options = path
-            path = nil
-          end
-
+        def item(name, path=nil, **options)
           if options[:group]
             group = Group.new(options[:group])
           elsif @current_group
@@ -43,11 +38,11 @@ def item(name, path=nil, options={})
           options = options.merge(group: group) if group
           options = options.merge(admin: @admin) if @admin
 
-          items << Item.new(name, path || default_path, options)
+          items << Item.new(name, path || default_path, **options)
         end
 
-        def group(name, options={})
-          @current_group = Group.new(name, options)
+        def group(name, **options)
+          @current_group = Group.new(name, **options)
           yield
         ensure
           @current_group = nil

lib/trestle/navigation/group.rb

@@ -3,7 +3,7 @@ class Navigation
     class Group
       attr_reader :name, :options
 
-      def initialize(name, options={})
+      def initialize(name, **options)
         @name, @options = name.to_s, options
       end
 
@@ -26,7 +26,7 @@ def <=>(other)
       end
 
       def merge(other)
-        self.class.new(name, options.merge(other.options))
+        self.class.new(name, **options.merge(other.options))
       end
 
       def priority

lib/trestle/navigation/item.rb

@@ -3,7 +3,7 @@ class Navigation
     class Item
       attr_reader :name, :path, :options
 
-      def initialize(name, path=nil, options={})
+      def initialize(name, path=nil, **options)
         @name, @path, @options = name.to_s, path, options
       end