2 ui/2 events/02 bubbling and capturing by dolgachio · Pull Request #199 · javascript-tutorial/ru.javascript.info

dolgachio · 2019-05-21T11:26:44Z

No description provided.

xcurveballx

еще есть комментарий на английском в файле https://github.com/javascript-tutorial/ru.javascript.info/blob/master/2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/script.js

2-ui/2-events/02-bubbling-and-capturing/article.md

javascript-translate-bot · 2019-05-23T14:03:33Z

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

2-ui/2-events/02-bubbling-and-capturing/article.md

javascript-translate-bot · 2019-05-31T12:41:12Z

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

dolgachio · 2019-06-02T11:24:18Z

/done

xcurveballx · 2019-06-02T11:35:12Z

произошел форс-пуш с такими результатами

329 changed files with 5,609 additions and 5,580 deletions.

Похоже, что что-то пошло не так. в этом PR был конфликт какой-то?

Как вариант, можно обновить форк, стянуть себе, создать заново ветку, черипикнуть нужные коммиты (с переводом и правками) из старой и запостить новый PR.

dolgachio · 2019-06-06T06:35:26Z

произошел форс-пуш с такими результатами

329 changed files with 5,609 additions and 5,580 deletions.

Похоже, что что-то пошло не так. в этом PR был конфликт какой-то?

Как вариант, можно обновить форк, стянуть себе, создать заново ветку, черипикнуть нужные коммиты (с переводом и правками) из старой и запостить новый PR.

@xcurveballx
Был конфликт, я делал ребейз и форс пуш.
Можете пояснить зачем нужно создавать новую ветку? У меня показывает в пулл реквесте только мои коммиты, ничего лишнего вроде не задел и не сломал.

dolgachio · 2019-06-07T19:11:47Z

@iliakan @xcurveballx @lex111 @dandgerson я поправил все комментарии, этот пулл реквест готов :)

2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/index.html

dolgachio · 2019-06-15T17:58:53Z

/done

iliakan

Спасибо за хороший перевод! Пожалуйста, внеси небольшие изменения.

iliakan · 2019-06-15T19:28:14Z

2-ui/2-events/02-bubbling-and-capturing/article.md

+Давайте начнём с примера.

-This handler is assigned to `<div>`, but also runs if you click any nested tag like `<em>` or `<code>`:
+Этот обработчик для `<div>` сработает, если вы кликните по любому из вложенных тегов `<em>` или `<code>`:


Этот обработчик для <div> сработает, если вы кликните по любому из вложенных тегов, будь то <em> или <code>:

Спасибо, готово

iliakan · 2019-06-15T19:30:56Z

2-ui/2-events/02-bubbling-and-capturing/article.md



-## Capturing
+## Погружение(перехват)


Погружение или перехват? Нам бы выбрать один термин =)

Второй можно дать в статье как "также используемый в других источниках"

Выбрал один, дописал фразу про другие источники, посмотри, пожалуйста, возможно можно улучшить.

iliakan · 2019-06-15T19:33:17Z

