1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
rails--rails/actionview/lib/action_view
claudiob 943ebcb5f1 Better docs for AV::RecordIdentifier
This commit intends to clarify the scope of ActionView::RecordIdentifier
methods `dom_id` and `dom_class`.

Most of the current documentation comes from da257eb8 (7 years ago) when
the decoupling of ActionView, ActiveRecord and ActiveModel was not a concern.

Since then, steps have been taken to reach such decoupling, especially
8ca17926 which duplicated ActionController::ModelNaming into
ActionView::ModelNaming explaining that:

> These are just a simple helpers for decoupling Active Model, so it does not
> make sense to extract it to Active Support, but the point is to decouple also
> Action View and Action Pack

As of today, ActionView::RecordIdentifier only includes `dom_id` and `dom_class`
so it makes sense to explicitly document those two methods, and leaving the
details of helpers like `div_for` in the corresponding files.

Moreover, I think it's important to mention in the documentation that
ActionView::RecordIdentifier **does not strictly depend on the ActiveRecord API**: any class `Post` implementing `post.to_key` and `post.model_name.param_key` will work.

[ci skip]
2014-12-23 02:01:53 +01:00
..
helpers [ci skip] Fix HTML format on comment 2014-12-18 13:07:32 +09:00
locale
renderer Fix warning in AV tests 2014-11-25 15:57:00 -07:00
tasks Fix cache_digest rake tasks 2014-06-10 13:52:36 +02:00
template Pass symbol as an argument instead of a block 2014-11-29 11:53:24 +01:00
testing Removed ActionView::Assertions. Getting ready to exchange with Rails::Dom::Testing::Assertions. 2014-06-16 21:04:17 +02:00
base.rb Fix link in documentation. [ci skip] 2014-11-30 00:15:56 +02:00
buffers.rb handle <%== nil %> cases 2014-09-14 17:11:13 -07:00
context.rb
dependency_tracker.rb Fix that render layout should also be picked up by the template dependency tracker, but only half-ways. You can add that layout option on the same render call, and both templates should be added to the dependency tree. But thats going to require a more serious rework of the tracker. Please do help fix this part of it too. For now, render layout needs to be on its own line. 2014-07-25 16:37:18 -07:00
digestor.rb Log digest as :debug instead of :info 2014-07-15 14:18:32 +10:00
flows.rb
gem_version.rb Start Rails 5 development 🎉 2014-11-28 15:00:06 -02:00
helpers.rb Make ActionView::Tags loading tread safe 2013-12-02 20:27:50 -02:00
layouts.rb Pass symbol as an argument instead of a block 2014-11-29 11:53:24 +01:00
log_subscriber.rb Prefer to pass block when logging. 2014-07-18 15:04:43 +08:00
lookup_context.rb Use &= instead of select with include? 2014-07-16 16:42:53 -03:00
model_naming.rb Use #model_name on instances instead of classes 2014-06-24 17:20:24 -07:00
path_set.rb
railtie.rb Remove the explicit order set for the initializer 2013-12-05 16:37:10 -02:00
record_identifier.rb Better docs for AV::RecordIdentifier 2014-12-23 02:01:53 +01:00
rendering.rb let mailer templates generate URLs by default [Xavier Noria, Richard Schneeman] 2014-11-24 14:33:50 -08:00
routing_url_for.rb let mailer templates generate URLs by default [Xavier Noria, Richard Schneeman] 2014-11-24 14:33:50 -08:00
template.rb Reduce object allocation 2014-10-25 12:59:37 +09:00
test_case.rb Make sure assert_select can assert body tag 2014-11-18 18:47:22 -02:00
version.rb Introduce Rails.gem_version 2014-03-05 12:37:38 -05:00
view_paths.rb edit pass over all warnings 2014-10-28 17:47:32 -07:00