aboutsummaryrefslogtreecommitdiffstats
path: root/actionview/lib/action_view/helpers
Commit message (Collapse)AuthorAgeFilesLines
* Allocate one less object using html_safe during content_tag constructionJosh Jordan2013-11-141-1/+1
|
* Revert "Used Yield instead of block.call" -- this causes all of ↵David Heinemeier Hansson2013-11-141-2/+2
| | | | | | atom_feed_helper_test.rb to fail with "SystemStackError: stack level too deep". This reverts commit d3a1ce1cdc60d593de1682c5f4e3230c8db9a0fd.
* Merge pull request #12889 from kuldeepaggarwal/speed_upsRafael Mendonça França2013-11-141-2/+2
|\ | | | | Used Yield instead of block.call
| * Used Yield instead of block.callKuldeep Aggarwal2013-11-151-2/+2
| |
* | Fix syntax error in atom_feed example [ci skip]Saulius Grigaliunas2013-11-131-1/+1
|/ | | | Builder's #tag! takes either String or Symbol as the first parameter
* Make the method name the first argumentRafael Mendonça França2013-11-091-7/+7
| | | | | This is the only argument that changes over the method calls so it is better to it be the first one
* Merge pull request #12760 from pseidemann/masterYves Senn2013-11-091-1/+1
|\ | | | | fix simple_format escapes own output when sanitize is set to true
| * fix simple_format escapes own output when sanitize is set to truepseidemann2013-11-081-1/+1
| |
* | Extract common code from number helpers to new delegator method.Vipul A M2013-11-091-30/+13
| |
* | Merge pull request #12788 from kylefritz/patch-1Rafael Mendonça França2013-11-061-2/+2
|/ | | | | | rdoc: favicon source shouldn't begin with a slash to reference asset pipeline resource [ci skip] Conflicts: actionview/lib/action_view/helpers/asset_tag_helper.rb
* Line up the comments for aestheticsDavid Heinemeier Hansson2013-11-031-5/+5
|
* Convert CDATA input to string before gsub'ingCarsten Zimmermann2013-10-291-1/+1
| | | | | Rails 3.2 API allowed arbitrary input for cdata_section; this change re-introduces the old behaviour.
* Remove warning of shadowing outer local variableRafael Mendonça França2013-10-271-2/+2
|
* Merge pull request #10471 from andyw8/button_to_paramsRafael Mendonça França2013-10-241-0/+7
|\ | | | | | | | | | | | | Add params option for button_to Conflicts: actionpack/CHANGELOG.md
| * Add params option for button_toAndy Waite2013-09-181-0/+7
| | | | | | | | | | | | The parameters are rendered as hidden form fields within the generated form. This is useful for when a record has multiple buttons associated with it, each of which target the same controller method, but which need to submit different attributes.
* | Fix typo in form_helper.rbAdam Jahnke2013-10-161-1/+1
| |
* | Merge pull request #12491 from shamanime/patch-1Rafael Mendonça França2013-10-131-2/+2
|\ \ | | | | | | Fix wrong variable name used in the select_day method documentation
| * | Fix wrong variable name used in the select_day method documentationJefferson Queiroz Venerando2013-10-101-2/+2
| | | | | | | | | The variable name created in the example is `my_date`, the methods were using `my_time` instead.
* | | Change `map` to `map!` to save extra array creation on new arrayVipul A M2013-10-131-1/+1
|/ /
* | Fixed return strings in documentationJoseph Zidell2013-10-031-3/+3
| |
* | Fixed typo in documentationJoseph Zidell2013-10-031-3/+3
| |
* | Use the given name in html_options for the hidden field in ↵Angel N. Sciortino2013-09-241-1/+2
| | | | | | | | collection_check_boxes
* | Merge pull request #10773 from wangjohn/link_and_routing_optionsRafael Mendonça França2013-09-231-2/+3
| | | | | | | | Adding documentation and tests to ``polymorphic_url`` and ``link_to``
* | Fix some edge cases for AV `select` helper with `:selected` optionBogdan Gusiev2013-09-231-1/+2
| |
* | Expand select documentation to tell about the blockRafael Mendonça França2013-09-231-0/+9
| | | | | | | | [ci skip]
* | Ability to pass block to AV#select helperBogdan Gusiev2013-09-232-5/+6
| | | | | | | | | | | | | | | | Example: = select(report, "campaign_ids") do - available_campaigns.each do |c| %option{:data => {:tags => c.tags.to_json}, :value => c.id}= c.name
* | handle `:namespace` form option in collection labelsVasiliy Ermolovich2013-09-222-1/+3
| |
* | Merge pull request #12112 from adamniedzielski/fix-form-for-namespace-and-asRafael Mendonça França2013-09-121-1/+2
|\ \ | | | | | | | | | | | | | | | | | | form_for - fix :namespace and :as options clash Conflicts: actionview/CHANGELOG.md
| * | form_for - fix :namespace and :as options clashAdam Niedzielski2013-09-091-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | :as option should not overwrite :namespace option when generating html id attribute of the form element. id should be prefixed by specified namespace even if :as option is present Add test case showing the issue and code fixing it
* | | Cleanup of excerpt helperPaul Nikitochkin2013-09-061-2/+3
|/ / | | | | | | | | * replaced String concatenation by joining * separator has default value to '', even it is nil
* | Fix a few typos. [ci skip]namusyaka2013-09-041-2/+2
| |
* | fix the typo [ci skip]Rajarshi Das2013-09-021-1/+1
| | | | | | fix the typo newstopics => new topics in action view cache helper
* | Add missing requires inside AVŁukasz Strzałkowski2013-08-251-0/+2
|/
* RDoc tweaksXavier Noria2013-08-071-5/+10
|
* Removed an unnecessary loop - it kills performance on large texts.Jan Szumiec2013-08-031-4/+6
|
* Add "extname" option to javascript_include_tagNathan Stitt2013-08-031-3/+6
| | | | | | | | | | | | ActionView::Helpers.asset_path is where the logic for javascript_include_tag resides. It takes an extname option for specifying the extension or false to not append it. This exposes that option to javascript_include_tag. Without the option files that didn't end with ".js" would get the extension appended to them. This broke JST templates and other file types that should be interpreted as JavaScript but who's file extension isn't ".js"
* Using URI.parser.unescapeArun Agrawal2013-08-031-2/+2
| | | | | | Fixes warning warning: URI.unescape is obsolete
* Make current_page? compare binary stringsRafael Mendonça França2013-08-011-3/+4
|
* Fix `current_page?` when the URL contains escaped charactersRafael Mendonça França2013-08-011-2/+2
| | | | | | | In some cases webservers like nginx send the escaped characters lowercased to the Rails application. The current_page? helper was comparing the escaped strings that are different since Ruby escapes the URL using uppercased characters.
* Change from `map` => `map!` and `collect!` to save creation of extra array.Vipul A M2013-07-311-2/+2
|
* text_area should handle nil value option like text_fieldJoel Cogen2013-07-242-3/+3
|
* Leave the knowledge of boolean tag values to content tagCarlos Antonio da Silva2013-07-221-3/+3
| | | | | | content tag already knows which tags are boolean and the values that should generate them when a truthy value is passed, so leave this logic for it instead of duplicating when generating options tags.
* Use merge! to avoid a new hash and change some spots to 1.9 hash styleCarlos Antonio da Silva2013-07-221-7/+7
|
* syntax fix for f.select doc. [ci skip] .Karunakar (Ruby)2013-07-221-1/+1
|
* add support for html attributes to grouped_options_for_selectVasiliy Ermolovich2013-07-201-1/+5
|
* Extract verbs array from helper to a constantVipul A M2013-07-201-2/+2
|
* Merge pull request #11383 from mattdbridges/simple-format-doc-fixRafael Mendonça França2013-07-091-1/+1
|\ | | | | Fix simple_format output example ending tag
| * Fix simple_format output example ending tagMatt Bridges2013-07-091-1/+1
| |
* | Merge pull request #11348 from sanemat/fix/link-to-blockRafael Mendonça França2013-07-071-1/+1
|\ \ | | | | | | Fix actionview link_to with block and url_hash
| * | Include block in a argument shiftsanemat2013-07-081-6/+2
| | |