bh 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6522abb89dd53d34215f678b086d1e9414181039
-  data.tar.gz: ce5a63947bd03e86eef43b9ea3801d8692605447
+  metadata.gz: bfa65b892278f33239299fce8ca41f8143bd0b21
+  data.tar.gz: c942fc8e67fad1ade0d91817d92e80d6b53dc9da
 SHA512:
-  metadata.gz: 154e1ec61fee0870b2513f76ec23d7c4be8dc7c8dba2b4a2d8cc1ef3be493eebeb9f405c8f6e9e1557908d8d871df66e6e426b01d6b5f9c06e6128c6f7223e30
-  data.tar.gz: 85e4fe4953a9cc6a6a1b221c76af892aecd330c22e4a499627298f748be4b0d7d938d831e815738e65154424b34a4a96ea977716816efa0e8cc0cd182a36f7ad
+  metadata.gz: 88f80274975d5bd6dbab430908032bbbf8f837d7e96d057769e62115a965594a0a3bfe990318d64cd7d9930d0c042fce831741ef3d5eb18f7fcc6a892fb61a1e
+  data.tar.gz: a7224ebd07b8f8443e61b37eca57afbfdf9a1eb392266d3ed113673b4b136dbe0b4c8590a5c70c3be26517ef60846232e162a7c18474d61d347131d4f15f0e4e

data/CHANGELOG.md

