aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
* Fix a number of validation/style errors:Sam Ruby2013-01-022-8/+7
| | | | | | | | | | * <pre> is not allowed to be nested inside of <p> elements in HTML * Indentation of </p> doesn't match corresponding <p> * <p> element not explicitly closed * One more </div> than <div> In each case, the template was fixed to match how a HTML5 parser would "see" the resulting page.
* take into account multipart when calculating tag attributesAaron Patterson2013-01-021-4/+7
|
* ask the form builder for form tag attributesAaron Patterson2013-01-021-2/+3
|
* do not mutate the options hash (when we do not need to)Aaron Patterson2013-01-021-2/+2
|
* Merge pull request #8694 from goshakkk/undupCarlos Antonio da Silva2013-01-021-2/+1
|\ | | | | Remove unnecessary #dup
| * remove unnecessary #dupGosha Arinich2013-01-021-2/+1
| |
* | Merge pull request #8688 from goshakkk/error-page-toggleGuillermo Iguaran2013-01-022-3/+7
|\ \ | |/ |/| Allow toggling dumps on error page
| * allow toggling dumps instead of just showingGosha Arinich2013-01-022-3/+7
| |
* | Cleanup some unnecessary CSS on the new error page and reformat some lines.Lucas Mazza2013-01-021-12/+13
| |
* | Merge pull request #8689 from goshakkk/error-pageRafael Mendonça França2013-01-021-0/+4
|\ \ | | | | | | Add source code left padding on error pages
| * | add source line paddingGosha Arinich2013-01-021-0/+4
| |/
* / No need to pass a empty block to content_tag_for anymoreRafael Mendonça França2013-01-021-11/+11
|/
* Make content_tag_for work without blockRafael Mendonça França2013-01-023-1/+17
| | | | This is version of #8640 for master
* remove meaningless AS::FrozenObjectErrorAkira Matsuda2013-01-021-1/+0
|
* Add CHANGELOG entry for changes in exception pagesGuillermo Iguaran2013-01-011-0/+6
|
* Fix indent in UnknownAction templateGuillermo Iguaran2013-01-011-1/+1
|
* Merge pull request #8668 from guilleiguaran/exceptionsDavid Heinemeier Hansson2013-01-0112-65/+258
|\ | | | | New exceptions pages for development
| * formatted_code_for should be privateGuillermo Iguaran2012-12-311-12/+12
| |
| * Summary and Details HTML elements aren't supported in all modern browsersGuillermo Iguaran2012-12-312-12/+12
| |
| * Fix test for DebugExceptions due to template changeGuillermo Iguaran2012-12-311-1/+1
| |
| * Add style to AV::Template::Error exception pageGuillermo Iguaran2012-12-313-18/+54
| |
| * Improve line-height to have better line spacing in exception messageGuillermo Iguaran2012-12-311-0/+1
| |
| * Add new style to Routing Error pageGuillermo Iguaran2012-12-311-19/+22
| |
| * Styling for exception pageGuillermo Iguaran2012-12-316-29/+150
| |
| * Add source extract to detailed exception pageGuillermo Iguaran2012-12-314-2/+34
| |
* | Updated copyright notices for 2013Andrew Nesbitt2012-12-314-4/+4
|/
* Use ActiveSupport::TestCase in the journey testsRafael Mendonça França2012-12-3113-13/+13
|
* Merge pull request #8666 from mekishizufu/fix_lambdasSantiago Pastorino2012-12-311-2/+2
|\ | | | | Fix usage of lambda as a Rack endpoint
| * Fix usage of lambda as a Rack endpointJiri Pospisil2012-12-311-2/+2
| | | | | | | | The response body needs to respond_to? :each.
* | Alias refute methods to assert_not and perfer assert_not on testsRafael Mendonça França2012-12-3113-23/+23
| |
* | Do not use the same tests descriptionRafael Mendonça França2012-12-311-3/+3
| |
* | Remove unneeded testsRafael Mendonça França2012-12-315-381/+0
| | | | | | | | These tests are needed only if we are using MiniTest::Spec
* | Add active_support/testing/autorunRafael Mendonça França2012-12-313-3/+3
| | | | | | | | | | minitest/autorun load minitest/spec polluting the global namespace with the DSL that we don't want on Rails
* | Inherit from MiniTest::Unit::TestCase instead of MiniTest::SpecRafael Mendonça França2012-12-313-13/+0
|/
* Merge pull request #8662 from ↵Santiago Pastorino2012-12-314-3/+25
|\ | | | | | | | | senny/8661_should_not_append_charset_if_already_present Charset should not be appended to image/* type
| * charset should not be appended for `head` responsesYves Senn2012-12-314-3/+25
| | | | | | | | | | | | | | 1) Failure: test_head_created_with_image_png_content_type(RenderTest) [test/controller/render_test.rb:1238]: Expected: "image/png" Actual: "image/png; charset=utf-8"
* | Define [], []=, permanent, signed and encrypted as the only allowed methods ↵Santiago Pastorino2012-12-301-9/+57
| | | | | | | | for the non Raw Cookie classes
* | Merge pull request #8546 from hsbt/fix-testcase-strict-warningSantiago Pastorino2012-12-301-1/+2
|\ \ | |/ |/| fix testcase: ruby-2.0.0 warned unused variables
| * change spy of after invoked controller action. because ruby-2.0.0 waned ↵SHIBATA Hiroshi2012-12-201-1/+2
| | | | | | | | unused variables
* | Merge pull request #8627 from schneems/schneems/routes-1-9Rafael Mendonça França2012-12-261-1/+1
|\ \ | | | | | | Ruby 1.9 style hash in Route Inspector
| * | Ruby 1.9 style hash in Route Inspectorschneems2012-12-271-1/+1
| | |
* | | Extract the component rendering of collection_radio_buttons andRafael Mendonça França2012-12-262-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 ofRafael Mendonça França2012-12-262-1/+8
| | | | | | | | | | | | `false`
* | | Merge pull request #8598 from NARKOZ/refactor-highlightSantiago Pastorino2012-12-261-2/+2
|\ \ \ | | | | | | | | refactor highlight method: return fast if text or phrases are blank
| * | | fetch highlighter option only when neededNihad Abbasov2012-12-241-2/+2
| | | |
* | | | Fix CHANGELOG style [ci skip]Rafael Mendonça França2012-12-261-1/+1
| | | |
* | | | We need to check explictly that env['action_dispatch.show_exceptions']Rafael Mendonça França2012-12-261-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | is false. If it is nil we can't raise the exception
* | | | more descriptive return parametersHrvoje Šimić2012-12-261-4/+3
|/ / /
* | | "warning: ambiguous first argument; put parentheses or even spaces"Akira Matsuda2012-12-241-1/+1
| | |
* | | we're not supporting SQLite 2Akira Matsuda2012-12-241-13/+5
| | |