From 08b6b28902e473ddbbbccfa8a65e83a4f5760e8a Mon Sep 17 00:00:00 2001 From: Riley Tomasek Date: Fri, 21 Nov 2014 12:05:39 -0500 Subject: [PATCH 1/2] Fix typo in comment of jquery-bootstrap example --- examples/jquery-bootstrap/js/app.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/jquery-bootstrap/js/app.js b/examples/jquery-bootstrap/js/app.js index 042c6720eb..e6f97a4152 100644 --- a/examples/jquery-bootstrap/js/app.js +++ b/examples/jquery-bootstrap/js/app.js @@ -12,7 +12,7 @@ var BootstrapButton = React.createClass({ }); var BootstrapModal = React.createClass({ - // The following two methods are the only places we need to + // The following four methods are the only places we need to // integrate with Bootstrap or jQuery! componentDidMount: function() { // When the component is added, turn it into a modal From 84a93fbacff57b8876f8bd3e5d84eb6f9f0c76dc Mon Sep 17 00:00:00 2001 From: Riley Tomasek Date: Thu, 18 Dec 2014 10:05:25 -0500 Subject: [PATCH 2/2] Improve comment in example code This should make it more clear that even though `$` is used in 4 methods, only 2 of them are crucial for integrating the modal into the components lifecycle methods and the other 2 are just helpers. --- examples/jquery-bootstrap/js/app.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/jquery-bootstrap/js/app.js b/examples/jquery-bootstrap/js/app.js index e6f97a4152..d64ba0e51a 100644 --- a/examples/jquery-bootstrap/js/app.js +++ b/examples/jquery-bootstrap/js/app.js @@ -12,8 +12,8 @@ var BootstrapButton = React.createClass({ }); var BootstrapModal = React.createClass({ - // The following four methods are the only places we need to - // integrate with Bootstrap or jQuery! + // The following two methods are the only places we need to + // integrate Bootstrap or jQuery with the components lifecycle methods. componentDidMount: function() { // When the component is added, turn it into a modal $(this.getDOMNode())