Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix date_select :selected option so you can pass it nil | Colin Burn-Murdoch | 2013-01-16 | 1 | -1/+1 |
| | |||||
* | Merge pull request #8938 from colinbm/date_select_value | Steve Klabnik | 2013-01-14 | 2 | -1/+6 |
|\ | | | | | Rename :value to :selected for date_select, and add missing tests/docs | ||||
| * | Rename :value option to :selected, in line with other select helpers | Colin Burn-Murdoch | 2013-01-14 | 2 | -1/+6 |
| | | | | | | | | | | Add tests for time & datetime. Add documentation. | ||||
* | | Revert benchmark helper regression. Use a #capture within a #benchmark | Jeremy Kemper | 2013-01-14 | 1 | -13/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | block. Breaks benchmark calls that return non-String values otherwise. Revert "add benchmark helper that works in erb" This reverts commit 904e544cc8f5846de7c31827bb5556c6a238c0de. Conflicts: actionpack/lib/action_view/helpers.rb actionpack/lib/action_view/helpers/benchmark_helper.rb actionpack/test/template/benchmark_helper_test.rb | ||||
* | | Merge pull request #8916 from josemota/collection-with-block | Rafael Mendonça França | 2013-01-13 | 1 | -4/+4 |
|\ \ | |/ |/| | | | | | Collection radio buttons and collection check boxes through FormBuilder render the provided block. Closes #8897 | ||||
| * | Collection radio buttons and collection check boxes through FormBuilder render | José Mota | 2013-01-12 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | | the provided block. In the case of having a form_for method being called, the block for each collection would not be passed and thus the result expected was always the same. This patch passes the block to the original method like it would be assumed. | ||||
* | | Allow value to be set on date_select | Colin Burn-Murdoch | 2013-01-13 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #8623 from virusman/form_helpers_collectionproxy_fix | Jon Leighton | 2013-01-11 | 1 | -1/+1 |
|\ | | | | | Fixed nested fields bug when called with AssociationProxy | ||||
| * | Test nested fields with AssociationProxy mockup & fix AssociationProxy ↵ | virusman | 2012-12-27 | 1 | -1/+1 |
| | | | | | | | | support in form helper | ||||
* | | Remove redundant double quotation marks | zires | 2013-01-10 | 1 | -1/+1 |
| | | |||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-01-09 | 2 | -5/+5 |
|\ \ | | | | | | | | | | | | | Conflicts: guides/source/getting_started.md | ||||
| * | | PUT => PATCH | Akira Matsuda | 2013-01-03 | 2 | -5/+5 |
| | | | |||||
* | | | view_cache_dependency API | Jamis Buck | 2013-01-08 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A declarative API for specifying dependencies that affect template cache digest computation. In your controller, specify any of said dependencies: view_cache_dependency { "phone" if using_phone? } When the block is evaluated, the resulting value is included in the cache digest calculation, allowing you to generate different digests for effectively the same template. (Mostly useful if you're mucking with template load paths.) | ||||
* | | | set 'alt' attribute for image_submit_tag | Nihad Abbasov | 2013-01-08 | 1 | -7/+7 |
| | | | |||||
* | | | Refactor the logic that checks whether or not to emit the hidden id field | Carlos Antonio da Silva | 2013-01-06 | 1 | -7/+6 |
| | | | | | | | | | | | | | | | By checking for object.persisted? first, we avoid the hash lookups for new objects. | ||||
* | | | Move the hidden :id field logic to where it belongs to | Carlos Antonio da Silva | 2013-01-06 | 1 | -6/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | When dealing with nested forms, Rails automatically generates a hidden field with the id value of the current object being generated by fields_for. This logic was inside the method that's available from the template object, but we just need it when really dealing with nested attributes, so moving the code to here makes more sense. | ||||
* | | | Eliminate the usage of parent_builder option from form_for | Carlos Antonio da Silva | 2013-01-06 | 1 | -19/+15 |
| | | | | | | | | | | | | Just use it internally from fields_for until we come up with a better solution. | ||||
* | | | Do not call fields_for from form_for, to avoid instantiating two builders | Carlos Antonio da Silva | 2013-01-06 | 1 | -5/+5 |
| | | | |||||
* | | | Remove unnecessary begin..rescue..end, use only rescue | Akira Matsuda | 2013-01-06 | 1 | -8/+6 |
| | | | |||||
* | | | Fix documentation to content_tag_for [ci skip] | Rafael Mendonça França | 2013-01-05 | 1 | -2/+2 |
| | | | |||||
* | | | Merge pull request #8705 from amparo-luna/change_update_attributes_to_update | Guillermo Iguaran | 2013-01-03 | 2 | -5/+5 |
|\ \ \ | | | | | | | | | Rename update_attributes method to update | ||||
| * | | | Change docs to use update instead of update_attributes | Amparo Luna + Guillermo Iguaran | 2013-01-03 | 2 | -5/+5 |
| | | | | |||||
* | | | | Fix typo on form_tag_helper.rb | Paulo Luis Franchini Casaretto | 2013-01-03 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | The inline documentantion had a typo that was somewhat confusing. | ||||
* | | | | take into account multipart when calculating tag attributes | Aaron Patterson | 2013-01-02 | 1 | -4/+7 |
| | | | | |||||
* | | | | ask the form builder for form tag attributes | Aaron Patterson | 2013-01-02 | 1 | -2/+3 |
| | | | | |||||
* | | | | do not mutate the options hash (when we do not need to) | Aaron Patterson | 2013-01-02 | 1 | -2/+2 |
|/ / / | |||||
* / / | Make content_tag_for work without block | Rafael Mendonça França | 2013-01-02 | 1 | -1/+5 |
|/ / | | | | | | | This is version of #8640 for master | ||||
* | | Extract the component rendering of collection_radio_buttons and | Rafael Mendonça França | 2012-12-26 | 2 | -2/+14 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | collection_check_boxes to a private method. It will make easier to extend. Example: class CollectionRadioButtons < ActionView::Helpers::Tags::CollectionRadioButtons include CollectionExtensions def render wrap_rendered_collection(super, @options) end private def render_component(builder) builder.radio_button + builder.label(:class => "collection_radio_buttons") end end | ||||
* | | Fix collection_radio_buttons with the option `:checked` with value of | Rafael Mendonça França | 2012-12-26 | 1 | -1/+2 |
| | | | | | | | | `false` | ||||
* | | fetch highlighter option only when needed | Nihad Abbasov | 2012-12-24 | 1 | -2/+2 |
|/ | |||||
* | Remove obfuscation support from mail_to helper | Nick Reed | 2012-12-18 | 1 | -54/+12 |
| | | | | | | Removes support for :encode, :replace_at, and :replace_dot options from the mail_to helper. Support for these options has been extracted to the 'actionview-encoded_mail_to' gem. | ||||
* | handle protocol option in stylesheet_link_tag and javascript_include_tag | Vasiliy Ermolovich | 2012-12-18 | 1 | -2/+6 |
| | | | | closes #8388 | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-12-15 | 1 | -0/+412 |
|\ | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/form_helper.rb railties/lib/rails/info_controller.rb | ||||
| * | repeating documentation for form helpers | Thiago Pinto | 2012-12-09 | 1 | -18/+406 |
| | | |||||
* | | Improve the documentation of cache_if and cache_unless | Rafael Mendonça França | 2012-12-14 | 1 | -4/+7 |
| | | |||||
* | | Merge pull request #8497 from acapilleri/conditional_cache | Rafael Mendonça França | 2012-12-14 | 1 | -12/+24 |
|\ \ | | | | | | | | | | Removed :if / :unless conditions to fragment cache in favour of *cache_i... | ||||
| * | | Removed :if and :unless from fragment cache option in favour of | Angelo capilleri | 2012-12-14 | 1 | -12/+24 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cache_if(condition, option, &block) and cache_unless(condition, option, &block). In the PR #8371 was introduced conditional options :if and :unless in the cache method. Example: <%= cache @model, if: some_condition(@model) do %> ... <%end%> This is a good feature but *cache_if* and and *cache_unless* are more concise and close to the standard of rails view helpers (ex: link_to_if and link_to_unless). Example: <%= cache_if condition, @model do %> ... <%end%> | ||||
* | | | Add missing require. | Rafael Mendonça França | 2012-12-13 | 2 | -0/+3 |
|/ / | | | | | | | | | I was trying to use those files without Rails and that require was missing. | ||||
* | | Remove deprecation message from Action Pack | Carlos Antonio da Silva | 2012-12-11 | 1 | -1/+1 |
| | | | | | | | | | | Related to the deprecation of Time.utc_time in favor of Time.utc, in 48583f8bf74d1cefefea3cd6591bd546a9eaff6c. | ||||
* | | Revert "API reader should look elsewhere for helper instructions" | Vijay Dev | 2012-12-09 | 1 | -24/+0 |
|/ | | | | | | | | | This reverts commit ba2d867b2961b5ef72aad353c2d647b04654cbe3. Reason: https://github.com/lifo/docrails/commit/ba2d867b2961b5ef72aad353c2d647b04654cbe3#commitcomment-2276670 [ci skip] | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-12-08 | 2 | -2/+40 |
|\ | |||||
| * | API reader should look elsewhere for helper instructions | Thiago Pinto | 2012-12-06 | 1 | -0/+24 |
| | | |||||
| * | complementary options guidelines for f.file_field and file_field_tag | Thiago Pinto | 2012-12-06 | 2 | -0/+8 |
| | | |||||
| * | adding example for f.file_input | Thiago Pinto | 2012-12-06 | 1 | -0/+3 |
| | | |||||
| * | adding example for f.file_input | Thiago Pinto | 2012-12-06 | 1 | -0/+3 |
| | | |||||
| * | correct bad jquery syntax | Thiago Pinto | 2012-12-06 | 1 | -2/+2 |
| | | |||||
* | | Allow fragment cache to accept :if and :unless options | Fabrizio Regini | 2012-12-05 | 1 | -1/+13 |
|/ | | | | [Stephen Ausman + Fabrizio Regini] | ||||
* | copy edits [ci skip] | Vijay Dev | 2012-12-04 | 4 | -27/+18 |
| | |||||
* | Cleans and removes 'Examples' tag [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -2/+0 |
| | |||||
* | Cleans and removes useless 'Examples' tag [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -48/+43 |
| |