bootstrap-navbar 3.0.2 → 3.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0b885250141aa7f2630946b353be7f141c08f138004d105b032a23c2985522cf
-  data.tar.gz: 40c79d7837d313b07b46a801acbfa0ae31a8885ea7918cf1f58b562cdf943f37
+  metadata.gz: 31c1c11b16764f02d05e907c90ec1485dc0ca435032b1492d873f3d79c3818c6
+  data.tar.gz: 1e8dfe8093cc0455703c9ce05eb80590ade4bcb93f8b01fc53d8fd7f3b41595d
 SHA512:
-  metadata.gz: f01f3f971263da163248e661cae6b6b1f4a59f7c5b86127d249970be28da861ec2bc71534f938f5b7f04941f7e7087f48a1e9e1e55d10e431333d3e1f0cf3f2c
-  data.tar.gz: 17a140492b19728d9b0d7aaba09c9175462013c5e71cc425701bb859f27b992a1b26f3d61979b93cbdbff6cef4ad409c69f3158560e1005c4d060b6df7284d80
+  metadata.gz: 1f5b2f052f44d4c98533712ed45e6fe323c1d15a6f041d7e1a67cb8590122bc12cbbc9b4564995da40280c194b58500855609257be197f08928013547b011064
+  data.tar.gz: 65ceeca8fe2f44d3ca81fecc926876936bf7b1907adbd35061befddea130d7d4593e8a8726b0d3e79701c21c64d8320792c419465e83a5b31013a97c9fca5e9a

data/README.md

