Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-08-16 | 1 | -1/+13 |
|\ | |||||
| * | copy edits [ci skip] | Vijay Dev | 2012-08-16 | 1 | -6/+7 |
| | | |||||
| * | Document the namespacing of controller actions per this discussion: | schneems | 2012-08-09 | 1 | -0/+11 |
| | | | | | | https://github.com/rails/journey/issues/40 | ||||
* | | Indentation should consider line number character count. | Lucas Uyezu | 2012-08-13 | 1 | -2/+2 |
| | | |||||
* | | Add support for start_hour and end_hour options in select_hour helper | Evan Tann | 2012-08-13 | 1 | -1/+11 |
| | | | | | | | | Updated documentation to demonstrate start_hour and end_hour options | ||||
* | | Grammar fix in ActionView docs/comments | Edward Ocampo-Gooding | 2012-08-12 | 1 | -1/+1 |
| | | |||||
* | | Ensure option_html_attributes does not modify the given option hashes | Carlos Antonio da Silva | 2012-08-11 | 1 | -1/+1 |
| | | | | | | | | We can avoid creating extra hashes with #merge, and use #merge! instead. | ||||
* | | Simplify html attributes generation for options_for_select | Carlos Antonio da Silva | 2012-08-11 | 1 | -3/+5 |
| | | | | | | | | | | | | | | Further simplify the option_html_attributes method after the changes introduced in dacbcbe55745aa9e5484b10b11f65ccca7db1c54 to not escape the html options here (since they're going to be escaped down the chain in content tag). | ||||
* | | check for nil or empty record in form_for | schneems | 2012-08-10 | 1 | -0/+1 |
| | | | | | | | | if nil or an empty array is passed into form_for you get a horrible error message, this one is much more indicative of what the programmer needs to know to fix the problem. | ||||
* | | Do not mark strip_tags result as html_safe | Santiago Pastorino | 2012-08-09 | 1 | -1/+1 |
| | | | | | | | | | | | | Thanks to Marek Labos & Nethemba CVE-2012-3465 | ||||
* | | Allow data attributes to be set as a first-level option for form_for, so you ↵ | David Heinemeier Hansson | 2012-08-08 | 1 | -0/+19 |
| | | | | | | | | can write `form_for @record, data: { behavior: 'autosave' }` instead of `form_for @record, html: { data: { behavior: 'autosave' } }` *DHH* | ||||
* | | Modernize the documentation for view caching somewhat | David Heinemeier Hansson | 2012-08-08 | 1 | -16/+16 |
| | | |||||
* | | Deprecate `button_to_function` and `link_to_function` helpers. | Rafael Mendonça França | 2012-08-07 | 1 | -0/+40 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We recommend the use of Unobtrusive JavaScript instead. For example: link_to "Greeting", "#", :class => "nav_link" $(function() { $('.nav_link').click(function() { // Some complex code return false; }); }); or link_to "Greeting", '#', onclick: "alert('Hello world!'); return false", class: "nav_link" for simple cases. This reverts commit 3acdd652e9fe99481c879c84c5807a84eb9ad724. | ||||
* | | Revert "Merge pull request #7033 from kron4eg/master". Not a a fan at all of ↵ | David Heinemeier Hansson | 2012-08-07 | 1 | -10/+4 |
| | | | | | | | | | | | | | | what this makes ERB files look like. This reverts commit 46b8bceedd3e47169c50a04c93161424909c75fb, reversing changes made to 2f58795e783150f2e1b1f6c64e305703f0061129. | ||||
* | | Restoring the '%' trim mode for ERb templates, allowing for a leading ↵ | Artiom Di | 2012-08-07 | 1 | -4/+10 |
| | | | | | | | | percent sign on a line to indicate non-inserted Ruby code. | ||||
* | | Rearrange example output of javascript_include_tag | Prem Sichanugrist | 2012-08-06 | 1 | -2/+2 |
| | | |||||
* | | Do not include application.js if it doesn't exists | Prem Sichanugrist | 2012-08-06 | 1 | -4/+7 |
|/ | | | | | | Rails were including 'application.js' to the pack when using `javascript_include_tag :all` even there's no application.js in the public directory. | ||||
* | Use content_tag here instead of manually building HTML | Santiago Pastorino | 2012-08-03 | 1 | -2/+6 |
| | |||||
* | load active_support/deprecation in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 |
| | |||||
* | load active_support/core_ext/module/delegation in active_support/rails | Xavier Noria | 2012-08-02 | 2 | -2/+0 |
| | |||||
* | load active_support/core_ext/class/attribute in active_support/rails | Xavier Noria | 2012-08-02 | 4 | -4/+0 |
| | |||||
* | load active_support/concern in active_support/rails | Xavier Noria | 2012-08-02 | 2 | -2/+0 |
| | |||||
* | load active_support/core_ext/object/blank in active_support/rails | Xavier Noria | 2012-08-02 | 11 | -11/+0 |
| | |||||
* | Check validity of options[:as] just once | José Valim | 2012-08-02 | 1 | -12/+18 |
| | |||||
* | Improve error handling when using partial name with hyphen. Fix #7079 | kennyj | 2012-08-03 | 1 | -7/+12 |
| | |||||
* | flushing output should write to the stream rather than mutating the response ↵ | Aaron Patterson | 2012-07-29 | 1 | -1/+1 |
| | | | | object | ||||
* | Replace the flush parameter with a Hash. | Lucas Mazza | 2012-07-24 | 1 | -4/+4 |
| | |||||
* | Merge pull request #7123 from nashby/data-in-options | Rafael Mendonça França | 2012-07-22 | 1 | -1/+1 |
|\ | | | | | don't escape options in option_html_attributes method | ||||
| * | don't escape options in option_html_attributes method | Vasiliy Ermolovich | 2012-07-22 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | we don't need to escape values in this method as we pass these html attributes to `tag_options` method that handle escaping as well. it fixes the case when we want to pass html5 data options | ||||
* | | Add back `:disable_with` and change deprecation horizon to 4.1 | Carlos Galdino + Rafael Mendonça França | 2012-07-21 | 2 | -2/+39 |
| | | |||||
* | | Add back `:confirm` and change deprecation horizon to 4.1 | Carlos Galdino + Rafael Mendonça França | 2012-07-21 | 2 | -0/+56 |
|/ | |||||
* | Remove `:confirm` in favor of `:data => { :confirm => 'Text' }` option | Carlos Galdino | 2012-07-18 | 2 | -46/+21 |
| | | | | | | | | | This applies to the following helpers: `button_to` `button_tag` `image_submit_tag` `link_to` `submit_tag` | ||||
* | Common behavior with adding formats to lookup_context for TemplateRenderer ↵ | Dmitry Vorotilin | 2012-07-18 | 4 | -4/+16 |
| | | | | and PartialRenderer | ||||
* | add second t to overwritten | Evan Travers | 2012-07-12 | 1 | -2/+2 |
| | | | | fixing a misspelling in the documentation | ||||
* | Fixed bug creating invalid HTML in select options | Rusty Geldmacher | 2012-07-10 | 2 | -3/+3 |
| | | | | | | When a select tag is created for a field with errors, then the inserted options will errantly have a <div class="field_with_errors"> wrapping them. | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-07-07 | 1 | -1/+1 |
|\ | |||||
| * | use config.action_controller instead of ActionController::Base in example ↵ | Nick Howard | 2012-06-30 | 1 | -1/+1 |
| | | | | | | | | code for asset_tag_helper | ||||
* | | Merge pull request #6904 from guilleiguaran/remove-amo-dependency-in-ap | Carlos Antonio da Silva | 2012-06-30 | 1 | -13/+8 |
|\ \ | |/ |/| | Remove Active Model dependency from Action Pack | ||||
| * | Remove ActiveModel dependency from ActionPack | Guillermo Iguaran | 2012-06-30 | 1 | -13/+8 |
| | | | | | | | | | | | | | | | | ActiveModel is used in ActionPack for ActiveModel::Naming for a few, mostly optional aspects of ActionPack related to automatically converting an ActiveModel compliant object into a key for params and routing. It uses only three methods of ActiveModel (ActiveModel::Naming.route_key, ActiveModel::Naming.singular_route_key and ActiveModel::Naming.param_key). | ||||
* | | removing the AV::Base.helpers class attribute as it is not needed. | Nick Sutterer | 2012-06-29 | 1 | -1/+0 |
| | | |||||
* | | remove AV.prepare and move all helper-related logic into the controller. ↵ | Nick Sutterer | 2012-06-29 | 1 | -16/+0 |
|/ | | | | | | this decouples the view since it no longer knows about routes internals. this is a result of an ongoing discussion at https://github.com/rails/rails/pull/6826. | ||||
* | :update_details method no more exists on @lookup_context | Akira Matsuda | 2012-06-26 | 1 | -2/+1 |
| | | | | the method has gone in this commit: 119e9e2dafb0cdc5b85613b730333679aef534af | ||||
* | Do not propagate the :raise option to AS number helpers | Carlos Antonio da Silva | 2012-06-24 | 1 | -7/+19 |
| | | | | | ActiveSupport::NumberHelper does not make use of :raise, so there's no need to propagate it down. | ||||
* | eliminating another string subclass | Aaron Patterson | 2012-06-22 | 1 | -3/+7 |
| | |||||
* | fix some accidental nils | Aaron Patterson | 2012-06-22 | 2 | -3/+3 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-06-22 | 2 | -1/+2 |
|\ | |||||
| * | fixes a few mistakes in api docs [ci skip] | Vijay Dev | 2012-06-22 | 1 | -1/+1 |
| | | |||||
| * | Fix misprint in UrlHelper module | Alexey Vakhov | 2012-06-18 | 1 | -1/+1 |
| | | |||||
| * | Add rdoc to phone_field | Gaurish Sharma | 2012-06-16 | 1 | -0/+1 |
| | | | | | | [API docs](http://api.rubyonrails.org/classes/ActionView/Helpers/FormHelper.html#method-i-phone_field) for `phone_field` provide no information on what the method actually does. no description nor access to source code. so added this | ||||
* | | use mutex_m rather than use a delegate system | Aaron Patterson | 2012-06-21 | 1 | -6/+3 |
| | |