Skip to content

Translate 1.2.12. Nullish coalescing operator#1118

Merged
iliakan merged 3 commits intomasterfrom
lex111/nullish-operator
Oct 1, 2020
Merged

Translate 1.2.12. Nullish coalescing operator#1118
iliakan merged 3 commits intomasterfrom
lex111/nullish-operator

Conversation

@lex111
Copy link
Member

@lex111 lex111 commented Oct 1, 2020

No description provided.

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Oct 1, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team October 1, 2020 13:16
@iliakan
Copy link
Member

iliakan commented Oct 1, 2020

Может, есть какая-то альтернатива для перевода "defined"?

А то фразы получаются типа "- a, если a — определённое значение," если не знать, о чем речь, можно не понять.

Мб посмотреть существующие переводы.. @lex111

@lex111
Copy link
Member Author

lex111 commented Oct 1, 2020

"Существующее" значение? То же, наверное, не очень. Можно начало статьи изменить:

В этой статье мы считаем выражение "определённым", если его значение отличается от null или undefined.

То есть мы раскрываем суть defined более явно:

В этой статье мы называем выражение "определённым", если его значение отличается от null или undefined.

@iliakan
Copy link
Member

iliakan commented Oct 1, 2020

Может быть так?

В этой статье мы будем говорить, что значение выражения "определено", если его значение отличается от null или undefined.

И далее перефразировать так, чтобы не было странного словосочетания "определенное выражение".

@lex111
Copy link
Member Author

lex111 commented Oct 1, 2020

Допустим, но все равно нужно как-то одним словом (прилагательным) его перевести, например здесь:

?? returns the first defined value.

Тут курсив используется. Может быть, действительное или заданное, или вообще установленное?

Copy link
Member

@iliakan iliakan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Внеси изменения плиз

@javascript-translate-bot 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 Oct 1, 2020
@javascript-translate-bot
Copy link
Contributor

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

@lex111
Copy link
Member Author

lex111 commented Oct 1, 2020

@iliakan спасибо, внес поправки. /done

@javascript-translate-bot 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 Oct 1, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team October 1, 2020 16:26
@iliakan
Copy link
Member

iliakan commented Oct 1, 2020

Спасибо! 🚀

@iliakan iliakan merged commit d5e3050 into master Oct 1, 2020
@iliakan iliakan deleted the lex111/nullish-operator branch October 1, 2020 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants