Address some possible issues in 1.2.11 #112

vsemozhetbyt · 2020-09-20T11:36:33Z

UkrainianCitizen

@tarasyyyk КРИТИЧНО: який переклад для truthy/falsy? В глосарії немає. Потрібно якось розділяти true/false та truthy/falsy в перекладі, це не одне і те саме. Я рев'ювив ПР і усюди бачив, що truthy/falsy перекладається як true/false.

UkrainianCitizen · 2020-10-02T14:34:16Z

1-js/02-first-steps/11-logical-operators/article.md

 alert( Boolean(null) ); // false
 ```

-Пріоритет НЕ `!` є навищим з усіх логічних операторів, тому він завжди виконується першим, перед `&&` або `||`.
+Пріоритет НЕ `!` є найвищим з усіх логічних операторів, тому він завжди виконується першим, перед `&&` або `||`.


краще "серед", серед усіх

згоден щодо «серед усіх». Звучить краще

1-js/02-first-steps/11-logical-operators/article.md

1-js/02-first-steps/11-logical-operators/4-alert-and/task.md

UkrainianCitizen · 2020-10-02T14:48:58Z

1-js/02-first-steps/11-logical-operators/article.md

-````smart header="Пріоритет І `&&` є більш високим за АБО `||`"
-Пріоритет оператора І `&&` є більш високим за АБО `||`.
+````smart header="Пріоритет І `&&` вище за АБО `||`"
+Пріоритет оператора І `&&` вище за АБО `||`.


Я б переклав "є вищим", якщо залишати так, то "вищий", не "вище"

«Пріоритет оператора ... вищий за <оператор> АБО».
Мабуть так буде краще за вище.
Також можна перефразувати для кращого розуміння: «Оператор І && має вищий пріоритет за АБО ||».

UkrainianCitizen · 2020-10-02T14:51:01Z

1-js/02-first-steps/11-logical-operators/article.md

@@ -84,7 +84,7 @@ result = value1 || value2 || value3;

 Значення повертається у первісному вигляді без конвертації.

-Іншими словами, ланцюжок з АБО `"||"` повертає перше правдиве значення або останнє, якщо не знайдено правдивого значення.
+Іншими словами, ланцюжок з АБО `||` повертає перше правдиве значення або останнє, якщо правдивого значення не знайдено.


Може краще вживати істинне замість правдиве? Я завжди перекладав як істинне. Що думаєш @tarasyyyk ? Може в глосарій додати.

@tarasyyyk КРИТИЧНО: який переклад для truthy/falsy? В глосарії немає. Потрібно якось розділяти true/false та truthy/falsy в перекладі, це не одне і те саме. Я рев'ювив ПР і усюди бачив, що truthy/falsy перекладається як true/false.

UkrainianCitizen · 2020-10-02T15:07:05Z

1-js/02-first-steps/11-logical-operators/article.md


-    Деколи люди використовують таку конструкцію щоб виконати команди, якщо умова в лівій частині невірна.
+    Деколи таку конструкцію використовують, щоб виконувати команди лише за умови неправдивості умови ліворуч від оператора.


Хибності. Чи тут falsy? Скоріше так, треба встановити переклад для truthy/falsy аби їх розрізняти.

UkrainianCitizen · 2020-10-02T15:22:07Z

1-js/02-first-steps/11-logical-operators/article.md

@@ -120,21 +120,21 @@ alert( undefined || null || 0 ); // 0 (усі не правдиві, повер
 2. **Обчислення короткого замикання.**

    Іншою особливістю оператора АБО `||` є так зване "обчислення короткого замикання".
-
+
    Це означає, що оператор `||` опрацьовує аргументи доти, доки не досягається перше правдиве значення, після чого це значення негайно повертається, без подальшого опрацювання решти аргументів.


@tarasyyyk КРИТИЧНО: який переклад для truthy/falsy? Тут усюдт вживається truthy/falsy. Потрібно якось розділяти true/false та truthy/falsy, в даному випадку, тут в оригіналі truthy, але переклад іде для "true", що є неправильним.

Написав свою думку в цьому коментарі.

UkrainianCitizen · 2020-10-02T15:35:58Z

1-js/02-first-steps/11-logical-operators/article.md


-    У наведеному нижче прикладі, виведеться лише друге повідомлення:
+    Важливість цієї поведінки стає очевидною, коли операнд не проста змінна, а вираз з побічною дією, як от визначення змінної або виклик функції.


Дослівний переклад.
~~коли операнд не проста змінна, а вираз з побічною дією, як от визначення змінної або виклик функції.~~ ->

Suggested change

Важливість цієї поведінки стає очевидною, коли операнд не проста змінна, а вираз з побічною дією, як от визначення змінної або виклик функції.

Важливість такої особливості стає очевидною, якщо операнд є не просто змінною, а виразом із побічним ефектом, як-от присвоєння змінної або виклик функції.

+1 питання до @tarasyyyk : side effect - побічна дія чи побічний ефект? Це треба теж в словник для консистентності. Я б переклав ефект поки що.

Я ще подивився в оригіналі, там іде feature, а не behavior, тобто іде мова про фічу(особливість) оператора - коротке замикання, а не про її поведінку - коротке замикання. Напевно, можна і те, і друге як синоніми вживати, але я був би схильним використати переклад для feature, тобто особливість.

Згоден щодо «побічного ефекту». «Побічна дія» більше для медицини підходить.

UPD: хоча, перечитавши ще декілька разів речення дійшов до висновку, що не так все однозначно — «побічна дія» теж хороший варіант в цьому контексті. Тому що «дія» тут виражається присвоєнням або викликом функції. Згідно з сайтом Горох, ефект краще використовувати в якості «сильне враження від чогось». Проте, знову-таки, ефект може означати також «результат, наслідок якихось дій».

Тут краще дивитися, ніж на Горосі, але тут до слова обміль на даний момент.

UkrainianCitizen · 2020-10-02T15:41:39Z

1-js/02-first-steps/11-logical-operators/article.md

@@ -273,17 +273,17 @@ alert( !0 ); // true
 Подвійний НЕ `!!` іноді використовується для перетворення значення на булевий тип:

 ```js run