@@ -2,7 +2,6 @@
 
 [![Gem Version](https://badge.fury.io/rb/bootstrap-navbar.png)](http://badge.fury.io/rb/bootstrap-navbar)
 [![Build Status](https://secure.travis-ci.org/bootstrap-ruby/bootstrap-navbar.png)](http://travis-ci.org/bootstrap-ruby/bootstrap-navbar)
-[![Dependency Status](https://gemnasium.com/bootstrap-ruby/bootstrap-navbar.png)](https://gemnasium.com/bootstrap-ruby/bootstrap-navbar)
 [![Code Climate](https://codeclimate.com/github/bootstrap-ruby/bootstrap-navbar.png)](https://codeclimate.com/github/bootstrap-ruby/bootstrap-navbar)
 
 Helpers to generate a Bootstrap style navbar
@@ -103,3 +102,7 @@ See [CHANGELOG](CHANGELOG.md)
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
+
+## Support
+
+If you like this project, consider [buying me a coffee](https://www.buymeacoffee.com/279lcDtbF)! :)

data/lib/bootstrap-navbar/helpers.rb

@@ -15,14 +15,14 @@ module BootstrapNavbar::Helpers
   def attributes_for_tag(hash)
     string = hash.map { |k, v| %(#{k}="#{v}") }.join(' ')
     if string.length > 0
-      ' ' << string
+      ' '.dup << string
     else
       string
     end
   end
 
   def current_url_or_sub_url?(url)
-    return false if url == '#'
+    return false if url == '#' || url =~ /\Atel:/
     uri, current_uri = [url, current_url].map do |url|
       URI.parse(url)
     end

data/lib/bootstrap-navbar/helpers/bootstrap4.rb

@@ -2,11 +2,26 @@ module BootstrapNavbar::Helpers::Bootstrap4
   def navbar(options = {}, &block)
     options = options.dup
     container = options.key?(:container) ? options.delete(:container) : false
+    brand = if options[:brand]
+      prepare_html <<~HTML
+                     <a href="#{options.delete(:brand_url) || '/'}" class="navbar-brand">
+                       #{options.delete(:brand)}
+                     </a>
+                   HTML
+    end
     wrapper options do
       if container
-        container(&block)
+        container container do
+          prepare_html <<~HTML
+                         #{brand}
+                         #{capture(&block) if block_given?}
+                       HTML
+        end
       else
-        capture(&block) if block_given?
+        prepare_html <<~HTML
+                       #{brand}
+                       #{capture(&block) if block_given?}
+                     HTML
       end
     end
   end
@@ -26,25 +41,25 @@ module BootstrapNavbar::Helpers::Bootstrap4
       'aria-expanded' => false,
       'aria-label'    => 'Toggle navigation'
     )
-    prepare_html <<-HTML.chomp!
-<button#{toggler_attributes}>
-  <span class="navbar-toggler-icon"></span>
-</button>
-<div#{attributes}>
-  #{capture(&block) if block_given?}
-</div>
-HTML
+    prepare_html <<~HTML
+                   <button#{toggler_attributes}>
+                     <span class="navbar-toggler-icon"></span>
+                   </button>
+                   <div#{attributes}>
+                     #{capture(&block) if block_given?}
+                   </div>
+                 HTML
   end
 
   def navbar_group(options = {}, &block)
     options = options.dup
     options[:class] = [options[:class], 'navbar-nav'].compact.join(' ')
     attributes = attributes_for_tag(options)
-    prepare_html <<-HTML.chomp!
-<ul#{attributes}>
-  #{capture(&block) if block_given?}
-</ul>
-HTML
+    prepare_html <<~HTML
+                   <ul#{attributes}>
+                     #{capture(&block) if block_given?}
+                   </ul>
+                 HTML
   end
 
   def navbar_item(text, url = nil, list_item_options = nil, link_options = nil, &block)
@@ -58,30 +73,31 @@ HTML
     link_options[:class]  = link_options[:class].join(' ')
     list_item_attributes = attributes_for_tag(list_item_options)
     link_attributes      = attributes_for_tag(link_options)
-    prepare_html <<-HTML.chomp!
-<li#{list_item_attributes}>
-  <a href="#{url}"#{link_attributes}>
-    #{text}
-  </a>
-</li>
-HTML
+    prepare_html <<~HTML
+                   <li#{list_item_attributes}>
+                     <a href="#{url}"#{link_attributes}>
+                       #{text}
+                     </a>
+                   </li>
+                 HTML
   end
 
   def navbar_dropdown(text, id = '', list_item_options = {}, link_options = {}, &block)
     list_item_options, link_options = list_item_options.dup, link_options.dup
     list_item_options[:class] = [list_item_options[:class], 'nav-item', 'dropdown'].compact.join(' ')
+    wrapper_class = [*list_item_options.delete(:wrapper_class), 'dropdown-menu'].compact.join(' ')
     list_item_attributes = attributes_for_tag(list_item_options)
     link_options[:class] = [link_options[:class], 'nav-link', 'dropdown-toggle'].compact.join(' ')
     id ||= "navbarDropdownMenuLink#{text}"
     link_attributes = attributes_for_tag(link_options)
-    prepare_html <<-HTML.chomp!
-<li#{list_item_attributes}>
-  <a href="#" data-toggle="dropdown" id="##{id}" aria-haspopup="true" aria-expanded="false" role="button" #{link_attributes}>#{text}</a>
-  <div class="dropdown-menu" aria-labelledby="#{id}">
-    #{capture(&block) if block_given?}
-  </div>
-</li>
-HTML
+    prepare_html <<~HTML
+                   <li#{list_item_attributes}>
+                     <a href="#" data-toggle="dropdown" id="##{id}" aria-haspopup="true" aria-expanded="false" role="button" #{link_attributes}>#{text}</a>
+                     <div class="#{wrapper_class}" aria-labelledby="#{id}">
+                       #{capture(&block) if block_given?}
+                     </div>
+                   </li>
+                 HTML
   end
 
   def navbar_dropdown_item(text, url = '#', link_options = {}, &block)
@@ -90,36 +106,39 @@ HTML
     link_options[:class] << 'active' if current_url_or_sub_url?(url)
     link_options[:class]  = link_options[:class].join(' ')
     link_attributes = attributes_for_tag(link_options)
-    prepare_html <<-HTML.chomp!
-<a href="#{url}"#{link_attributes}>
-  #{text}
-</a>
-    HTML
+    prepare_html <<~HTML
+                   <a href="#{url}"#{link_attributes}>
+                     #{text}
+                   </a>
+                 HTML
   end
 
-  private
-
-  def container(&block)
-    prepare_html <<-HTML.chomp!
-<div class="container">
-  #{capture(&block) if block_given?}
-</div>
-HTML
+  def navbar_dropdown_divider
+    '<div class="dropdown-divider"></div>'
   end
 
-  def brand_link(text, url = nil)
-    prepare_html <<-HTML.chomp!
-<a href="#{url || '/'}" class="navbar-brand">
-  #{text}
-</a>
-HTML
+  private
+
+  def container(container, &block)
+    container_class = [
+      'container',
+      (container unless container == true)
+    ].compact.join('-')
+    attributes = attributes_for_tag(class: container_class)
+    prepare_html <<~HTML
+                    <div#{attributes}>
+                      #{capture(&block) if block_given?}
+                    </div>
+                  HTML
   end
 
   def wrapper(options, &block)
     options = options.dup
     options[:class] = [options[:class], 'navbar'].compact
     options[:class] << "navbar-#{options.key?(:color_scheme) ? options.delete(:color_scheme) : 'dark'}"
-    options[:class] << "bg-#{options.delete(:bg) || 'dark'}" unless options[:bg] == false
+    if bg = options.delete(:bg)
+      options[:class] << "bg-#{bg == true ? 'dark' : bg}"
+    end
     if options.key?(:sticky) && options.delete(:sticky) === true
       options[:class] << 'sticky-top'
     elsif options.key?(:placement)
@@ -127,13 +146,11 @@ HTML
     end
     options[:class] << "navbar-expand-#{options.delete(:expand_at) || 'sm'}"
     options[:class] = options[:class].join(' ')
-    brand = brand_link(options.delete(:brand), options.delete(:brand_url)) if options[:brand]
     attributes = attributes_for_tag(options)
-    prepare_html <<-HTML.chomp!
-<nav#{attributes}>
-  #{brand}
-  #{capture(&block) if block_given?}
-</nav>
-HTML
+    prepare_html <<~HTML
+                    <nav#{attributes}>
+                      #{capture(&block) if block_given?}
+                    </nav>
+                  HTML
   end
 end

data/lib/bootstrap-navbar/version.rb

@@ -1,3 +1,3 @@
 module BootstrapNavbar
-  VERSION = '3.0.2'
+  VERSION = '3.0.7'
 end

data/spec/bootstrap-navbar/helpers/bootstrap4_spec.rb

@@ -38,7 +38,7 @@ describe BootstrapNavbar::Helpers::Bootstrap4 do
 
     context 'with "placement" parameter' do
       it 'generates the correct HTML' do
-        %w(full fixed-top fixed-bottom).each do |placement|
+        %w(top bottom).each do |placement|
           expect(renderer.navbar(placement: placement)).to have_tag(:nav, with: { class: "navbar navbar-dark bg-dark fixed-#{placement}" })
         end
       end
@@ -141,4 +141,22 @@ describe BootstrapNavbar::Helpers::Bootstrap4 do
       end
     end
   end
+
+  describe '#navbar_dropdown' do
+    context 'with block' do
+      it 'generates the correct HTML' do
+        expect(renderer.navbar_dropdown('Foo', 'foo') { 'link-text' }).to have_tag(:li, with: { class: 'nav-item dropdown'}) do
+          with_tag :div, with: { class: 'dropdown-menu' }
+        end
+      end
+    end
+
+    context 'with right aligned menu' do
+      it 'generates the correct HTML' do
+        expect(renderer.navbar_dropdown('Foo', 'foo', { wrapper_class: 'dropdown-menu-right' }) { 'link-text' }).to have_tag(:li, with: { class: 'nav-item dropdown' }) do
+          with_tag :div, with: { class: 'dropdown-menu dropdown-menu-right' }
+        end
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bootstrap-navbar
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 3.0.7
 platform: ruby
 authors:
 - Manuel Meurer
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-02-14 00:00:00.000000000 Z
+date: 2020-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -153,7 +153,7 @@ homepage: http://bootstrap-ruby.github.io/bootstrap-navbar
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -168,9 +168,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.5
-signing_key: 
+rubygems_version: 3.1.3
+signing_key:
 specification_version: 4
 summary: Helpers to generate a Bootstrap style navbar
 test_files: