Closed pblyp closed 10 months ago
pblyp
commented
1 year ago Антон, спасибо за то, что потратили на меня время и очень быстро ответили! Впервые за очень долгое время перевел что-то техническое, очень нравится, когда быстро реагируют на твои предложения)
pblyp
commented
1 year ago Просмотрел вроде бы все предложения, спасибо! Посмотрите, пожалуйста, что я еще не закрыл из них.
ntishkevich
commented
1 year ago Просмотрел вроде бы все предложения, спасибо! Посмотрите, пожалуйста, что я еще не закрыл из них.
@pblyp пропустили 4 комментария:
pblyp
commented
11 months ago @ntishkevich четвертый коммент принял, по остальным у меня остались те же вопросы, посмотрите на них, пожалуйста, мои предложения висят в pending и без четкого согласия не хочу их сливать сам
Первый https://github.com/reactjs/ru.react.dev/pull/846#discussion_r1178473667 Второй https://github.com/reactjs/ru.react.dev/pull/846#discussion_r1181826705 Третий https://github.com/reactjs/ru.react.dev/pull/846#discussion_r1182225202
ntishkevich
commented
11 months ago по остальным у меня остались те же вопросы, посмотрите на них, пожалуйста, мои предложения висят в pending и без четкого согласия не хочу их сливать сам
Если они висят в пендинг, то вы их не опубликовали и кроме вас их никто не видит, поэтому ваших предложений не видел ни я, ни @gcor. Опубликуйте их и мы сможем их посмотреть.
pblyp
commented
11 months ago https://github.com/orgs/community/discussions/10369#discussion-3815011 Извините, крайне неочевидный момент был для меня, учитывая мой скромный опыт работы с GitHub
ntishkevich
commented
11 months ago Довольно странно, но на некоторые ваши комментарии не могу сделать реплай 😲
@pblyp Прочитал все ваши комментарии. Где мог оставить коммент, я оставил. Остальные комменты я осветил здесь.
Касаемо "оставить слово свойство", то я бы не оставлял, это может вызвать путаницу между свойством className у HTMLElement и атрибутом className у React-элемента.
В вашем предложении мне нравится первый предложенный вариант. Можем использовать его.
По поводу строчки 493. Согласен, что звучит сложно, но выкидывать предложения не стоит. Я могу предложить такой вариант перевода:
При нажатии на кнопку срабатывает обработчик `onClick`. Каждой кнопке в качестве значения пропа `onClick` задана функция `handleClick` из `MyApp`, поэтому выполняется соответствующий код. Этот код вызывает функцию `setCount(count + 1)`, увеличивая значение состояния `count`. Новое значение `count` передается каждой кнопке качестве пропа, поэтому они все отображают новое значение. Это называется "подъёмом состояния вверх". Поднимая состояние вверх, вы делаете его общим для всех компонентов.Избавимся от словосочетаний "таким образом" и используем предложенный вами вариант с "функцией".
pblyp
commented
11 months ago @ntishkevich надеюсь, я справился с замечаниями
ntishkevich
commented
11 months ago @gcor посмотри, плиз, как будет время.
github-actions[bot]
commented
10 months ago 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.
Если ваш пулреквест является исправлением бага, а не переводом, то сперва убедитесь, что проблема относится ТОЛЬКО к https://ru.reactjs.org, а не к https://reactjs.org. Если это не так, то пулреквест следует открыть в родительском репозитории.