-
Notifications
You must be signed in to change notification settings - Fork 385
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Обновить tutorial-tic-tac-toe.md #952
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Спасибо за перевод! Оставил несколько комментариев
|
||
You can see what it will look like when you're finished here: | ||
Вы можете увидеть, как это будет выглядеть, когда вы закончите здесь: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Вы можете увидеть, как это будет выглядеть, когда вы закончите здесь: | |
Вы можете увидеть ожидаемый результат здесь: |
предлагаю упростить
@@ -57,9 +57,9 @@ function Board({ xIsNext, squares, onPlay }) { | |||
const winner = calculateWinner(squares); | |||
let status; | |||
if (winner) { | |||
status = 'Winner: ' + winner; | |||
status = 'Выйграл: ' + winner; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
status = 'Выйграл: ' + winner; | |
status = 'Победитель: ' + winner; |
1. Unzip the archive, then open a terminal and `cd` to the directory you unzipped | ||
1. Скачайте [Node.js](https://nodejs.org/en/) | ||
1. На вкладке CodeSandbox, которую вы открыли ранее, нажмите кнопку в левом верхнем углу, чтобы открыть меню, а затем выберите Скачать Sandbox в этом меню, чтобы загрузить архив файлов локально. | ||
1. Разархивируйте архив, затем откройте терминал и перейдите в каталог, который вы разархивировали. | ||
1. Install the dependencies with `npm install` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
две строчки остались не переведены
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
я не успел
1. Install [Node.js](https://nodejs.org/en/) | ||
1. In the CodeSandbox tab you opened earlier, press the top-left corner button to open the menu, and then choose **Download Sandbox** in that menu to download an archive of the files locally | ||
1. Unzip the archive, then open a terminal and `cd` to the directory you unzipped | ||
1. Скачайте [Node.js](https://nodejs.org/en/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
надо согласовать вводное предложение и пункты.
сейчас получается "Для этого вам необходимо скачайте Node.js"
@titovmx А как а как продолжать дальше переводить по новой все с начала? или ждать когда вы добавите перевод на сайт |
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Co-authored-by: Maxim Titov <[email protected]>
Я дополнил перевод tutorial-tic-tac-toe и буду переводить дальше...
Обновить tutorial-tic-tac-toe.md
Дополнил перевод крестики-нолики
@SmaylikYoutuber привет! Обычно процесс такой:
Если я правильно понял, сейчас вы правки внесли в новой ветке, но перевод ещё не закончен? Если так, то давайте закроем этот ПР и продолжим перевод там. Пока перевод не закончен, вы можете пометить пулл реквест как Черновик (Draft), чтобы статус был прозрачен для тех, кто будет проводить ревью. |
@titovmx а можно по подробнее я сильно не разбираюсь в GitHub, я тут как новичок пришел просто помочь перевести, может какое-то видео подскажешь как продолжить перевод, а то я вообще не понял что было в верхней строчке. (Извини что не до конца понял и прошу написать по подробнее, что из за это-го я мучаю тебя, если ответишь буду рад) |
вот может ссылку кинуть на новый перевод #953 |
Если ваш пулреквест является исправлением бага, а не переводом, то сперва убедитесь, что проблема относится ТОЛЬКО к https://ru.reactjs.org, а не к https://reactjs.org. Если это не так, то пулреквест следует открыть в родительском репозитории.