-alert( !!"не пустий рядок" ); // true
+alert( !!"непустий рядок" ); // true


"не пустий" якраз правильно. Я був би схильний написати просто "рядок", те, що він не пустий само собою зрозуміло.

UkrainianCitizen · 2020-10-02T15:45:13Z

1-js/02-first-steps/11-logical-operators/article.md


 ```js run
-alert( Boolean("не пустий рядок") ); // true
+alert( Boolean("непустий рядок") ); // true


"не пустий" якраз правильно. Я був би схильний написати просто "рядок", те, що він не пустий само собою зрозуміло.

javascript-translate-bot · 2020-10-02T15:46:19Z

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

vsemozhetbyt · 2020-10-02T16:11:44Z

@UkrainianCitizen Дякую за виправлення. Будь ласка, якщо вам не важко, оформлюйте їх відразу як готові пропозиції, тоді @tarasyyyk зможе одразу застосувати їх я коміти.

UkrainianCitizen · 2020-10-02T16:16:15Z

Не зрозумів, що ви маєте на увазі під

готові пропозиції

vsemozhetbyt · 2020-10-02T16:28:21Z

Якщо ви натиснете кнопку, яку обведено червоним, в коментар будет вставлена пропозиція. Спочатку вона виглядає як повторення частини PR. Якщо ви зробите в ній виправлення, ваш коментар потім будет містити diff, який можна застосувати як коміт з виправленням цієї частини PR. Пропозицію может прийняти автор PR або будь-хто за правом запису в репозиторій.

UkrainianCitizen · 2020-10-02T16:34:42Z

Я зрозумів, так і думав, просто не бачив як його застосувати, хоча я контриб'ютор.

vsemozhetbyt · 2020-10-02T16:45:16Z

Я чомусь теж не бачу тепер інтерфейсу прийняття пропозицій. Мабуть, це згубний наслідок того, що я видалив свій форк репозиторію, коли не зміг більше бути корисним проєкту. Мені шкода, якщо це так. Сподіваюсь, зміни можна будет внести адміністратору при злитті PR. Якщо ні, тоді вони потребуватимуть нового PR. Вибачте(

vsemozhetbyt · 2020-10-02T20:17:57Z

@tarasyyyk Як ви порадите? Я не думав, що видалення форку унеможливить прийняття пропозицій, думав інтерфейсу GitHub буде достатньо, щоб внести можливі правки до PR. Я бачу два способи: або я закриваю всі свої реквести, роблю новий форк, копіюю туди файли з реквестів і відкриваю нові PR, де можна будет приймати поправки. Або хтось з колабораторів потім додасть поправки після злиття цих проблемних реквестів. Перепрошую за незручності. Можливо, є простіші способи. Зрештою, можна взагалі проігнорувати всі ці мої PR і закрити їх, все одно в них лише купа дріб’язку, рано чи пізно всі слушні зміни хтось і так внесе.

UkrainianCitizen · 2020-10-03T14:35:07Z

@vsemozhetbyt Зміни в ПР - відповідальність людини, котра його робить. Ви не мали видаляти форк in the first place, бо задача рев'ювера - лише рев'ювити. Робіть ПР із нового форку, а цей ПР закривайте і зареференсіть в новому ПР, бажано з усіми коментами. Так само зробіть із іншими ПР, які отримали рев'ю.

vsemozhetbyt · 2020-10-03T14:52:53Z

Я не збирався не робити змін у відповідь на ревью, просто думав, що інтерфейс GitHub-у буде працювати і без форку. Тепер запам’ятаю, що ні.

В моїх минулих PR @tarasyyyk сам робив всі зміни за допомогою пропозицій і приймав їх сам перед злиттям. Мабуть, це теж дало мені помилокову надію.

Гаразд. Поки закриваю наявні, трохи пізніше зроблю нові.

vsemozhetbyt · 2020-10-03T15:37:41Z

PR на заміну з застосованими виправленнями: #119

Обидва інших PR також перевідкрив, але вони ще не мали ревью.

Address some possible issues in 1.2.11

7ed1e48

javascript-translate-bot added the review needed label Sep 20, 2020

javascript-translate-bot requested a review from a team September 20, 2020 11:36

UkrainianCitizen suggested changes Oct 2, 2020

View reviewed changes

javascript-translate-bot removed the review needed label Oct 2, 2020

javascript-translate-bot added the changes requested label Oct 2, 2020

vsemozhetbyt closed this Oct 3, 2020

vsemozhetbyt mentioned this pull request Oct 3, 2020

Address some possible issues in translation of 1.2.11 #119

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Address some possible issues in 1.2.11 #112

Address some possible issues in 1.2.11 #112

vsemozhetbyt commented Sep 20, 2020

UkrainianCitizen left a comment

UkrainianCitizen Oct 2, 2020

tarasyyyk Oct 2, 2020

UkrainianCitizen Oct 2, 2020

tarasyyyk Oct 2, 2020

UkrainianCitizen Oct 2, 2020 •

edited

Loading

UkrainianCitizen Oct 2, 2020

UkrainianCitizen Oct 2, 2020

tarasyyyk Oct 2, 2020

UkrainianCitizen Oct 2, 2020 •

edited

Loading

UkrainianCitizen Oct 2, 2020 •

edited

Loading

tarasyyyk Oct 2, 2020 •

edited

Loading

UkrainianCitizen Oct 2, 2020

UkrainianCitizen Oct 2, 2020

UkrainianCitizen Oct 2, 2020

javascript-translate-bot commented Oct 2, 2020

vsemozhetbyt commented Oct 2, 2020

UkrainianCitizen commented Oct 2, 2020

vsemozhetbyt commented Oct 2, 2020 •

edited

Loading

UkrainianCitizen commented Oct 2, 2020

vsemozhetbyt commented Oct 2, 2020 •

edited

Loading

vsemozhetbyt commented Oct 2, 2020

UkrainianCitizen commented Oct 3, 2020 •

edited

Loading

vsemozhetbyt commented Oct 3, 2020

vsemozhetbyt commented Oct 3, 2020


		Деколи люди використовують таку конструкцію щоб виконати команди, якщо умова в лівій частині невірна.
		Деколи таку конструкцію використовують, щоб виконувати команди лише за умови неправдивості умови ліворуч від оператора.


		У наведеному нижче прикладі, виведеться лише друге повідомлення:
		Важливість цієї поведінки стає очевидною, коли операнд не проста змінна, а вираз з побічною дією, як от визначення змінної або виклик функції.

	Важливість цієї поведінки стає очевидною, коли операнд не проста змінна, а вираз з побічною дією, як от визначення змінної або виклик функції.
	Важливість такої особливості стає очевидною, якщо операнд є не просто змінною, а виразом із побічним ефектом, як-от присвоєння змінної або виклик функції.

Address some possible issues in 1.2.11 #112

Address some possible issues in 1.2.11 #112

Conversation

vsemozhetbyt commented Sep 20, 2020

UkrainianCitizen left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

UkrainianCitizen Oct 2, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

UkrainianCitizen Oct 2, 2020 • edited Loading

Choose a reason for hiding this comment

UkrainianCitizen Oct 2, 2020 • edited Loading

Choose a reason for hiding this comment

tarasyyyk Oct 2, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

javascript-translate-bot commented Oct 2, 2020

vsemozhetbyt commented Oct 2, 2020

UkrainianCitizen commented Oct 2, 2020

vsemozhetbyt commented Oct 2, 2020 • edited Loading

UkrainianCitizen commented Oct 2, 2020

vsemozhetbyt commented Oct 2, 2020 • edited Loading

vsemozhetbyt commented Oct 2, 2020

UkrainianCitizen commented Oct 3, 2020 • edited Loading

vsemozhetbyt commented Oct 3, 2020

vsemozhetbyt commented Oct 3, 2020

UkrainianCitizen Oct 2, 2020 •

edited

Loading

UkrainianCitizen Oct 2, 2020 •

edited

Loading

UkrainianCitizen Oct 2, 2020 •

edited

Loading

tarasyyyk Oct 2, 2020 •

edited

Loading

vsemozhetbyt commented Oct 2, 2020 •

edited

Loading

vsemozhetbyt commented Oct 2, 2020 •

edited

Loading

UkrainianCitizen commented Oct 3, 2020 •

edited

Loading