Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | fix typo asser url [ci skip] | Mehmet Emin İNAÇ | 2015-04-16 | 1 | -6/+6 |
| | |||||
* | Merge pull request #19774 from davydovanton/doc-update-asser-url | Abdelkader Boudih | 2015-04-15 | 1 | -0/+30 |
|\ | | | | | [Skip ci] Add information about :host options | ||||
| * | [Skip ci] Add information about :host options | Anton Davydov | 2015-04-16 | 1 | -0/+30 |
| | | | | | | | | | | | | Added informations about :host options for each url helper (#javascript_url, #video_url, #stylesheet_url, etc) based on #asser_url method. | ||||
* | | Merge pull request #19736 from kmcphillips/master | Rafael Mendonça França | 2015-04-14 | 2 | -1/+4 |
|\ \ | | | | | | | Set default form builder for a controller | ||||
| * | | Override default form builder for a controller | Kevin McPhillips | 2015-04-13 | 2 | -1/+4 |
| | | | |||||
* | | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-04-14 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | |||||
| * | | [ci skip] Add `,` | yui-knk | 2015-04-08 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #19566 from aditya-kapoor/remove-dup-doc | Richard Schneeman | 2015-04-11 | 1 | -22/+10 |
|\ \ \ | | | | | | | | | [ci skip] remove duplicate doc for current_page? | ||||
| * | | | [ci skip] remove duplicate doc for current_page? | Aditya Kapoor | 2015-04-09 | 1 | -22/+10 |
| | | | | |||||
* | | | | Merge pull request #17512 from ↵ | Zachary Scott | 2015-04-10 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | JackDanger/danger/use-default-view-pattern-in-docs [docs] Using the real resolver pattern in docs | ||||
| * | | | | Using the real resolver pattern in docs | Jack Danger Canty | 2014-11-04 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If someone copies the docs into their app they'll find it simply doesn't work because the locale pattern doesn't have the same logic. This makes the doc examples work exactly as written. [ci skip] | ||||
* | | | | | Merge pull request #19685 from vngrs/actionview_parent_layout_bug | Rafael Mendonça França | 2015-04-08 | 1 | -8/+17 |
|\ \ \ \ \ | |_|_|_|/ |/| | | | | fix for actionview parent layout bug | ||||
| * | | | | fix for actionview parent layout bug | Mehmet Emin İNAÇ | 2015-04-08 | 1 | -8/+17 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit fixes issue #19626 Don't need to check layout conditions if there is no condition test for parent layout bug fix | ||||
* | | | | | Merge pull request #19661 from Azdaroth/fields-for-child-index-lambda | Rafael Mendonça França | 2015-04-06 | 1 | -1/+5 |
|\ \ \ \ \ | |/ / / / |/| | | | | Accept lambda as child_index option in #fields_for method | ||||
| * | | | | Accept lambda as child_index option in #fields_for method | Karol Galanciak | 2015-04-05 | 1 | -1/+5 |
| | | | | | |||||
* | | | | | head no_content when there is no template or action performed | Stephen Bussey | 2015-04-05 | 1 | -1/+1 |
|/ / / / | |||||
* | | | | Merge pull request #19644 from y-yagi/auto_discovery_link_tag_doc | Yves Senn | 2015-04-05 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | correct output of auto_discovery_link_tag [ci skip] | ||||
| * | | | | correct output of auto_discovery_link_tag [ci skip] | yuuji.yaginuma | 2015-04-04 | 1 | -1/+1 |
| | | | | | |||||
* | | | | | Allow an array to be a default translation value. | Adam Prescott | 2015-04-04 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4.2.1 introduced a change to the way `translate`/`t` works with an option of `default: [[]]`. In 4.2.0, this would give a default value of `[]`, but in 4.2.1, it leads to a missing translation. `default: [[]]` is again allowed for cases where a default of `[]` is needed. This addresses GitHub issue 19640. | ||||
* | | | | | [skip ci] Update information about #link_to attributes | Anton Davydov | 2015-04-04 | 1 | -4/+3 |
|/ / / / | |||||
* | | | | Freeze static arguments for gsub | brainopia | 2015-04-02 | 1 | -1/+1 |
| | | | | |||||
* | | | | Prefer string patterns for gsub | brainopia | 2015-04-02 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/ruby/ruby/pull/579 - there is a new optimization since ruby 2.2 Previously regexp patterns were faster (since a string was converted to regexp underneath anyway). But now string patterns are faster and better reflect the purpose. Benchmark.ips do |bm| bm.report('regexp') { 'this is ::a random string'.gsub(/::/, '/') } bm.report('string') { 'this is ::a random string'.gsub('::', '/') } bm.compare! end # string: 753724.4 i/s # regexp: 501443.1 i/s - 1.50x slower | ||||
* | | | | [skip ci] Add information about #link_to target option | Anton Davydov | 2015-04-01 | 1 | -0/+6 |
| |/ / |/| | | |||||
* | | | Merge pull request #19535 from JuanitoFatas/doc/av-comment | Yves Senn | 2015-03-27 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | [ci skip] Unify the code comment in ActionView::Renderer#render_template... | ||||
| * | | | [ci skip] Unify the code comment in ActionView::Renderer#render_template. | Juanito Fatas | 2015-03-27 | 1 | -1/+1 |
| | |/ | |/| | |||||
* / | | Add missing :html option in determine_template error message. | Juanito Fatas | 2015-03-27 | 1 | -1/+1 |
|/ / | |||||
* | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-03-26 | 3 | -10/+11 |
|\ \ | | | | | | | | | | | | | Conflicts: guides/source/4_0_release_notes.md | ||||
| * | | [ci skip] Fix `&>` with `%>` | yui-knk | 2015-03-25 | 1 | -9/+9 |
| | | | |||||
| * | | [ci skip] Add a line between code example and docs | yui-knk | 2015-03-21 | 1 | -0/+1 |
| | | | |||||
| * | | [ci skip] Add space after erb block. | yui-knk | 2015-03-12 | 1 | -1/+1 |
| | | | |||||
* | | | Simplify setting button form options | Carlos Antonio da Silva | 2015-03-22 | 1 | -2/+3 |
| | | | | | | | | | | | | No need to merge hashes when simply setting options does the job. | ||||
* | | | Rework form helper example to use Person.new | Carlos Antonio da Silva | 2015-03-21 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous change 92a3c8dcdf174dab5e8759031b6bbe6cd891fe3c improved this example, but calling .create + #valid? ends up running validations twice, and we don't want to showcase that. Lets use the normal use case of building a new object and calling #save on it, which is what a basic scaffold with Active Record does. [ci skip] | ||||
* | | | [ci skip] Check a result of `valid?` instead of `create` | yui-knk | 2015-03-21 | 1 | -1/+2 |
| | | | |||||
* | | | Merge pull request #19421 from jcoyne/translate_defaults_with_nil | Rafael Mendonça França | 2015-03-20 | 1 | -1/+1 |
| | | | | | | | | | | | | Strip nils out of default translations. Fixes #19419 | ||||
* | | | Remove additional handling of boolean attributes on button to helper | Carlos Antonio da Silva | 2015-03-18 | 1 | -30/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | This logic was just doing duplicated work, since the button_to helper relies on tag/content_tag to generate the button html, which already handles all boolean attributes it knows about. The code dates back to 2005: 43c470fae468ef63e0d5c3dc1e202925685fd47b. | ||||
* | | | [ci skip] correct output of select | Kuldeep Aggarwal | 2015-03-13 | 1 | -15/+15 |
| | | | |||||
* | | | [CI SKIP] Correct output of options_for_select | Akshay Vishnoi | 2015-03-13 | 1 | -5/+5 |
| | | | |||||
* | | | Let strip_tags leave HTML escaping to Rails. | Kasper Timm Hansen | 2015-03-10 | 1 | -1/+1 |
| | | | | | | | | | | | | Prevents double escaping errors, such as "&" becoming "&". | ||||
* | | | don't need to rescue Exception in this case | Aaron Patterson | 2015-03-05 | 1 | -1/+1 |
| | | | |||||
* | | | [ci skip] Delete excess white in documentation | Anton Davydov | 2015-03-05 | 1 | -1/+1 |
| | | | |||||
* | | | [ci skip] Add information about sanitize option in TextHelper#highlight | Anton Davydov | 2015-03-05 | 1 | -1/+6 |
| | | | |||||
* | | | Merge pull request #19144 from y-yagi/fix_streaming_buffer | Rafael Mendonça França | 2015-03-04 | 1 | -1/+3 |
|\ \ \ | | | | | | | | | fix ActionView::Template::Error when using Streaming with capture. | ||||
| * | | | fix ActionView::Template::Error when using Streaming with capture. | yuuji.yaginuma | 2015-03-01 | 1 | -1/+3 |
| |/ / | | | | | | | | | | can't acquire a encoding from StreamingBuffer. fixes #12001 | ||||
* | | | Fix spelling [ci skip] | Yu Haidong | 2015-03-04 | 1 | -8/+8 |
| | | | |||||
* | | | move _generate_paths_by_default to where it is used | Aaron Patterson | 2015-03-01 | 1 | -0/+6 |
|/ / | | | | | | | | | _generate_paths_by_default wasn't used in AD::Routing::UrlFor, so we should be able to move it where it is used in AV::Routing | ||||
* | | Fix regression when passing a value different of String. | Ulisses Almeida | 2015-02-26 | 1 | -1/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous version of rails(4.2.0) you can pass objects to the default option of translation helper. For example: ```ruby t('foo', default: 1) ``` But on rails 4.2.1 version this kind of use stopped to work, because started only to accept String types. Now with this fix we can use orther value types on this helper again. | ||||
* | | Fix typos in ActionView::Helpers::FormBuilder comment [ci skip] | Yu Haidong | 2015-02-26 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #19074 from ↵ | Rafael Mendonça França | 2015-02-25 | 1 | -4/+15 |
|\ \ | | | | | | | | | | | | | Mession/fix_test_dependency_of_erb_template_partial_with_layout Fix dependency tracker bug | ||||
| * | | Fix dependency tracker bug | Juho Leinonen | 2015-02-25 | 1 | -4/+15 |
| | | | |||||
* | | | Merge pull request #18948 from kaspth/automatic-collection-caching | Rafael Mendonça França | 2015-02-25 | 6 | -2/+145 |
|\ \ \ | |/ / |/| | | Merge multi_fetch_fragments. |