Skip to content
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

Rutranslation/quick start #846

Merged
merged 13 commits into from
Jul 5, 2023
Merged

Conversation

pblyp
Copy link
Contributor

@pblyp pblyp commented Apr 25, 2023

Если ваш пулреквест является исправлением бага, а не переводом, то сперва убедитесь, что проблема относится ТОЛЬКО к https://ru.reactjs.org, а не к https://reactjs.org. Если это не так, то пулреквест следует открыть в родительском репозитории.

@pblyp pblyp marked this pull request as ready for review April 25, 2023 19:55
@gcor gcor added the needs review A pull request ready to be reviewed label Apr 25, 2023
Copy link
Collaborator

@gcor gcor left a comment

Choose a reason for hiding this comment

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

Спасибо большое! 👍

Вот такие комментарии и предложения:

src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
Предложения от gcor, самостоятельно замеченные случаи неиспользования буквы `ё`

Co-authored-by: Anton Ahatov <[email protected]>
@pblyp
Copy link
Contributor Author

pblyp commented Apr 26, 2023

Антон, спасибо за то, что потратили на меня время и очень быстро ответили! Впервые за очень долгое время перевел что-то техническое, очень нравится, когда быстро реагируют на твои предложения)

Copy link
Collaborator

@gcor gcor left a comment

Choose a reason for hiding this comment

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

Еще немного правок после повторного прочтения :)

src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@ntishkevich ntishkevich left a comment

Choose a reason for hiding this comment

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

Спасибо за перевод. Вот еще несколько моментов, которые можно исправить.

src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
@pblyp
Copy link
Contributor Author

pblyp commented May 4, 2023

Просмотрел вроде бы все предложения, спасибо! Посмотрите, пожалуйста, что я еще не закрыл из них.

@pblyp pblyp requested a review from ntishkevich May 17, 2023 07:11
@ntishkevich
Copy link
Collaborator

ntishkevich commented May 18, 2023

Просмотрел вроде бы все предложения, спасибо! Посмотрите, пожалуйста, что я еще не закрыл из них.

@pblyp пропустили 4 комментария:

Co-authored-by: Nick Tishkevich <[email protected]>
@pblyp
Copy link
Contributor Author

pblyp commented Jun 14, 2023

@ntishkevich четвертый коммент принял, по остальным у меня остались те же вопросы, посмотрите на них, пожалуйста, мои предложения висят в pending и без четкого согласия не хочу их сливать сам

Первый #846 (comment)
Второй #846 (comment)
Третий #846 (comment)

@ntishkevich
Copy link
Collaborator

по остальным у меня остались те же вопросы, посмотрите на них, пожалуйста, мои предложения висят в pending и без четкого согласия не хочу их сливать сам

Если они висят в пендинг, то вы их не опубликовали и кроме вас их никто не видит, поэтому ваших предложений не видел ни я, ни @gcor. Опубликуйте их и мы сможем их посмотреть.

src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
src/content/learn/index.md Outdated Show resolved Hide resolved
@pblyp
Copy link
Contributor Author

pblyp commented Jun 14, 2023

https://github.com/orgs/community/discussions/10369#discussion-3815011
Извините, крайне неочевидный момент был для меня, учитывая мой скромный опыт работы с GitHub

  • еще какие-то из комментов стали outdated вдруг по непонятной причине

@ntishkevich
Copy link
Collaborator

Довольно странно, но на некоторые ваши комментарии не могу сделать реплай 😲

@pblyp Прочитал все ваши комментарии. Где мог оставить коммент, я оставил. Остальные комменты я осветил здесь.
Касаемо "оставить слово свойство", то я бы не оставлял, это может вызвать путаницу между свойством className у HTMLElement и атрибутом className у React-элемента.

В вашем предложении мне нравится первый предложенный вариант. Можем использовать его.

По поводу строчки 493. Согласен, что звучит сложно, но выкидывать предложения не стоит. Я могу предложить такой вариант перевода:

При нажатии на кнопку срабатывает обработчик `onClick`. Каждой кнопке в качестве значения пропа `onClick` задана функция `handleClick` из `MyApp`, поэтому выполняется соответствующий код. Этот код вызывает функцию `setCount(count + 1)`, увеличивая значение состояния `count`. Новое значение `count` передается каждой кнопке  качестве пропа, поэтому они все отображают новое значение. Это называется "подъёмом состояния вверх". Поднимая состояние вверх, вы делаете его общим для всех компонентов.

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

@pblyp
Copy link
Contributor Author

pblyp commented Jun 18, 2023

@ntishkevich надеюсь, я справился с замечаниями

Copy link
Collaborator

@ntishkevich ntishkevich left a comment

Choose a reason for hiding this comment

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

LGTM, но просьба поправить небольшое замечание - пропустили ё.

src/content/learn/index.md Outdated Show resolved Hide resolved
@ntishkevich
Copy link
Collaborator

@gcor посмотри, плиз, как будет время.

Co-authored-by: Nick Tishkevich <[email protected]>
@github-actions
Copy link

github-actions bot commented Jul 5, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

Three Pages Changed Size

The following pages changed size from the code in this PR compared to its base branch:

Page Size (compressed) First Load
/404 75.34 KB (🟢 -62 B) 178.49 KB
/500 75.33 KB (🟢 -62 B) 178.49 KB
/[[...markdownPath]] 76.75 KB (🟢 -62 B) 179.91 KB
Details

Only the gzipped size is provided here based on an expert tip.

First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If next/link is used, subsequent page loads would only need to download that page's bundle (the number in the "Size" column), since the global bundle has already been downloaded.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

Next to the size is how much the size has increased or decreased compared with the base branch of this PR. If this percentage has increased by 10% or more, there will be a red status indicator applied, indicating that special attention should be given to this.

@gcor gcor merged commit 38b4eef into reactjs:main Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review A pull request ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants