From e37d4ac498a8e6af2ecf072ba08edd4f27dc2129 Mon Sep 17 00:00:00 2001
From: Ilya Kantor <iliakan@gmail.com>
Date: Wed, 14 Jun 2017 10:35:59 +0300
Subject: [PATCH] fix

---
 1-js/02-first-steps/01-hello-world/article.md |  4 ++--
 .../09-alert-prompt-confirm/article.md        |  4 +---
 .../01-property-descriptors/article.md        | 21 ++++++++++++-------
 .../bubble-target.view/script.js              |  1 -
 .../1-animate-logo-css/task.md                |  4 ++--
 3-animation/2-css-animations/article.md       | 17 ++++++++-------
 6 files changed, 28 insertions(+), 23 deletions(-)

diff --git a/1-js/02-first-steps/01-hello-world/article.md b/1-js/02-first-steps/01-hello-world/article.md
index 3307f1a6..9d12ae1a 100644
--- a/1-js/02-first-steps/01-hello-world/article.md
+++ b/1-js/02-first-steps/01-hello-world/article.md
@@ -63,7 +63,7 @@ Comments before and after scripts.
     //--></script>
     ```
 
-    These comments were supposed to hide the code from an old browser that didn't know about a `<script>` tag. But all browsers born in the past 15+ years don't have any issues. It is only mentioned here, because it serves as a sign. If you see that somewhere -- that code is probably really old and not worth looking into.
+    These comments were supposed to hide the code from an old browser that didn't know about a `<script>` tag. But all browsers born in the past 15+ years don't have any issues. We mention it here, because such comments serve as a sign. If you see that somewhere -- that code is probably really old and not worth looking into.
 
 
 ## External scripts
@@ -105,7 +105,7 @@ That saves traffic and makes pages faster.
 ```
 
 ````warn header="If `src` is set, the script content is ignored."
-A single `<script>` tag may not have both, `src` attribute and the code inside.
+A single `<script>` tag may not have both `src` attribute and the code inside.
 
 This won't work:
 
diff --git a/1-js/02-first-steps/09-alert-prompt-confirm/article.md b/1-js/02-first-steps/09-alert-prompt-confirm/article.md
index fd449b79..807326ab 100644
--- a/1-js/02-first-steps/09-alert-prompt-confirm/article.md
+++ b/1-js/02-first-steps/09-alert-prompt-confirm/article.md
@@ -2,9 +2,7 @@
 
 This part of the tutorial aims to cover JavaScript "as is", without environment-specific tweaks.
 
-But still we use a browser as the demo environment. So we should know at least few user-interface functions.
-
-In this chapter we'll get familiar with the browser-specific functions `alert`, `prompt` and `confirm`.
+But still we use a browser as the demo environment. So we should know at least few user-interface functions. In this chapter we'll get familiar with the browser functions `alert`, `prompt` and `confirm`.
 
 [cut]
 
diff --git a/1-js/07-object-oriented-programming/01-property-descriptors/article.md b/1-js/07-object-oriented-programming/01-property-descriptors/article.md
index f9830da7..bda88d41 100644
--- a/1-js/07-object-oriented-programming/01-property-descriptors/article.md
+++ b/1-js/07-object-oriented-programming/01-property-descriptors/article.md
@@ -145,9 +145,7 @@ user.name = "Alice"; // Error
 
 Now let's add a custom `toString` to `user`.
 
-Normally, a built-in `toString` for objects is non-enumerable, it does not show up in `for..in`. But if we add `toString` of our own, then by default it shows up in `for..in`.
-
-...But if we don't like it, then we can set `enumerable:false`. Then it won't appear in `for..in` loop, just like the built-in one:
+Normally, a built-in `toString` for objects is non-enumerable, it does not show up in `for..in`. But if we add `toString` of our own, then by default it shows up in `for..in`, like this:
 
 ```js run
 let user = {
@@ -159,6 +157,17 @@ let user = {
 
 // By default, both our properties are listed:
 for(let key in user) alert(key); // name, toString
+```
+
+If we don't like it, then we can set `enumerable:false`. Then it won't appear in `for..in` loop, just like the built-in one:
+
+```js run
+let user = {
+  name: "John",
+  toString() {
+    return this.name;
+  }
+};
 
 Object.defineProperty(user, "toString", {
 *!*
@@ -167,7 +176,7 @@ Object.defineProperty(user, "toString", {
 });
 
 *!*
-// Now toString disappears:
+// Now our toString disappears:
 */!*
 for(let key in user) alert(key); // name
 ```
@@ -178,11 +187,9 @@ Non-enumerable properties are also excluded from `Object.keys`:
 alert(Object.keys(user)); // name
 ```
 
-
-
 ## Non-configurable
 
-The non-configurable flag (`configurable:false`) is usually set for built-in objects and properties.
+The non-configurable flag (`configurable:false`) is sometimes preset for built-in objects and properties.
 
 A non-configurable property can not be deleted or altered with `defineProperty`.
 
diff --git a/2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/script.js b/2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/script.js
index 35997b04..d3959a68 100644
--- a/2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/script.js
+++ b/2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/script.js
@@ -1,4 +1,3 @@
-let form = document.querySelector('form');
 
 form.onclick = function(event) {
   event.target.style.backgroundColor = 'yellow';
diff --git a/3-animation/2-css-animations/1-animate-logo-css/task.md b/3-animation/2-css-animations/1-animate-logo-css/task.md
index ee08cbbb..ed10d4ac 100644
--- a/3-animation/2-css-animations/1-animate-logo-css/task.md
+++ b/3-animation/2-css-animations/1-animate-logo-css/task.md
@@ -8,7 +8,7 @@ Show the animation like on the picture below (click the plane):
 
 [iframe src="solution" height=300]
 
-- The picture grows on click from 40x24px to 400x240px.
+- The picture grows on click from `40x24px` to `400x240px` (10 times larger).
 - The animation takes 3 seconds.
 - At the end output: "Done!".
-- During the animation process, there may be more clicks. They shouldn't "break" anything.
+- During the animation process, there may be more clicks on the plane. They shouldn't "break" anything.
diff --git a/3-animation/2-css-animations/article.md b/3-animation/2-css-animations/article.md
index 5c2b9925..75501c11 100644
--- a/3-animation/2-css-animations/article.md
+++ b/3-animation/2-css-animations/article.md
@@ -399,24 +399,25 @@ Here's an example with explanations:
 </style>
 ```
 
-There are many articles about `@keyframes` and a [detailed specification](https://drafts.csswg.org/css-animations/), so we won't go into more details here.
+There are many articles about `@keyframes` and a [detailed specification](https://drafts.csswg.org/css-animations/).
 
-Probably you won't need it often, unless everything is in the constant move on your sites.
+Probably you won't need `@keyframes` often, unless everything is in the constant move on your sites.
 
 ## Summary
 
-CSS animations allow to smoothly (or not) animate changes to one or multiple CSS properties.
+CSS animations allow to smoothly (or not) animate changes of one or multiple CSS properties.
 
 They are good for most animation tasks. We're also able to use JavaScript for animations, the next chapter is devoted to that.
 
 Limitations of CSS animations compared to JavaScript animations:
 
-```compare
-- JavaScript animations are more flexible. They can implement any animation logic, like "explosion" of an element. We can create new elements in JavaScript for purposes of animation.
-+ Simple things done simple.
-+ Fast and easy for CPU.
+```compare plus="CSS animations" minus="JavaScript animations"
++ Simple things done simply.
++ Fast and lightweight for CPU.
+- JavaScript animations are flexible. They can implement any animation logic, like an "explosion" of an element.
+- Not just property changes. We can create new elements in JavaScript for purposes of animation.
 ```
 
-The vast majority of animations uses the described CSS properties. And maybe the `transitionend` event to bind some JavaScript after it.
+The majority of animations can be implemented using CSS as described in this chapter. And  `transitionend` event allows to run JavaScript after the animation, so it integrates fine with the code.
 
 But in the next chapter we'll do some JavaScript animations to cover more complex cases.