2-ui/2-events/02-bubbling-and-capturing/article.md

 ```

-There are two possible values of the `capture` option:
+Существуют два варианта значений аргумента `capture`:


аргумента -> опции

Спасибо, сделал

iliakan · 2019-06-15T19:33:37Z

2-ui/2-events/02-bubbling-and-capturing/article.md

- If it's `false` (default), then the handler is set on the bubbling phase.
- If it's `true`, then the handler is set on the capturing phase.
+- Если аргумент `false` (по умолчанию), то событие будет поймано при всплытии.
+- Если аргумент `true`, то событие будет перехвачено по дороге вниз при погружении.


лишнее "по дороге вниз"

Спасибо, убрал

iliakan · 2019-06-15T19:35:13Z

2-ui/2-events/02-bubbling-and-capturing/article.md

+Любой обработчик может остановить событие вызовом `event.stopPropagation()`, но делать это не рекомендуется, так как в дальнейшем это событие может понадобиться, иногда для самых неожиданных вещей.

-The capturing phase is used very rarely, usually we handle events on bubbling. And there's a logic behind that.
+В современной разработке стадия погружения используется очень редко, обычно события обрабатываются во время всплытия. И в этом есть скрытый смысл.


скрытый смысл -> логика

Спасибо, готово

javascript-translate-bot · 2019-06-15T19:36:13Z

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

… translation

dolgachio · 2019-06-16T10:28:17Z

/done

iliakan · 2019-06-16T13:42:54Z

2-ui/2-events/02-bubbling-and-capturing/article.md

+## Погружение

-Существует ещё одна фаза из жизненного цикла события -- "погружение". Она очень редко используется в реальном коде, однако тоже может быть полезной.
+Существует ещё одна фаза из жизненного цикла события -- "погружение", в других источниках её могут называть -- "перехват", но мы будем использовать первый вариант. Она очень редко используется в реальном коде, однако тоже может быть полезной.


Существует ещё одна фаза из жизненного цикла события -- "погружение" (иногда её называют "перехват"). Она очень редко используется в реальном коде, однако тоже может быть полезной.

Спасибо, сделал

javascript-translate-bot · 2019-06-16T16:04:23Z

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

dolgachio · 2019-06-16T16:19:54Z

/done

iliakan · 2019-06-16T19:42:32Z

🚀

javascript-translate-bot added the review needed Review needed, please approve or request changes label May 21, 2019

javascript-translate-bot requested a review from a team May 21, 2019 11:26

iliakan mentioned this pull request May 21, 2019

Russian Translation Progress #60

Closed

xcurveballx reviewed May 23, 2019

View reviewed changes

xcurveballx requested changes May 23, 2019

View reviewed changes

javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels May 23, 2019

xcurveballx requested changes May 31, 2019

View reviewed changes

2-ui/2-events/02-bubbling-and-capturing/article.md Outdated Show resolved Hide resolved

dolgachio added 10 commits June 2, 2019 14:21

02-bubbling-and-capturing: Translate the first part

a807951

02-bubbling-and-capturing: Translate all about bubbling

ff58a1d

02-bubbling-and-capturing: Translate capture phase paragraph

1e10349

02-bubbling-and-capturing: translate summary

a71ed74

02-bubbling-and-capturing: Replace e with yo

347167b

02-bubbling-and-capturing: Change translation for event.target

864c09c

02-bubbling-and-capturing: Translate bubble-target example template

cdf666a

02-bubbling-and-capturing: replace e with yo, add word

e5f7c1e

02-bubbling-and-capturing: Fix typos, punctuation and spelling

0d78146

02-bubbling-and-capturing: Remove redundant line

5c0ea17

dolgachio force-pushed the 2-ui/2-events/02-bubbling-and-capturing branch from b508a2b to 5c0ea17 Compare June 2, 2019 11:23

javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jun 2, 2019

javascript-translate-bot requested a review from a team June 2, 2019 11:24

javascript-translate-bot removed the review needed Review needed, please approve or request changes label Jun 7, 2019

xcurveballx requested changes Jun 7, 2019

View reviewed changes

2-ui/2-events/02-bubbling-and-capturing/bubble-target.view/index.html Outdated Show resolved Hide resolved

02-bubbling-and-capturing: Translate comment in example, fix punctuation

cc5574e

javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jun 15, 2019

javascript-translate-bot requested a review from a team June 15, 2019 17:58

javascript-translate-bot removed the review needed Review needed, please approve or request changes label Jun 15, 2019

xcurveballx approved these changes Jun 15, 2019

View reviewed changes

javascript-translate-bot added the needs +1 One more review needed label Jun 15, 2019

iliakan requested changes Jun 15, 2019

View reviewed changes

javascript-translate-bot added the changes requested Waiting for changes and /done from PR author label Jun 15, 2019

dolgachio added 3 commits June 16, 2019 13:18

02-bubbling-and-capturing: Improve translation

e3de6c2

02-bubbling-and-capturing: Stick with unified version of 'capturing'…

4020eaa

… translation

02-bubbling-and-capturing: Minor translation improvements

46790be

javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jun 16, 2019

javascript-translate-bot requested a review from a team June 16, 2019 10:28

iliakan requested changes Jun 16, 2019

View reviewed changes

javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Jun 16, 2019

02-bubbling-and-capturing: Change phrase in capturing paragraph

767ac48

javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jun 16, 2019

javascript-translate-bot requested a review from a team June 16, 2019 16:19

javascript-translate-bot merged commit 6cb451d into javascript-tutorial:master Jun 16, 2019

dolgachio deleted the 2-ui/2-events/02-bubbling-and-capturing branch June 16, 2019 19:44

Conversation

dolgachio commented May 21, 2019

Uh oh!

xcurveballx left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commented May 23, 2019

Uh oh!

Uh oh!

javascript-translate-bot commented May 31, 2019

Uh oh!

dolgachio commented Jun 2, 2019

Uh oh!

xcurveballx commented Jun 2, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dolgachio commented Jun 6, 2019

Uh oh!

dolgachio commented Jun 7, 2019

Uh oh!

Uh oh!

dolgachio commented Jun 15, 2019

Uh oh!

iliakan left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

javascript-translate-bot commented Jun 15, 2019

Uh oh!

dolgachio commented Jun 16, 2019

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

javascript-translate-bot commented Jun 16, 2019

Uh oh!

dolgachio commented Jun 16, 2019

Uh oh!

iliakan commented Jun 16, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

xcurveballx commented Jun 2, 2019 •

edited

Loading