merge

iliakan · iliakan · commit 562bbd90b020 · 2019-06-10T10:38:04.000+03:00
diff --git a/1-js/02-first-steps/03-strict-mode/article.md b/1-js/02-first-steps/03-strict-mode/article.md
@@ -53,9 +53,17 @@ alert("some code");
 
 Иногда, когда `use strict` имеет значение, вы можете получить неправильные результаты.
 
-Даже если мы нажмём `key:Shift+Enter` для ввода нескольких строк и напишем вверху `use strict` -- это не сработает. Это связано с тем, как консоль выполняет код внутри.
+Можно использовать `key:Shift+Enter` для ввода нескольких строк и написать вверху `use strict`:
 
-Надёжный способ сделать так, чтобы `use strict` заработал, это ввести код в консоль следующим образом:
+```js
+'use strict'; <Shift+Enter для перехода на новую строку>
+//  ...ваш код...
+<Enter для запуска>
+```
+
+В большинстве браузеров, включая Chrome и Firefox, это работает.
+
+В старых браузерах консоль не учитывает такой `use strict`, там можно "оборачивать" код в функцию, вот так:
 
 ```js
 (function() {
diff --git a/1-js/06-advanced-functions/11-currying-partials/article.md b/1-js/06-advanced-functions/11-currying-partials/article.md
@@ -169,7 +169,7 @@ function curry(f) {
 
 Продвинутое каррирование позволяет вызывать функцию, как обычно, так и с частичным применением.
 
-Например, у нас есть функция логирования `log(date, importance, message)`, которая форматирует и выводит информацию. В реальных проектах у таких функций есть много других полезных возможностей, например, посылать логи по сети:
+Например, у нас есть функция логирования `log(date, importance, message)`, которая форматирует и выводит информацию. В реальных проектах у таких функций есть много других полезных возможностей, например, посылать логи по сети, здесь для простоты используем `alert`:
 
 ```js
 function log(date, importance, message) {
@@ -192,25 +192,26 @@ log(new Date(), "DEBUG", "some debug");
 ...Но также работает вариант с каррированием:
 
 ```js
+log(new Date(), "DEBUG", "some debug"); // log(a,b,c)
 log(new Date())("DEBUG")("some debug"); // log(a)(b)(c)
 ```
 
 Давайте сделаем удобную функцию для логов с текущим временем:
 
 ```js
 // todayLog будет частичным применением функции log с фиксированным первым аргументом
-let todayLog = log(new Date());
+let logNow = log(new Date());
 
 // используем её
-todayLog("INFO", "message"); // [HH:mm] INFO message
+logNow("INFO", "message"); // [HH:mm] INFO message
 ```
 
 А теперь удобная функция для логов отладки с текущим временем:
 
 ```js
-let todayDebug = todayLog("DEBUG");
+let debugNow = logNow("DEBUG");
 
-todayDebug("message"); // [HH:mm] DEBUG message
+debugNow("message"); // [HH:mm] DEBUG message
 ```
 
 Итак:
diff --git a/2-ui/1-document/08-styles-and-classes/article.md b/2-ui/1-document/08-styles-and-classes/article.md
@@ -66,10 +66,10 @@ So we can operate both on the full class string using `className` or on individu
 Methods of `classList`:
 
 - `elem.classList.add/remove("class")` -- adds/removes the class.
-- `elem.classList.toggle("class")` -- if the class exists, then removes it, otherwise adds it.
+- `elem.classList.toggle("class")` -- adds the class if it doesn't exist, otherwise removes it.
 - `elem.classList.contains("class")` -- returns `true/false`, checks for the given class.
 
-Besides that, `classList` is iterable, so we can list all classes like this:
+Besides, `classList` is iterable, so we can list all classes with `for..of`, like this:
 
 ```html run
 <body class="main page">
@@ -147,7 +147,7 @@ To set the full style as a string, there's a special property `style.cssText`:
 </script>
 ```
 
-We rarely use it, because such assignment removes all existing styles: it does not add, but replaces them. May occasionally delete something needed. But still can be done for new elements when we know we won't delete something important.
+This property is rarely used, because such assignment removes all existing styles: it does not add, but replaces them. May occasionally delete something needed. But we can safely use it for new elements, when we know we won't delete an existing style.
 
 The same can be accomplished by setting an attribute: `div.setAttribute('style', 'color: red...')`.
 ````
diff --git a/2-ui/5-loading/02-script-async-defer/article.md b/2-ui/5-loading/02-script-async-defer/article.md
@@ -188,11 +188,13 @@ loadScript("/article/script-async-defer/small.js");
 | `defer` | *Порядок документа* (как расположены в документе). |  Выполняется после того, как документ загружен и обработан  (ждёт), непосредственно перед `DOMContentLoaded`. |
 
 ```warn header="Страница без скриптов должна быть рабочей"
-Пожалуйста, помните, что когда вы используете `defer`, страница видна до того, как скрипт загрузится и подключит все графические компоненты.
+Пожалуйста, помните, что когда вы используете `defer`, страница видна до того, как скрипт загрузится.
 
-Поэтому кнопки должны быть отключены с помощью CSS или другим образом. 
+Пользователь может знакомиться с содержимым страницы, читать её, но графические компоненты пока отключены.
 
-На практике `defer` используется для скриптов, которым требуется доступ ко всему DOM и/или важен их относительный порядок выполнения.
+Поэтому обязательно должна быть индикация загрузки, нерабочие кнопки - отключены с помощью CSS или другим образом. Чтобы пользователь явно видел, что уже готово, а что пока нет.
+```
 
+На практике `defer` используется для скриптов, которым требуется доступ ко всему DOM и/или важен их относительный порядок выполнения.
 
-А `async` используется для независимых скриптов, например счётчиков и рекламы, относительный порядок выполнения которых не играет роли.
+А `async` хорош для независимых скриптов, например счётчиков и рекламы, относительный порядок выполнения которых не играет роли.
diff --git a/6-data-storage/03-indexeddb/article.md b/6-data-storage/03-indexeddb/article.md
@@ -154,8 +154,8 @@ let openRequest = indexedDB.open("db", 1);
 // создаём хранилище объектов для books, если ешё не существует
 openRequest.onupgradeneeded = function() {
   let db = openRequest.result;
-  if (!db.objectStoreNames.contains('books')) {
-    db.createObjectStore('books', {keyPath: 'id'});
+  if (!db.objectStoreNames.contains('books')) { // if there's no "books" store
+    db.createObjectStore('books', {keyPath: 'id'}); // create it
   }
 };
 ```
@@ -345,7 +345,7 @@ request.onerror = function(event) {
   if (request.error.name == "ConstraintError") {
     console.log("Книга с таким id уже существует"); // обрабатываем ошибку
     event.preventDefault(); // предотвращаем отмену транзакции
-    ...можно попробовать использовать другой ключ...
+    // ...можно попробовать использовать другой ключ...
   } else {
     // неизвестная ошибка
     // транзакция будет отменена
@@ -523,7 +523,7 @@ let request = priceIndex.getAll(IDBKeyRange.upperBound(5));
 
 ## Удаление из хранилища
 
-Метод `delete` удаляет значения по запросу, формат вызова такой же как в `getAll`.
+Метод `delete` удаляет значения по запросу, формат вызова такой же как в `getAll`:
 
 - **`delete(query)`** -- производит удаление соответствующих запросу значений.
 
@@ -671,7 +671,7 @@ try {
 
 ### Обработка ошибок
 
-Если мы не перехватим ошибку, то она "провалится" дальше, до ближайшего внешнего `try..catch`.
+Если мы не перехватим ошибку, то она "вывалится" наружу, вверх по стеку вызовов, до ближайшего внешнего `try..catch`.
 
 Необработанная ошибка становится событием "unhandled promise rejection" в объекте `window`.
 
diff --git a/9-regular-expressions/14-regexp-lookahead-lookbehind/article.md b/9-regular-expressions/14-regexp-lookahead-lookbehind/article.md
@@ -60,9 +60,9 @@ alert( str.match(/(?<!\$)\d+/) ); // 2 (skipped the price)
 
 Generally, what's inside the lookaround (a common name for both lookahead and lookbehind) parentheses does not become a part of the match.
 
-E.g. in the pattern `pattern:\d+(?!€)`, the `pattern:€` sign doesn't get captured as a part of the match.
+E.g. in the pattern `pattern:\d+(?=€)`, the `pattern:€` sign doesn't get captured as a part of the match. That's natural: we look for a number `pattern:\d+`, while `pattern:(?=€)` is just a test that it should be followed by `subject:€`.
 
-But if we want to capture the whole lookaround expression or a part of it, that's possible. Just need to wrap that into additional parentheses.
+But in some situations we might want to capture the lookaround expression as well, or a part of it. That's possible. Just  wrap that into additional parentheses.
 
 For instance, here the currency `pattern:(€|kr)` is captured, along with the amount:
 
@@ -89,9 +89,13 @@ Usually parentheses are numbered left-to-right, but lookbehind is an exception,
 
 ## Summary
 
-Lookahead and lookbehind (commonly referred to as "lookaround") are useful for simple regular expressions, when we'd like not to take something into the match depending on the context before/after it.
+Lookahead and lookbehind (commonly referred to as "lookaround") are useful when we'd like to take something into the match depending on the context before/after it.
 
-Sometimes we can do the same manually, that is: match all and filter by context in the loop. Remember, `str.matchAll` and `reg.exec` return matches with `.index` property, so we know where exactly in the text it is. But generally regular expressions can do it better.
+For simple regexps we can do the similar thing manually. That is: match everything, in any context, and then filter by context in the loop.
+
+Remember, `str.matchAll` and `reg.exec` return matches with `.index` property, so we know where exactly in the text it is, and can check the context.
+
+But generally regular expressions are more convenient.
 
 Lookaround types:
 
@@ -102,4 +106,4 @@ Lookaround types:
 | `pattern:(?<=y)x` |  Positive lookbehind | `x` if after `y` |
 | `pattern:(?<!y)x` | Negative lookbehind | `x` if not after `y` |
 
-Lookahead can also used to disable backtracking. Why that may be needed -- see in the next chapter.
+Lookahead can also used to disable backtracking. Why that may be needed and other details  -- see in the next chapter.
