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

Translate "Building Your Own Hooks" #150

Merged
merged 4 commits into from
Feb 28, 2019

Conversation

tesler
Copy link
Contributor

@tesler tesler commented Feb 15, 2019

Не осилил нормально перевести предложение
ac692e5#diff-76bf3361c22ba42425c012b2b1f04f3dL144

Custom Hooks are a convention that naturally follows from the design of Hooks, rather than a React feature.

Название раздела и ссылок на другие выбирал исходя из обсуждений и пулов.

@tesler tesler mentioned this pull request Feb 15, 2019
88 tasks
@gcor
Copy link
Collaborator

gcor commented Feb 15, 2019

Локальное состояние → внутреннее

@tesler
Copy link
Contributor Author

tesler commented Feb 15, 2019

@gcor спасибо за замечание, поправил

@lex111 lex111 added the needs review A pull request ready to be reviewed label Feb 18, 2019
@lex111
Copy link
Member

lex111 commented Feb 19, 2019

Custom Hooks are a convention that naturally follows from the design of Hooks, rather than a React feature.

Пользовательские хуки — это скорее соглашение, соответсвующее [естественным образом вытекает из] дизайну [задумке] хуков, нежели чем [а не] возможность [функционал] самого React.

Т.е. например, пользовательские хуки можно назвать blah, а не useBlah, и это сработает (если плагин для линтера, который проверяет хуки, т.е. если он пользовательский хук назван useBlah, то он проверяется в соответствие с определёнными правилами. Либо я не так понимаю, но примерно как-то так.

@tesler
Copy link
Contributor Author

tesler commented Feb 19, 2019

Т.е. например, пользовательские хуки можно назвать blah, а не useBlah, и это сработает (если плагин для линтера, который проверяет хуки, т.е. если он пользовательский хук назван useBlah, то он проверяется в соответствие с определёнными правилами. Либо я не так понимаю, но примерно как-то так.

Это относится скорее к следующему пункту
5895413#diff-76bf3361c22ba42425c012b2b1f04f3dR146

Должен ли я писать “use” в начале названия хука? Очень желательно. Это важное условие, без которого мы не сможем автоматически проверять нарушения правил хуков, потому что не определим, содержит ли определённая функция вызовы хуков внутри.

Но твой вариант перевода лучше. Поправлю

@another-guy another-guy self-assigned this Feb 25, 2019
Copy link
Collaborator

@another-guy another-guy left a comment

Choose a reason for hiding this comment

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

Убрать бы небольшие стилистические шероховатости. В целом, читается легко! Спасибо @tesler

content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
content/docs/hooks-custom.md Outdated Show resolved Hide resolved
@another-guy another-guy removed their assignment Feb 25, 2019
@lex111
Copy link
Member

lex111 commented Feb 28, 2019

@another-guy внёс твои правки + добавил собственные. Посмотри ещё раз пулреквест, и если всё ок, то сливай.

P.S. билд не проходит, потому что мастер старый, так что можно смело сливать, если что.

@lex111 lex111 self-assigned this Feb 28, 2019
@lex111 lex111 assigned another-guy and unassigned lex111 Feb 28, 2019
@netlify
Copy link

netlify bot commented Feb 28, 2019

Deploy preview for ru-reactjs failed.

Built with commit 9f190cc

https://app.netlify.com/sites/ru-reactjs/deploys/5c772cc6c1a56a00079923fb

@another-guy another-guy merged commit 470c690 into reactjs:master Feb 28, 2019
@another-guy another-guy removed their assignment Mar 2, 2019
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.

4 participants