ui_bibz 2.5.2 → 3.0.0.alpha3

data/lib/ui_bibz/ui/core/lists/components/list.rb

@@ -20,7 +20,6 @@ module UiBibz::Ui::Core::Lists::Components
   # * +state+ - status of element with symbol value:
   #   (+:active+, +:disabled+)
   # * +url+
-  # * +tap+ - Boolean to add header and body
   # * +status+ - status of element with symbol value:
   #   (+:default+, +:primary+, +:info+, +:warning+, +:danger+)
   # * +glyph+ - Add glyph with name or hash options
@@ -30,7 +29,7 @@ module UiBibz::Ui::Core::Lists::Components
   #
   # ==== Signatures
   #
-  #   UiBibz::Ui::Core::Lists::Components::List.new(tap: true) do |l|
+  #   UiBibz::Ui::Core::Lists::Components::List.new do |l|
   #     l.header content =  nil, options = nil, html_options = nil, &block
   #     l.body content =  nil, options = nil, html_options = nil, &block
   #   end
@@ -49,7 +48,7 @@ module UiBibz::Ui::Core::Lists::Components
   #       'Test 2'
   #   end.render
   #
-  #   UiBibz::Ui::Core::List.new(tap: true, state: :active) do |l|
+  #   UiBibz::Ui::Core::List.new(state: :active) do |l|
   #     l.header 'My title', nil, class: 'my-title'
   #     l.body do
   #       'My content'
@@ -57,16 +56,15 @@ module UiBibz::Ui::Core::Lists::Components
   #   end.render
   #
   class List < UiBibz::Ui::Core::Component
+    include UiBibz::Ui::Concerns::HtmlConcern
+
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render
       content_tag tag_type, html_options do
         concat glyph_and_content_html if @content
-        concat header_html if @body
+        concat header_html if @header
         concat body_html   if @body
         concat badge_html  if @options[:badge]
       end
@@ -105,7 +103,7 @@ module UiBibz::Ui::Core::Lists::Components
     end
 
     def header_classes
-      'flex-column align-items-start' unless options[:tap].nil?
+      'flex-column align-items-start' if @options[:tap]
     end
 
     def tag_type_class

data/lib/ui_bibz/ui/core/lists/components/list/list_body.rb

@@ -17,9 +17,6 @@ module UiBibz::Ui::Core::Lists::Components
   # You can pass arguments in options attribute:
   class ListBody < UiBibz::Ui::Core::Component
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/lists/components/list/list_header.rb

@@ -17,9 +17,6 @@ module UiBibz::Ui::Core::Lists::Components
   # You can pass arguments in options attribute:
   class ListHeader < UiBibz::Ui::Core::Component
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/lists/list_group.rb

@@ -22,7 +22,7 @@ module UiBibz::Ui::Core::Lists
   #
   # ==== Signatures
   #
-  #   UiBibz::Ui::Core::ListGroup.new().tap |lg|
+  #   UiBibz::Ui::Core::ListGroup.new.tap |lg|
   #     ...
   #     lg.list content = nil, options = nil, html_options = nil, &block
   #     ...
@@ -30,7 +30,7 @@ module UiBibz::Ui::Core::Lists
   #
   # ==== Examples
   #
-  #   UiBibz::Ui::Core::ListGroup.new().tap do |d|
+  #   UiBibz::Ui::Core::ListGroup.new.tap do |d|
   #     d.list 'Test', status: :success
   #     d.list 'Test2', status: :primary
   #   end.render
@@ -40,7 +40,7 @@ module UiBibz::Ui::Core::Lists
   #     d.list(status: :primary) do
   #       'Test 2'
   #     end
-  #     d.list(tap: true, state: :active) do |l|
+  #     d.list(state: :active) do |l|
   #       l.header 'My title', nil, class: 'my-title'
   #       l.body do
   #         'My content'
@@ -50,12 +50,12 @@ module UiBibz::Ui::Core::Lists
   #
   # ==== Helper
   #
-  #   list_group( options = { tap: true }, html_options = {}) do |l|
+  #   list_group( options = {}, html_options = {}) do |l|
   #     l.list(content, options = {}, html_options = {})
   #     l.list(options = {}, html_options = {}) do
   #       content
   #     end
-  #     l.list(options = { tap: true }, html_options = {}) do |li|
+  #     l.list(options = {}, html_options = {}) do |li|
   #       li.header(content, options = {}, html_options = {})
   #       # or
   #       li.header(options = {}, html_options = {}) do
