Permalink
Browse files

Merge pull request #5787 from frankieroberto/itemscope-patch

Adding itemscope to list of boolean attributes.
  • Loading branch information...
2 parents 68a9788 + 283133a commit e065312a146a642045d66d213b727128135e8144 @spastorino spastorino committed Apr 9, 2012
Showing with 3 additions and 3 deletions.
  1. +1 −1 actionpack/lib/action_view/helpers/tag_helper.rb
  2. +2 −2 actionpack/test/template/tag_helper_test.rb
@@ -14,7 +14,7 @@ module TagHelper
BOOLEAN_ATTRIBUTES = %w(disabled readonly multiple checked autobuffer
autoplay controls loop selected hidden scoped async
defer reversed ismap seemless muted required
- autofocus novalidate formnovalidate open pubdate).to_set
+ autofocus novalidate formnovalidate open pubdate itemscope).to_set
BOOLEAN_ATTRIBUTES.merge(BOOLEAN_ATTRIBUTES.map {|attribute| attribute.to_sym })
PRE_CONTENT_STRINGS = {
@@ -30,8 +30,8 @@ def test_tag_options_accepts_blank_option
end
def test_tag_options_converts_boolean_option
- assert_equal '<p disabled="disabled" multiple="multiple" readonly="readonly" />',
- tag("p", :disabled => true, :multiple => true, :readonly => true)
+ assert_equal '<p disabled="disabled" itemscope="itemscope" multiple="multiple" readonly="readonly" />',
+ tag("p", :disabled => true, :itemscope => true, :multiple => true, :readonly => true)
end
def test_content_tag

0 comments on commit e065312

Please sign in to comment.