Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Update docs for appendTo to make it clear it is not a function to be used often #753

Merged
merged 1 commit into from

3 participants

@lukemelia
Collaborator

No description provided.

@travisbot

This pull request passes (merged dfa37f4 into c209f12).

@wagenet wagenet merged commit 8c2bd04 into emberjs:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 27, 2012
  1. @lukemelia
This page is out of date. Refresh to see the latest.
Showing with 7 additions and 1 deletion.
  1. +7 −1 packages/ember-views/lib/views/view.js
View
8 packages/ember-views/lib/views/view.js
@@ -608,7 +608,7 @@ Ember.View = Ember.Object.extend(Ember.Evented,
// we should return the className directly
if (!!val && className) {
return className;
-
+
// If value is a Boolean and true, return the dasherized property
// name.
} else if (val === true) {
@@ -704,6 +704,12 @@ Ember.View = Ember.Object.extend(Ember.Evented,
element will not be appended to the given element until all bindings have
finished synchronizing.
+ This is not typically a function that you will need to call directly
+ when building your application. You might consider using Ember.ContainerView
+ instead. If you do need to use appendTo, be sure that the target element you
+ are providing is associated with an Ember.Application and does not have an
+ ancestor element that is associated with an Ember view.
+
@param {String|DOMElement|jQuery} A selector, element, HTML string, or jQuery object
@returns {Ember.View} receiver
*/
Something went wrong with that request. Please try again.