@@ -6,6 +6,11 @@ For more information about changelogs, check
 [Keep a Changelog](http://keepachangelog.com) and
 [Vandamme](http://tech-angels.github.io/vandamme).
 
+## 0.0.8 - 2014-08-25
+
+* [FEATURE] Add `button_to` helper
+* [FEATURE] Add `navbar` helper
+
 ## 0.0.7 - 2014-08-25
 
 * [FEATURE] Add `nav` helper

data/README.md

@@ -46,7 +46,7 @@ How to install
 
 Bh is meant to be included in Rails apps by adding this line to the Gemfile:
 
-    gem 'bh', '~> 0.0.7'
+    gem 'bh', '~> 0.0.8'
 
 Since the gem follows [Semantic Versioning](http://semver.org),
 indicating the full version in your Gemfile (~> *major*.*minor*.*patch*)

data/lib/bh/helpers/base_helper.rb

@@ -10,9 +10,16 @@ module Bh
 
   private
 
-    def append_class!(hash, new_class)
-      existing_class = hash[:class]
-      hash[:class] = [existing_class, new_class].compact.join ' '
+    def append_class!(hash, new_class, attribute = :class)
+      existing_class = hash[attribute]
+      hash[attribute] = [existing_class, new_class].compact.join ' '
+    end
+
+    def append_class_as!(attribute, new_class, *args, &block)
+      html_options = (block_given? ? args[1] : args[2]) || {}
+      append_class! html_options, new_class, attribute
+      block_given? ? args[1] = html_options : args[2] = html_options
+      args
     end
   end
 end

data/lib/bh/helpers/button_to_helper.rb

@@ -0,0 +1,19 @@
+require 'bh/helpers/base_helper'
+
+module Bh
+  module ButtonToHelper
+    include BaseHelper
+
+    # Overrides ActionView +button_to+ to be able to add the 'btn' class
+    # to the link (or 'navbar-btn' if inside a navbar).
+    # @see http://getbootstrap.com/css/#buttons
+    # @see http://getbootstrap.com/components/#navbar-buttons
+    def button_to(*args, &block)
+      args = append_class_as! :class, 'btn', *args, &block
+      if @navbar_id
+        args = append_class_as! :form_class, 'navbar-form', *args, &block
+      end
+      super *args, &block
+    end
+  end
+end

data/lib/bh/helpers/link_to_helper.rb

@@ -0,0 +1,32 @@
+require 'bh/helpers/base_helper'
+
+module Bh
+  module LinkToHelper
+    include BaseHelper
+
+    # Overrides ActionView +link_to+ to be able to add the 'alert-link' class
+    # to the link in case the link is inside of an alert.
+    # @see http://getbootstrap.com/components/#alerts-links
+    # Overrides ActionView +link_to+ to be able to surround the link in a
+    # '<li>' item in case the link is inside of a nav.
+    # Overrides ActionView +link_to+ to be able to add the 'navbar-brand'
+    # class to the link in case the link is inside of an alert.
+    def link_to(*args, &block)
+      args = add_link_class!('alert-link', *args, &block) if @alert_link
+      args = add_link_class!('navbar-brand', *args, &block) if @navbar_vertical
+      link = super *args, &block
+      @nav_link ? content_tag(:li, link, nav_list_item_options(*args, &block)) : link
+    end
+
+  private
+
+    def add_link_class!(new_class, *args, &block)
+      append_class_as! :class, new_class, *args, &block
+    end
+
+    def nav_list_item_options(*args, &block)
+      options = (block_given? ? args[0] : args[1]) || {}
+      {class: 'active'} if current_page? options
+    end
+  end
+end

data/lib/bh/helpers/nav_helper.rb

@@ -30,29 +30,22 @@ module Bh
     # @see http://getbootstrap.com/components/#nav
     def nav(options = {}, &block)
       @nav_link = true
-      nav = content_tag :ul, role: 'tablist', class: nav_class(options), &block
+      nav = content_tag :ul, nav_options(options), &block
       nav.tap{ @nav_link = false }
     end
 
-    # Overrides ActionView +link_to+ to be able to surround the link in a
-    # '<li>' item in case the link is inside of a nav.
-    def link_to(*args, &block)
-      link = super *args, &block
-      @nav_link ? content_tag(:li, link, nav_list_item_options(*args)) : link
-    end
-
   private
 
-    def nav_list_item_options(*args)
-      options = (block_given? ? args[0] : args[1]) || {}
-      {class: 'active'} if current_page? options
-    end
-
-    def nav_class(options = {})
+    def nav_options(options = {})
       append_class! options, 'nav'
-      append_class! options, "nav-#{options.fetch :as, 'tabs'}"
-      append_class! options, "nav-#{options[:layout]}" if options[:layout]
-      options[:class]
+      if @navbar_id
+        append_class! options, 'navbar-nav'
+      else
+        options[:role] = 'tablist'
+        append_class! options, "nav-#{options.fetch :as, 'tabs'}"
+        append_class! options, "nav-#{options[:layout]}" if options[:layout]
+      end
+      options.slice :role, :class
     end
   end
 end

data/lib/bh/helpers/navbar_helper.rb

@@ -0,0 +1,182 @@
+require 'bh/helpers/base_helper'
+
+module Bh
+  # Provides methods to include navbars.
+  # @see http://getbootstrap.com/components/#navbar
+  module NavbarHelper
+    include BaseHelper
+
+    # Returns an HTML block tag that follows the Bootstrap documentation
+    # on how to display *navbars*.
+    #
+    # The skeleton of the nav is a <div> container inside a <nav> tag; its
+    # content is passed as a block.
+    #
+    # Bootstrap allows to differentiate between the content of a navtab that
+    # should remain visible even on a mobile (in a 'vertical' layout) and the
+    # content that should only be visible when there is enough horizontal space
+    # (in a 'horizontal') layout.
+    #
+    # The +vertical+ and +horizontal+ helpers can be used inside a navbar to
+    # specify these types of content.
+    # @example An inverted navbar with the company logo and two links.
+    #   navbar inverted: false do
+    #     vertical image_tag('logo')
+    #     horizontal nav do
+    #       link_to 'Home', '/'
+    #       link_to 'Profile', '/profile'
+    #     end
+    #   end
+    #
+    # @return [String] an HTML block tag for a navbar.
+    # @param [Hash] options the display options for the nav.
+    # @option options [Boolean] :fluid (false) whether to use a fluid container
+    #   to surround the navbar content.
+    # @option options [Boolean] :inverted (false) whether to use an inverted
+    #   palette of colors.
+    # @option options [#to_s] :position (nil) if set, the position of the
+    #   navbar. Valid values are: :top (alias :fixed_top), :bottom
+    #   (alias :fixed_bottom) and :static (alias :static_top).
+    # @option options [#to_s] :padding (70) if position is set to :top or
+    #   :bottom, then this padding will be added at the top (or bottom) of
+    #   <body> to prevent the navbar from overlaying the content.
+    # @yield block the content of the nav
+    # @see http://getbootstrap.com/components/#navbar
+    def navbar(options = {}, &block)
+      @navbar_id = rand 10**7
+      nav_tag = navbar_string options, &block
+      navbar = safe_join [navbar_style_tag(options), nav_tag].compact, "\n"
+      navbar.tap{ @navbar_id = nil }
+    end
+
+    def vertical(content_or_options_with_block = nil, options = nil, &block)
+      if block_given?
+        vertical_string (content_or_options_with_block || {}), &block
+      else
+        vertical_string (options || {}), &Proc.new { content_or_options_with_block }
+      end
+    end
+
+    def horizontal(content_or_options_with_block = nil, options = nil, &block)
+      if block_given?
+        horizontal_string (content_or_options_with_block || {}), &block
+      else
+        horizontal_string (options || {}), &Proc.new { content_or_options_with_block }
+      end
+    end
+
+  private
+
+    def navbar_string(options = {}, &block)
+      content_tag :nav, role: 'navigation', class: navbar_class(options) do
+        content_tag :div, class: navbar_container_class(options), &block
+      end
+    end
+
+    # @private
+    # The fixed navbar will overlay your other content, unless you add padding
+    # to the top or bottom of the <body>. Try out your own values or use our
+    # snippet below. Tip: By default, the navbar is 50px high.
+    # @see http://getbootstrap.com/components/#navbar-fixed-top
+    def navbar_style_tag(options = {})
+      padding_value = options.fetch :padding, 70
+      if padding_value && padding_type = padding_type_for(options[:position])
+        content_tag :style, "body {padding-#{padding_type}: #{padding_value}px}"
+      end
+    end
+
+    def padding_type_for(position)
+      /fixed-(?<type>top|bottom)$/ =~ navbar_position_class_for(position)
+      type
+    end
+
+    # Yields the block as the vertical part of the navbar and sets a variable
+    # so that every +link_to+ helper called inside the block gets the
+    # "navbar-brand" class added, for links to look better.
+    def vertical_string(options = {}, &block)
+      @navbar_vertical = true
+      append_class! options, 'navbar-header'
+      vertical = content_tag :div, options do
+        safe_join [toggle_button, capture(&block)], "\n"
+      end
+      vertical.tap{ @navbar_vertical = false }
+    end
+
+    def horizontal_string(options = {}, &block)
+      append_class! options, 'collapse navbar-collapse'
+      options[:id] = navbar_id
+      content_tag :div, options, &block
+    end
+
+    def toggle_button(options = {})
+      options['type'] = 'button'
+      options['class'] = 'navbar-toggle'
+      options['data-toggle'] = 'collapse'
+      options['data-target'] =  "##{navbar_id}"
+      content_tag :button, options do
+        safe_join [toggle_text, toggle_bar, toggle_bar, toggle_bar], "\n"
+      end
+    end
+
+    def toggle_text
+      content_tag :span, 'Toggle navigation', class: 'sr-only'
+    end
+
+    def toggle_bar
+      content_tag :span, nil, class: 'icon-bar'
+    end
+
+  private
+
+    def navbar_id
+      "navbar-collapse-#{@navbar_id}"
+    end
+
+    def navbar_class(options = {})
+      style = options[:inverted] ? 'inverse' : 'default'
+      position = navbar_position_class_for options[:position]
+      append_class! options, 'navbar'
+      append_class! options, "navbar-#{style}"
+      append_class! options, "navbar-#{position}" if position
+      options[:class]
+    end
+
+    def navbar_position_class_for(position)
+      case position.to_s
+        when 'static', 'static_top' then 'static-top'
+        when 'top', 'fixed_top' then 'fixed-top'
+        when 'bottom', 'fixed_bottom' then 'fixed-bottom'
+      end
+    end
+
+    def navbar_container_class(options = {})
+      options[:fluid] ? 'container-fluid' : 'container'
+    end
+  end
+end
+
+
+# STILL TO DO
+# 2) Add class to form and button and nav and p inside navbar
+
+
+#
+# <%= navbar do %>
+#   <%= vertical do %>
+#     <%= link_to image_tag('/assets/fullscreen-standard-logo-white.png', alt: 'Fullscreen'), root_path %>
+#   <% end %>
+#   <%= horizontal do %>
+#     <%= nav do %>
+#       <%= link_to 'Campaigns', root_path %>
+#       <%= link_to 'Users', users_path if current_user.admin? %>
+#     <% end %>
+#     <%= nav class: 'navbar-right' do %>
+#       <!-- button_to needs to:
+#            * be surrounded in <li> AND use navbar-form inside a navbar
+#            * add navbar-btn is inside a navbar, else add btn
+#       -->
+#       <%= button_to 'Sign out', destroy_user_session_path, class: 'btn-link', method: :delete %>
+#     <% end %>
+#   <% end %>
+# <% end if user_signed_in? %>
+#

data/lib/bh/railtie.rb

@@ -1,25 +1,29 @@
 require 'bh/helpers/alert_helper'
+require 'bh/helpers/button_to_helper'
 require 'bh/helpers/cdn_helper'
 require 'bh/helpers/form_for_helper'
 require 'bh/helpers/glyphicon_helper'
+require 'bh/helpers/link_to_helper'
 require 'bh/helpers/modal_helper'
 require 'bh/helpers/nav_helper'
+require 'bh/helpers/navbar_helper'
 require 'bh/helpers/panel_helper'
 require 'bh/helpers/panel_row_helper'
-require 'bh/helpers/url_helper'
 
 module Bh
   class Railtie < Rails::Railtie
     initializer 'bh.add_helpers' do
       ActionView::Base.send :include, AlertHelper
+      ActionView::Base.send :include, ButtonToHelper
       ActionView::Base.send :include, CdnHelper
       ActionView::Base.send :include, FormForHelper
       ActionView::Base.send :include, GlyphiconHelper
+      ActionView::Base.send :include, LinkToHelper
       ActionView::Base.send :include, ModalHelper
       ActionView::Base.send :include, NavHelper
+      ActionView::Base.send :include, NavbarHelper
       ActionView::Base.send :include, PanelHelper
       ActionView::Base.send :include, PanelRowHelper
-      ActionView::Base.send :include, UrlHelper
     end
 
     initializer 'bh.add_views' do |app|

data/lib/bh/version.rb

@@ -1,3 +1,3 @@
 module Bh
-  VERSION = '0.0.7'
+  VERSION = '0.0.8'
 end

data/spec/helpers/alert_helper_spec.rb

@@ -1,10 +1,11 @@
 # encoding: UTF-8
 
 require 'spec_helper'
+
 require 'bh/helpers/alert_helper'
-require 'bh/helpers/url_helper'
+require 'bh/helpers/link_to_helper'
 include Bh::AlertHelper
-include Bh::UrlHelper
+include Bh::LinkToHelper
 
 describe 'alert_box' do
   describe 'accepts as parameters:' do

data/spec/helpers/button_to_helper_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+require 'bh/helpers/button_to_helper'
+require 'bh/helpers/navbar_helper'
+
+include Bh::ButtonToHelper
+include Bh::NavbarHelper
+
+describe 'button_to' do
+  let(:protect_against_forgery?) { false }
+  attr_accessor :output_buffer
+  let(:url) { '/' }
+
+  context 'used without a block' do
+    let(:button) { button_to 'Home', url }
+
+    specify 'by default, applies the "btn" class' do
+      expect(button).to include 'input class="btn" type="submit" value="Home"'
+    end
+
+    specify 'inside a navbar, also applies the "navbar-form" class to the form' do
+      expect(navbar { button }).to match %r{<form .+?class="navbar-form" .+?input class="btn"}
+    end
+  end
+end

data/spec/helpers/link_to_helper_spec.rb

@@ -0,0 +1,100 @@
+require 'spec_helper'
+require 'action_dispatch'
+
+require 'bh/helpers/alert_helper'
+require 'bh/helpers/link_to_helper'
+require 'bh/helpers/nav_helper'
+require 'bh/helpers/navbar_helper'
+
+include Bh::AlertHelper
+include Bh::LinkToHelper
+include Bh::NavHelper
+include Bh::NavbarHelper
+
+describe 'link_to' do
+  let(:request) { ActionDispatch::Request.new request_options }
+  let(:request_options) { {'REQUEST_METHOD' => 'GET'} }
+  let(:url) { '/' }
+
+  context 'used without a block' do
+    let(:link) { link_to 'Home', url }
+
+    specify 'by default, does not apply the "alert-link" class' do
+      expect(link).not_to include 'alert-link'
+    end
+
+    specify 'inside an alert, applies the "alert-link"' do
+      expect(alert_box { link }).to include 'alert-link'
+    end
+
+    specify 'by default, does not apply the "navbar-brand" class' do
+      expect(link).not_to include 'navbar-brand'
+    end
+
+    specify 'inside the vertical part of a navbar, applies the "navbar-brand"' do
+      expect(navbar { vertical { link } }).to include 'navbar-brand'
+    end
+
+    specify 'by default, does not surround the link in a list item' do
+      expect(link).not_to include '<li>'
+    end
+
+    context 'inside a nav' do
+      let(:html) { nav { link } }
+
+      specify 'surrounds the link in a list item' do
+        expect(html).to include '<li><a href="/">Home</a></li>'
+      end
+
+      specify 'by default, does not apply the "active" class' do
+        expect(link).not_to include 'active'
+      end
+
+      context 'given the link points to the current page, applies the "active" class' do
+        let(:url) { '' }
+        it { expect(html).to include 'li class="active"' }
+      end
+    end
+  end
+
+  context 'used with a block' do
+    let(:link) { link_to(url) { 'Home' } }
+
+    specify 'by default, does not apply the "alert-link" class' do
+      expect(link).not_to include 'alert-link'
+    end
+
+    specify 'inside an alert, applies the "alert-link"' do
+      expect(alert_box { link }).to include 'alert-link'
+    end
+
+    specify 'by default, does not apply the "navbar-brand" class' do
+      expect(link).not_to include 'navbar-brand'
+    end
+
+    specify 'inside the vertical part of a navbar, applies the "navbar-brand"' do
+      expect(navbar { vertical { link } }).to include 'navbar-brand'
+    end
+
+    specify 'by default, does not surround the link in a list item' do
+      expect(link).not_to include '<li>'
+    end
+
+    context 'inside a nav' do
+      let(:html) { nav { link } }
+
+      specify 'surrounds the link in a list item' do
+        expect(html).to include '<li><a href="/">Home</a></li>'
+      end
+
+      specify 'by default, does not apply the "active" class' do
+        expect(link).not_to include 'active'
+      end
+
+      context 'given the link points to the current page, applies the "active" class' do
+        let(:url) { '' }
+        it { expect(html).to include 'li class="active"' }
+      end
+    end
+  end
+end

data/spec/helpers/nav_helper_spec.rb

@@ -1,10 +1,11 @@
 # encoding: UTF-8
 
 require 'spec_helper'
-require 'action_dispatch'
 require 'bh/helpers/nav_helper'
+require 'bh/helpers/navbar_helper'
 
 include Bh::NavHelper
+include Bh::NavbarHelper
 
 describe 'nav' do
   let(:html) { nav options, &block }
@@ -43,19 +44,10 @@ describe 'nav' do
     end
   end
 
-  describe 'given a +link_to+ in the content' do
-    let(:block) { Proc.new { link_to 'Home', url} }
-    let(:url) { '/any-page' }
-    let(:request) { ActionDispatch::Request.new request_options }
-    let(:request_options) { {'REQUEST_METHOD' => 'GET'} }
-
-    specify 'surrounds the link in a list item' do
-      expect(html).to include '<li><a href="/any-page">Home</a></li>'
-    end
-
-    context 'sets the "active" class if the link points to the same page' do
-      let(:url) { '' }
-      it { expect(html).to include 'li class="active"' }
+  describe 'within a navbar' do
+    let(:html) { navbar { nav options, &block } }
+    specify 'applies roles and classes specific to navbar' do
+      expect(html).to include 'ul class="nav navbar-nav">'
     end
   end
 end

data/spec/helpers/navbar_helper_spec.rb

@@ -0,0 +1,194 @@
+# encoding: UTF-8
+
+require 'spec_helper'
+require 'bh/helpers/navbar_helper'
+require 'bh/helpers/nav_helper'
+
+include Bh::NavbarHelper
+include Bh::NavHelper
+
+describe 'navbar' do
+  let(:html) { navbar options, &block }
+  let(:block) { Proc.new {} }
+  let(:options) { {} }
+
+  specify 'applies Boostrap attributes to the navbar' do
+    expect(html).to match %r{<nav.+? role="navigation">}
+  end
+
+  describe 'with the :fluid option' do
+    specify 'not set, adds a "container" div to the navbar' do
+      expect(html).to include '<div class="container">'
+    end
+
+    context 'set to false, adds a "container" div to the navbar' do
+      let(:options) { {fluid: false} }
+      it { expect(html).to include '<div class="container">' }
+    end
+
+    context 'set to true, adds a "container-fluid" div to the navbar' do
+      let(:options) { {fluid: true} }
+      it { expect(html).to include '<div class="container-fluid">' }
+    end
+  end
+
+  describe 'with the :inverted option' do
+    specify 'not set, shows a "default" navbar' do
+      expect(html).to include 'nav class="navbar navbar-default"'
+    end
+
+    context 'set to false, shows a "default" nav' do
+      let(:options) { {inverted: false} }
+      it { expect(html).to include 'nav class="navbar navbar-default"' }
+    end
+
+    context 'set to true, shows an "inverse" nav' do
+      let(:options) { {inverted: true} }
+      it { expect(html).to include 'nav class="navbar navbar-inverse"' }
+    end
+  end
+
+  describe 'with the :position option' do
+    specify 'not set, does not set a navbar position' do
+      expect(html).to include 'nav class="navbar navbar-default"'
+    end
+
+    context 'set to :static, shows the navbar at the top (static) without padding' do
+      let(:options) { {position: :static} }
+      it { expect(html).to include 'nav class="navbar navbar-default navbar-static-top"' }
+      it { expect(html).not_to include 'padding' }
+    end
+
+    context 'set to :static_top, shows the navbar at the top (static) without padding' do
+      let(:options) { {position: :static_top} }
+      it { expect(html).to include 'nav class="navbar navbar-default navbar-static-top"' }
+      it { expect(html).not_to include 'padding' }
+    end
+
+    context 'set to :top, shows the navbar at the top (fixed)' do
+      let(:options) { {position: :top} }
+      it { expect(html).to include 'nav class="navbar navbar-default navbar-fixed-top"' }
+
+      context 'with the :padding' do
+        specify 'not set, adds 70px to the top padding of the body' do
+          expect(html).to include '<style>body {padding-top: 70px}</style>'
+        end
+
+        context 'set to a value, adds those pixels to the top padding of the body' do
+          let(:options) { {position: :top, padding: 100} }
+
+          it { expect(html).to include '<style>body {padding-top: 100px}</style>' }
+        end
+
+        context 'set to nil, does not add a top padding to the body' do
+          let(:options) { {position: :top, padding: nil} }
+
+          it { expect(html).not_to include 'padding' }
+        end
+      end
+    end
+
+    context 'set to :fixed_top, shows the navbar at the top (fixed)' do
+      let(:options) { {position: :fixed_top} }
+      it { expect(html).to include 'nav class="navbar navbar-default navbar-fixed-top"' }
+    end
+
+    context 'set to :bottom, shows the navbar at the bottom (fixed)' do
+      let(:options) { {position: :bottom} }
+      it { expect(html).to include 'nav class="navbar navbar-default navbar-fixed-bottom"' }
+
+      context 'with the :padding' do
+        specify 'not set, adds 70px to the bottom padding of the body' do
+          expect(html).to include '<style>body {padding-bottom: 70px}</style>'
+        end
+
+        context 'set to a value, adds those pixels to the bottom padding of the body' do
+          let(:options) { {position: :bottom, padding: 100} }
+
+          it { expect(html).to include '<style>body {padding-bottom: 100px}</style>' }
+        end
+
+        context 'set to nil, does not add a bottom padding to the body' do
+          let(:options) { {position: :bottom, padding: nil} }
+
+          it { expect(html).not_to include 'padding' }
+        end
+      end
+    end
+
+    context 'set to :fixed_bottom, shows the navbar at the bottom (fixed)' do
+      let(:options) { {position: :fixed_bottom} }
+      it { expect(html).to include 'nav class="navbar navbar-default navbar-fixed-bottom"' }
+    end
+  end
+end
+
+describe 'vertical' do
+  describe 'accepts as parameters:' do
+    let(:behave) { be_a String }
+
+    specify 'a string (content)' do
+      expect(vertical 'content').to behave
+    end
+
+    specify 'a block (content)' do
+      expect(vertical { 'content' }).to behave
+    end
+
+    specify 'a string (content) + a hash (options)' do
+      expect(vertical 'content', class: :important).to behave
+    end
+
+    specify 'a hash (options) + a block (content)' do
+      expect(vertical(class: :important) { 'content' }).to behave
+    end
+  end
+
+  describe 'adds a toggle button and bars' do
+    let(:html) { vertical 'content' }
+    it { expect(html).to match %r{<button class="navbar-toggle" data-target="#.+?" data-toggle="collapse" type="button"><span class="sr-only">Toggle navigation</span>\n<span class="icon-bar"></span>\n<span class="icon-bar"></span>\n<span class="icon-bar"></span></button}m }
+  end
+end
+
+describe 'horizontal' do
+  describe 'accepts as parameters:' do
+    let(:behave) { be_a String }
+
+    specify 'a string (content)' do
+      expect(horizontal 'content').to behave
+    end
+
+    specify 'a block (content)' do
+      expect(horizontal { 'content' }).to behave
+    end
+
+    specify 'a string (content) + a hash (options)' do
+      expect(horizontal 'content', class: :important).to behave
+    end
+
+    specify 'a hash (options) + a block (content)' do
+      expect(horizontal(class: :important) { 'content' }).to behave
+    end
+  end
+
+  describe 'adds a collapsable div' do
+    let(:html) { horizontal 'content' }
+    it { expect(html).to match %r{<div class="collapse navbar-collapse" id=".+?">content</div>} }
+  end
+end
+
+describe 'multiple navbars' do
+  let(:navbar_1) { navbar inverted: true do
+    safe_join [vertical(link_to 'Home', '/'), horizontal(content_tag :p, '')]
+  end }
+  let(:navbar_2) { navbar position: :top do
+    safe_join [vertical(content_tag :p, ''), horizontal(link_to 'Home', '/')]
+  end }
+  let(:html) { safe_join [navbar_1, navbar_2], "\n"}
+
+  specify 'toggle their own horizontal part when clicking on vertical' do
+    navbar_ids = html.scan %r{<nav.+?data-target="#(.+?)".+?id="(\1)".+?</nav>}m
+    expect(navbar_ids.size).to eq 2
+    expect(navbar_ids.uniq.size).to eq 2
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bh
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors:
 - Claudio Baccigalupo
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-26 00:00:00.000000000 Z
+date: 2014-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -145,6 +145,7 @@ files:
 - lib/bh/form_builders/form_builder.rb
 - lib/bh/helpers/alert_helper.rb
 - lib/bh/helpers/base_helper.rb
+- lib/bh/helpers/button_to_helper.rb
 - lib/bh/helpers/cdn_helper.rb
 - lib/bh/helpers/form/base_helper.rb
 - lib/bh/helpers/form/check_box_helper.rb
@@ -158,15 +159,17 @@ files:
 - lib/bh/helpers/form/submit_helper.rb
 - lib/bh/helpers/form_for_helper.rb
 - lib/bh/helpers/glyphicon_helper.rb
+- lib/bh/helpers/link_to_helper.rb
 - lib/bh/helpers/modal_helper.rb
 - lib/bh/helpers/nav_helper.rb
+- lib/bh/helpers/navbar_helper.rb
 - lib/bh/helpers/panel_helper.rb
 - lib/bh/helpers/panel_row_helper.rb
-- lib/bh/helpers/url_helper.rb
 - lib/bh/railtie.rb
 - lib/bh/version.rb
 - lib/bh/views/bh/_modal.html.erb
 - spec/helpers/alert_helper_spec.rb
+- spec/helpers/button_to_helper_spec.rb
 - spec/helpers/cdn_helper_spec.rb
 - spec/helpers/form/check_box_helper_spec.rb
 - spec/helpers/form/field_helper_spec.rb
@@ -179,11 +182,12 @@ files:
 - spec/helpers/form/submit_helper_spec.rb
 - spec/helpers/form_for_helper_spec.rb
 - spec/helpers/glyphicon_helper_spec.rb
+- spec/helpers/link_to_helper_spec.rb
 - spec/helpers/modal_helper_spec.rb
 - spec/helpers/nav_helper_spec.rb
+- spec/helpers/navbar_helper_spec.rb
 - spec/helpers/panel_helper_spec.rb
 - spec/helpers/panel_row_helper_spec.rb
-- spec/helpers/url_helper_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/Fullscreen/bh
 licenses:
@@ -212,6 +216,7 @@ summary: Bh provides a set of powerful helpers that streamlines the use of Boots
   components in Rails views.
 test_files:
 - spec/helpers/alert_helper_spec.rb
+- spec/helpers/button_to_helper_spec.rb
 - spec/helpers/cdn_helper_spec.rb
 - spec/helpers/form/check_box_helper_spec.rb
 - spec/helpers/form/field_helper_spec.rb
@@ -224,10 +229,11 @@ test_files:
 - spec/helpers/form/submit_helper_spec.rb
 - spec/helpers/form_for_helper_spec.rb
 - spec/helpers/glyphicon_helper_spec.rb
+- spec/helpers/link_to_helper_spec.rb
 - spec/helpers/modal_helper_spec.rb
 - spec/helpers/nav_helper_spec.rb
+- spec/helpers/navbar_helper_spec.rb
 - spec/helpers/panel_helper_spec.rb
 - spec/helpers/panel_row_helper_spec.rb
-- spec/helpers/url_helper_spec.rb
 - spec/spec_helper.rb
 has_rdoc: 

data/lib/bh/helpers/url_helper.rb

@@ -1,23 +0,0 @@
-require 'action_view'
-
-module Bh
-  module UrlHelper
-    # Overrides ActionView +link_to+ to be able to add the 'alert_link' class
-    # to the link in case the link is inside of an alert.
-    # @see http://getbootstrap.com/components/#alerts-links
-    def link_to(*args, &block)
-      args = add_alert_class_to_link!(*args, &block) if @alert_link
-      super *args, &block
-    end
-
-  private
-
-    def add_alert_class_to_link!(*args, &block)
-      html_options = (block_given? ? args[1] : args[2]) || {}
-      klass = html_options[:class]
-      html_options[:class] = [klass, 'alert-link'].compact.join ' '
-      block_given? ? args[1] = html_options : args[2] = html_options
-      args
-    end
-  end
-end

data/spec/helpers/url_helper_spec.rb

@@ -1,31 +0,0 @@
-require 'spec_helper'
-require 'bh/helpers/alert_helper'
-require 'bh/helpers/url_helper'
-include Bh::AlertHelper
-include Bh::UrlHelper
-
-describe 'link_to' do
-  context 'used without a block' do
-    let(:link) { link_to 'Home', '/' }
-
-    specify 'does not apply the "alert_link" class if used outside of an alert' do
-      expect(link).not_to include 'alert-link'
-    end
-
-    specify 'applies the "alert_link" class if used inside an alert' do
-      expect(alert_box { link }).to include 'alert-link'
-    end
-  end
-
-  context 'used with a block' do
-    let(:link) { link_to('/') { 'Home' } }
-
-    specify 'does not apply the "alert_link" class if used outside of an alert' do
-      expect(link).not_to include 'alert-link'
-    end
-
-    specify 'applies the "alert_link" class if used inside an alert' do
-      expect(alert_box { link }).to include 'alert-link'
-    end
-  end
-end