@@ -71,6 +71,8 @@ module UiBibz::Ui::Core::Lists
   #   end
   #
   class ListGroup < UiBibz::Ui::Core::Component
+    include UiBibz::Ui::Concerns::HtmlConcern
+
     # See UiBibz::Ui::Core::Component.initialize
     def initialize(content = nil, options = nil, html_options = nil, &block)
       super
@@ -87,7 +89,8 @@ module UiBibz::Ui::Core::Lists
     def list(content = nil, options = {}, html_options = nil, &block)
       options = options.merge({ tag_type: @options[:tag_type] }) unless @options[:tag_type].nil?
 
-      @lists << if tap?(content, options)
+      @lists << if tapped?(block)
+                  content = (content || {}).merge(tap: true)
                   UiBibz::Ui::Core::Lists::Components::List.new(content, options, html_options).tap(&block).render
                 else
                   UiBibz::Ui::Core::Lists::Components::List.new(content, options, html_options, &block).render

data/lib/ui_bibz/ui/core/navigations/breadcrumb.rb

@@ -8,7 +8,6 @@ module UiBibz::Ui::Core::Navigations
   # Separators are automatically added in CSS through +:before+ and +content+.
   #
   # This element is an extend of UiBibz::Ui::Core::Component.
-  # You can use tap method to add link items.
   #
   # ==== Attributes
   #
@@ -56,7 +55,7 @@ module UiBibz::Ui::Core::Navigations
   #
   # ==== Helper
   #
-  #   ui_breadcrumb({tap: true}) do |b|
+  #   ui_breadcrumb do |b|
   #     b.link(content, options = {}, html_options = {})
   #     b.link(options = {}, html_options = {}) do
   #       content
@@ -69,6 +68,8 @@ module UiBibz::Ui::Core::Navigations
   #   ui_breadcrumb(@users, { link_label: name, link_url: user_path(:id) })
   #
   class Breadcrumb < UiBibz::Ui::Core::Component
+    include UiBibz::Ui::Concerns::HtmlConcern
+
     # See UiBibz::Ui::Core::Component.initialize
     def initialize(content = nil, options = nil, html_options = nil, &block)
       super

data/lib/ui_bibz/ui/core/navigations/components/breadcrumb_link.rb

@@ -41,9 +41,6 @@ module UiBibz::Ui::Core::Navigations::Components
   #
   class BreadcrumbLink < UiBibz::Ui::Core::Navigations::Link
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/navigations/components/nav_dropdown.rb

@@ -44,9 +44,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class NavDropdown < UiBibz::Ui::Core::Forms::Dropdowns::Dropdown
     # See UiBibz::Ui::Core::Dropdown.initialize
