Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #30165 from tjschuck/code_formatting_fix | Rafael França | 2017-08-11 | 1 | -10/+10 |
|\ | | | | | Add code formatting and minor grammar clarifications | ||||
| * | Add code formatting and minor grammar clarifications | T.J. Schuck | 2017-08-09 | 1 | -10/+10 |
| | | | | | | [ci skip] | ||||
* | | Fix RDoc formatting: `+` doesn't work with `@` | ohbarye | 2017-08-11 | 1 | -1/+1 |
|/ | | | | | | | | | | | | | | refs: https://github.com/rails/rails/pull/30161 ``` $ echo "+@size+" | rdoc --pipe <p>+@size+</p> $ echo "<tt>@size</tt>" | rdoc --pipe <p><code>@size</code></p> ``` [ci skip] | ||||
* | Fix broken RDoc formatting | T.J. Schuck | 2017-08-09 | 1 | -1/+1 |
| | | | | | The `@` ivar format doesn’t work with RDoc's `+` code formatting; needs `<tt>`. [ci skip] | ||||
* | Extend image_tag to accept ActiveStorage Attachments and Variants (#30084) | Anton Khamets | 2017-08-07 | 2 | -5/+23 |
| | | | | | | | | | | | | | | | | | | * Extend image_tag to accept ActiveStorage's Attachments and Variants * Flip resolve_image_source around * Add tests for the new use-cases of image_tag * Remove the higher-level test * Update image_tag documentation * Add error states into the test suite * Re-raise polymorhic_url's NoMethodError as ArgumentError * delegate_missing_to will raise DelegationError instead of NoMethodError | ||||
* | Merge pull request #30020 from rails/active-storage-import | David Heinemeier Hansson | 2017-08-04 | 5 | -2/+89 |
|\ | | | | | Add Active Storage to Rails | ||||
| * | Remove duplicated convert_direct_upload_option_to_url | Rafael Mendonça França | 2017-08-03 | 2 | -7/+4 |
| | | | | | | | | FormHelper includes FormTagHelper so we don't need to define two methods | ||||
| * | Refactor convert_direct_upload_option_to_url | Rafael Mendonça França | 2017-08-03 | 2 | -3/+35 |
| | | | | | | | | Also make sure file_field doesn't mutate the original options passed in. | ||||
| * | Make sure Action View doesn't break with Active Storage | Rafael Mendonça França | 2017-08-03 | 3 | -1/+50 |
| | | | | | | | | | | When Active Storage is not loaded and direct_upload is used on file_field_tag we should not raise an exception. | ||||
| * | Merge branch 'master' into active-storage-import | David Heinemeier Hansson | 2017-08-02 | 1 | -1/+1 |
| |\ | |||||
| * | | Add require | David Heinemeier Hansson | 2017-08-01 | 1 | -0/+1 |
| | | | |||||
| * | | Move the direct_upload: true convenience option from the activestorage ↵ | David Heinemeier Hansson | 2017-07-31 | 2 | -2/+10 |
| | | | | | | | | | | | | helper into actionview | ||||
* | | | Merge pull request #29984 from colorfulfool/form-helpers-unicode-sanitization | Rafael França | 2017-08-02 | 2 | -1/+15 |
|\ \ \ | |_|/ |/| | | Allow non-English values for collection_radio_buttons | ||||
| * | | Allow non-English values for collection_radio_buttons/check_boxes | colorfulfool | 2017-08-02 | 2 | -1/+15 |
| |/ | |||||
* / | Use File::NULL instead of "/dev/null" | Kazuhiro NISHIYAMA | 2017-07-31 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #30001 from y-yagi/fix_test_directory | Kasper Timm Hansen | 2017-07-30 | 3 | -3/+3 |
|\ | | | | | Fix test directory to correct path | ||||
| * | Fix test directory to correct path | yuuji.yaginuma | 2017-07-30 | 3 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Together, fix to the following lint violation. ``` rails/actionview/test/ujs/public/test/data-confirm.js 303:11 error Strings must use singlequote quotes rails/actionview/test/ujs/public/test/data-remote.js 414:32 error Extra semicolon semi ✖ 2 problems (2 errors, 0 warnings) ``` | ||||
* | | Fix testing helpers that use Action View's capturing helpers (e.g. content_for) | George Claghorn | 2017-07-25 | 2 | -0/+10 |
|/ | |||||
* | Suppress warning from actionview controller helper. | Christina Thompson | 2017-07-24 | 1 | -1/+1 |
| | | | | | | Explicitly pass a destructed array to the delegate method. Signed-off-by: Yuki Nishijima <yk.nishijima@gmail.com> | ||||
* | Merge pull request #29897 from kirs/frozen-actionview | Rafael França | 2017-07-24 | 177 | -1/+350 |
|\ | | | | | Use frozen string literal in actionview/ | ||||
| * | Use frozen string literal in actionview/ | Kir Shatrov | 2017-07-24 | 177 | -1/+350 |
| | | |||||
* | | Merge pull request #29792 from lugray/delegate_respond_to | Rafael França | 2017-07-24 | 2 | -2/+21 |
|\ \ | |/ |/| | Delegate respond_to? in ActionView::Helpers::ControllerHelper | ||||
| * | Delegate respond_to? in ActionView::Helpers::ControllerHelper | Lisa Ugray | 2017-07-19 | 2 | -2/+21 |
| | | | | | | | | | | | | Since methods defined in the controller helper are mostly delegated to the controller, delegate respond_to? as well, so that for example `respond_to?(:params)` behaves as expected. | ||||
* | | Make actionview ready for frozen strings | Kir Shatrov | 2017-07-23 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #29884 from padi/update_rails_5_upgrade_guide | Rafael França | 2017-07-21 | 1 | -2/+2 |
|\ \ | | | | | | | Updates Rails upgrade guide on `ActionView::Helpers::RecordTagHelper` | ||||
| * | | Suppress documentation for content_tag_for and div_for since they were ↵ | Marc Rendl Ignacio | 2017-07-21 | 1 | -2/+2 |
| | | | | | | | | | | | | removed already | ||||
* | | | Remove log directory in test postprocessing | yuuji.yaginuma | 2017-07-21 | 1 | -1/+1 |
| | | | | | | | | | | | | The "log" is directory. So it is necessary to use `FileUtils.rm_rf`. | ||||
* | | | Add jQuery to test vendor files | Marc Rendl Ignacio | 2017-07-20 | 2 | -1/+9832 |
| |/ |/| | | | | | | | ... so that we can run most, if not all, of rails-ujs tests without necessarily requiring an internet connection. | ||||
* | | [ci skip]Replace jquery-ujs with rails-ujs | willnet | 2017-07-19 | 1 | -1/+1 |
| | | |||||
* | | Enable `Layout/FirstParameterIndentation` cop | Ryuta Kamizono | 2017-07-17 | 2 | -24/+24 |
|/ | | | | | | | We have some indentation cops. But now there is a little inconsistent params indentations. Enable `Layout/FirstParameterIndentation` cop to prevent newly inconsistent indentation added and auto-correct to existing violations. | ||||
* | [Action View] `rubocop -a --only Layout/EmptyLineAfterMagicComment` | Koichi ITO | 2017-07-11 | 3 | -0/+3 |
| | |||||
* | Prepare AP and AR to be frozen string friendly | Kir Shatrov | 2017-07-06 | 2 | -1/+3 |
| | |||||
* | Merge branch 'master' into require_relative_2017 | Xavier Noria | 2017-07-02 | 24 | -236/+257 |
|\ | |||||
| * | Expectation first | Akira Matsuda | 2017-07-02 | 2 | -8/+8 |
| | | |||||
| * | Avoid shadowed variable | Matthew Draper | 2017-07-02 | 1 | -2/+2 |
| | | |||||
| * | Revert "Merge pull request #29540 from kirs/rubocop-frozen-string" | Matthew Draper | 2017-07-02 | 191 | -191/+1 |
| | | | | | | | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa. | ||||
| * | Merge pull request #29540 from kirs/rubocop-frozen-string | Matthew Draper | 2017-07-02 | 191 | -1/+191 |
| |\ | | | | | | | | | | Enforce frozen string in Rubocop | ||||
| | * | Enforce frozen string in Rubocop | Kir Shatrov | 2017-07-01 | 191 | -1/+191 |
| | | | |||||
| * | | Merge pull request #29506 from pat/frozen-string-literals | Matthew Draper | 2017-07-02 | 21 | -225/+225 |
| |\ \ | | | | | | | | | | | | | Make ActiveSupport frozen-string-literal friendly. | ||||
| | * | | Make ActionView frozen string literal friendly. | Pat Allan | 2017-06-20 | 19 | -220/+220 |
| | | | | | | | | | | | | | | | | Plus a couple of related ActionPack patches. | ||||
| | * | | Make ActionMailer frozen string literal friendly. | Pat Allan | 2017-06-20 | 2 | -3/+3 |
| | | | | |||||
| | * | | Make ActiveModel frozen string literal friendly. | Pat Allan | 2017-06-20 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | Includes two external changes because they're referenced within the ActiveModel test suite. | ||||
| * | | | Merge pull request #29349 from robertomiranda/responsive-images | Matthew Draper | 2017-07-02 | 3 | -3/+24 |
| |\ \ \ | | | | | | | | | | | | | | | | Add srcset option to image_tag helper | ||||
| | * | | | Add `srcset` option to `image_tag` helper | Roberto Miranda | 2017-06-29 | 3 | -2/+23 |
| | | |/ | | |/| | |||||
* | / | | [Action View] require => require_relative | Akira Matsuda | 2017-07-01 | 25 | -35/+35 |
|/ / / | |||||
* / / | Fallback Parameters#to_s to Hash#to_s | Kir Shatrov | 2017-06-30 | 1 | -0/+6 |
|/ / | | | | | Fixes https://github.com/rails/rails/issues/29617 | ||||
* | | Merge pull request #29588 from greysteil/add-gemspec-links | Rafael França | 2017-06-28 | 1 | -0/+5 |
|\ \ | | | | | | | Add source code and changelog links to gemspecs | ||||
| * | | Add source code and changelog links to gemspecs | Grey Baker | 2017-06-28 | 1 | -0/+5 |
| | | | |||||
* | | | :scissors: | Ryuta Kamizono | 2017-06-28 | 1 | -4/+4 |
|/ / | | | | | | | [ci skip] | ||||
* | | Add changes to CHANGELOG | savroff | 2017-06-19 | 1 | -0/+6 |
| | | | | | | | | | | | | fix fix |