diff --git a/app/helpers/navbar_helper.rb b/app/helpers/navbar_helper.rb index a6a01c02..691fd195 100644 --- a/app/helpers/navbar_helper.rb +++ b/app/helpers/navbar_helper.rb @@ -16,7 +16,7 @@ def menu_group(options={}, &block) def menu_item(name=nil, path="#", *args, &block) path = name || path if block_given? options = args.extract_options! - content_tag :li, :class => is_active?(path, options) do + content_tag :li, :role => "presentation", :class => is_active?(path, options) do if block_given? link_to path, options, &block else diff --git a/spec/lib/twitter_bootstrap_rails/navbar_helper_spec.rb b/spec/lib/twitter_bootstrap_rails/navbar_helper_spec.rb index 9f11cbf8..cdb8b233 100644 --- a/spec/lib/twitter_bootstrap_rails/navbar_helper_spec.rb +++ b/spec/lib/twitter_bootstrap_rails/navbar_helper_spec.rb @@ -171,6 +171,13 @@ with_tag(:a, text: " Home", with: { href: "/"}) } end + + it "should return a link with role 'presentation' attribute" do + allow(self).to receive(:current_page?) { false } + + element = menu_item("Home", "/") + expect(element).to have_tag(:li, with: {role: "presentation"}) + end end describe "drop_down" do