-    def initialize(content, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render
@@ -63,7 +60,7 @@ module UiBibz::Ui::Core::Navigations
     end
 
     def link_html
-      link_to button_content, '#', class: join_classes(size, 'dropdown-toggle', 'nav-link'), 'data-toggle' => 'dropdown', 'aria-expanded' => false
+      link_to button_content, '#', class: join_classes(size, state, 'dropdown-toggle', 'nav-link'), 'data-toggle' => 'dropdown', 'aria-expanded' => false
     end
   end
 end

data/lib/ui_bibz/ui/core/navigations/components/nav_link.rb

@@ -41,6 +41,8 @@ module UiBibz::Ui::Core::Navigations
   #   end.render
   #
   class NavLink < UiBibz::Ui::Core::Component
+    include UiBibz::Ui::Concerns::HtmlConcern
+
     # See UiBibz::Ui::Core::Component.initialize
     def initialize(content = nil, options = nil, html_options = nil, &block)
       super
@@ -49,7 +51,7 @@ module UiBibz::Ui::Core::Navigations
 
     # Render html tag
     def pre_render
-      if options[:nav_tags] == :a || options[:nav_type] == 'nav-links'
+      if options[:nav_tags] == :a || %w[nav-links list-group].include?(options[:nav_type])
         UiBibz::Ui::Core::Navigations::NavLinkLink.new(content, options, html_options).render
       else
         cont = if options[:tag_type] == :span

data/lib/ui_bibz/ui/core/navigations/components/nav_link_link.rb

@@ -20,9 +20,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class NavLinkLink < UiBibz::Ui::Core::Component
     # UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render
@@ -35,11 +32,23 @@ module UiBibz::Ui::Core::Navigations
     private
 
     def component_html_classes
-      'nav-link'
+      if options[:nav_type] == 'list-group'
+        'list-group-item list-group-item-action'
+      else
+        'nav-link'
+      end
     end
 
     def component_html_options
-      options[:nav_type] == 'nav-tabs' ? { 'data-toggle' => 'tab', role: 'tab' } : {}
+      html_tag_base = { 'data-toggle' => 'tab', role: 'tab' }
+      case options[:nav_type]
+      when 'nav-tabs'
+        html_tag_base
+      when 'list-group'
+        html_tag_base.merge("aria-controls": sanitize_text(content))
+      else
+        {}
+      end.merge(options[:a_html] || {})
     end
   end
 end

data/lib/ui_bibz/ui/core/navigations/components/nav_link_list.rb

@@ -16,10 +16,6 @@ module UiBibz::Ui::Core::Navigations
   # You can add HTML attributes using the +html_options+.
   # You can pass arguments in options attribute:
   class NavLinkList < UiBibz::Ui::Core::Component
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
-
     # Render html tag
     def pre_render
       content_tag :li, content, html_options

data/lib/ui_bibz/ui/core/navigations/components/nav_link_span.rb

@@ -20,9 +20,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class NavLinkSpan < UiBibz::Ui::Core::Component
     # UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render
@@ -44,7 +41,7 @@ module UiBibz::Ui::Core::Navigations
 
     def component_html_data
       super
-      add_html_data 'link', options[:link] if options[:link]
+      add_html_data 'link', value: options[:link] if options[:link]
     end
   end
 end

data/lib/ui_bibz/ui/core/navigations/components/nav_text.rb

@@ -20,9 +20,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class NavText < UiBibz::Ui::Core::Component
     # UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/navigations/components/navbar_brand.rb

@@ -17,9 +17,6 @@ module UiBibz::Ui::Core::Navigations
   # You can pass arguments in options attribute:
   class NavbarBrand < UiBibz::Ui::Core::Component
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/navigations/components/navbar_form.rb

@@ -53,7 +53,7 @@ module UiBibz::Ui::Core::Navigations
       @content      = block
       @model_or_url = model_or_url
       @options      = options
-      @html_options = !%i[form_for ui_form_for].include?(type) ? html_options_for_form(@model_or_url, new_option) : new_option
+      @html_options = %i[form_for ui_form_for].exclude?(type) ? html_options_for_form(@model_or_url, new_option) : new_option
     end
 
     # Render html tag
@@ -64,8 +64,8 @@ module UiBibz::Ui::Core::Navigations
       when :ui_form_for
         begin
           @form = ui_form_for(@model_or_url, html_options, &@content)
-          ## rescue
-          ## raise 'You must install simple form to use ui_form_for.'
+        rescue e
+          raise 'You must install simple form to use ui_form_for.'
         end
       else
         @form = form_tag_with_body(html_options, capture(&@content))
@@ -75,7 +75,7 @@ module UiBibz::Ui::Core::Navigations
     private
 
     def new_option
-      (@options || {}).merge({ class: UiBibz::Utils::Screwdriver.join_classes('navbar-form', 'form-inline', 'my-lg-0', 'my-2', @options[:class]) })
+      (@options || {}).merge({ class: UiBibz::Utils::Screwdriver.join_classes('d-flex', @options[:class]) })
     end
 
     def protect_against_forgery?

data/lib/ui_bibz/ui/core/navigations/components/navbar_nav.rb

@@ -32,16 +32,13 @@ module UiBibz::Ui::Core::Navigations
   #
   # ==== Examples
   #
-  #   UiBibz::Ui::Core::Navigations::NavbarNav.new().tap do |n|
+  #   UiBibz::Ui::Core::Navigations::NavbarNav.new.tap do |n|
   #     n.link 'Test', url: '#test'
   #     n.link 'Test2', url: '#test2', state: :active
   #   end.render
   #
   class NavbarNav < UiBibz::Ui::Core::Navigations::Nav
     # See UiBibz::Ui::Core::Navigations::Nav.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     private
 

data/lib/ui_bibz/ui/core/navigations/components/navbar_text.rb

@@ -34,9 +34,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class NavbarText < UiBibz::Ui::Core::Component
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/navigations/components/pagination_link.rb

@@ -39,9 +39,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class PaginationLink < UiBibz::Ui::Core::Component
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/navigations/components/toolbar_form.rb

@@ -53,7 +53,7 @@ module UiBibz::Ui::Core::Navigations
       @content      = block
       @model_or_url = model_or_url
       @options      = options
-      @html_options = !%i[form_for ui_form_for].include?(type) ? html_options_for_form(@model_or_url, new_option) : new_option
+      @html_options = %i[form_for ui_form_for].exclude?(type) ? html_options_for_form(@model_or_url, new_option) : new_option
     end
 
     # Render html tag
@@ -64,8 +64,8 @@ module UiBibz::Ui::Core::Navigations
       when :ui_form_for
         begin
           @form = ui_form_for(@model_or_url, html_options, &@content)
-          ## rescue
-          ## raise 'You must install simple form to use ui_form_for.'
+        rescue e
+          raise 'You must install simple form to use ui_form_for.'
         end
       else
         @form = form_tag_with_body(html_options, capture(&@content))

data/lib/ui_bibz/ui/core/navigations/link.rb

@@ -41,9 +41,6 @@ module UiBibz::Ui::Core::Navigations
   #
   class Link < UiBibz::Ui::Core::Component
     # See UiBibz::Ui::Core::Component.initialize
-    def initialize(content = nil, options = nil, html_options = nil, &block)
-      super
-    end
 
     # Render html tag
     def pre_render

data/lib/ui_bibz/ui/core/navigations/nav.rb

@@ -52,7 +52,7 @@ module UiBibz::Ui::Core::Navigations
   #
   # ==== Helper
   #
-  #   nav(options = { tap: true }, html_options = {}) do |n|
+  #   nav(options = {}, html_options = {}) do |n|
   #     n.link(content, options = {}, html_options = {})
   #     n.link(options = {}, html_options = {}) do
   #       content
@@ -66,6 +66,8 @@ module UiBibz::Ui::Core::Navigations
   #   end
   #
   class Nav < UiBibz::Ui::Core::Component
+    include UiBibz::Ui::Concerns::HtmlConcern
+
     # See UiBibz::Ui::Core::Component.initialize
     def initialize(content = nil, options = nil, html_options = nil, &block)
       super
@@ -74,18 +76,24 @@ module UiBibz::Ui::Core::Navigations
 
     # Render html tag
     def pre_render
-      content_tag tag, @items.map(&:render).join.html_safe, html_options
+      content_tag htlm_tag, @items.map(&:render).join.html_safe, html_options
     end
 
     # Add nav link items
     # See UiBibz::Ui::Core::Navigations::NavLink
     def link(content = nil, options = {}, html_options = nil, &block)
-      block_given? ? content.merge!({ nav_type: type, nav_tags: nav_tags }) : options.merge!({ nav_type: type, nav_tags: nav_tags })
+      if block_given?
+        content[:nav_type] = type
+        content[:nav_tags] = nav_tags
+      else
+        options[:nav_type] = type
+        options[:nav_tags] = nav_tags
+      end
       @items << NavLink.new(content, options, html_options, &block)
     end
 
     def text(content = nil, options = {}, html_options = nil, &block)
-      block_given? ? content.merge!({ nav_type: type }) : options.merge!({ nav_type: type })
+      block_given? ? content[:nav_type] = type : options[:nav_type] = type
       @items << NavText.new(content, options, html_options, &block)
     end
 
@@ -108,12 +116,20 @@ module UiBibz::Ui::Core::Navigations
     protected
 
     def component_html_classes
-      ['nav', type, position, stacked, justify, fill]
+      [nav_class, type, position, stacked, justify, fill]
+    end
+
+    def component_html_options
+      @options[:type] == :list ? { role: 'tablist' } : super
     end
 
     # tabs or pills
     def type
-      "nav-#{@options[:type]}" unless @options[:type].nil?
+      if @options[:type] == :list
+        'list-group'
+      else
+        "nav-#{@options[:type]}" unless @options[:type].nil?
+      end
     end
 
     def position
@@ -139,11 +155,22 @@ module UiBibz::Ui::Core::Navigations
       'flex-column' if @options[:stacked]
     end
 
-    def tag
-      if options[:tag]
-        options[:tag]
+    def htlm_tag
+      options[:tag] || tag_type
+    end
+
+    def nav_class
+      'nav' if type != 'list-group'
+    end
+
+    def tag_type
+      case type
+      when 'nav-links'
+        :nav
+      when 'list-group'
+        :div
       else
-        type == 'nav-links' ? :nav : :ul
+        :ul
       end
     end