diff options
author | Semyon Perepelitsa <sema@sema.in> | 2013-01-24 04:26:49 +0800 |
---|---|---|
committer | Semyon Perepelitsa <sema@sema.in> | 2013-01-31 01:28:57 +0800 |
commit | 540ebe37cd1a9551b739c552a0d4efd2adc7ff22 (patch) | |
tree | 083ed8c399ec6791f907698fa0aa5c3398505834 /actionpack/CHANGELOG.md | |
parent | aa202965603fb8511bdd2448677d19dfb7afa87f (diff) | |
download | rails-540ebe37cd1a9551b739c552a0d4efd2adc7ff22.tar.gz rails-540ebe37cd1a9551b739c552a0d4efd2adc7ff22.tar.bz2 rails-540ebe37cd1a9551b739c552a0d4efd2adc7ff22.zip |
Fix `content_tag_for` with array html option.
It would embed array as string instead of joining it like `content_tag` does:
content_tag(:td, class: ["foo", "bar"]){}
#=> '<td class="foo bar"></td>'
Before:
content_tag_for(:td, item, class: ["foo", "bar"]){}
#=> '<td class="item ["foo", "bar"]" id="item_1"></td>'
After:
content_tag_for(:td, item, class: ["foo", "bar"]){}
#=> '<td class="item foo bar" id="item_1"></td>'
Diffstat (limited to 'actionpack/CHANGELOG.md')
-rw-r--r-- | actionpack/CHANGELOG.md | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/actionpack/CHANGELOG.md b/actionpack/CHANGELOG.md index a2cf94fb42..82438cb766 100644 --- a/actionpack/CHANGELOG.md +++ b/actionpack/CHANGELOG.md @@ -1,5 +1,23 @@ ## Rails 4.0.0 (unreleased) ## +* Fix `content_tag_for` with array html option. + It would embed array as string instead of joining it like `content_tag` does: + + content_tag(:td, class: ["foo", "bar"]){} + #=> '<td class="foo bar"></td>' + + Before: + + content_tag_for(:td, item, class: ["foo", "bar"]) + #=> '<td class="item ["foo", "bar"]" id="item_1"></td>' + + After: + + content_tag_for(:td, item, class: ["foo", "bar"]) + #=> '<td class="item foo bar" id="item_1"></td>' + + *Semyon Perepelitsa* + * Remove `BestStandardsSupport` middleware, !DOCTYPE html already triggers standards mode per http://msdn.microsoft.com/en-us/library/jj676915(v=vs.85).aspx and ChromeFrame header has been moved to `config.action_dispatch.default_headers` |