Skip to content

Translate "Components and Props" into Russian - #34

Merged
gaearon merged 6 commits into
masterfrom
comps-props
Feb 10, 2019
Merged

Translate "Components and Props" into Russian#34
gaearon merged 6 commits into
masterfrom
comps-props

Conversation

@gaearon

@gaearongaearon commented Feb 9, 2019

Copy link
Copy Markdown
Member

No description provided.

@gaearongaearon changed the title Translate "Components and Props" into RussiaTranslate "Components and Props" into RussianFeb 9, 2019
@gaearon
gaearon requested a review from lex111February 9, 2019 20:36
@tesseralistesseralis mentioned this pull request Feb 9, 2019
Closed
88 tasks
@gaearongaearon added the needs review A pull request ready to be reviewed label Feb 9, 2019
lex111
lex111 suggested changes Feb 9, 2019
View reviewed changes
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
lex111
lex111 reviewed Feb 9, 2019
View reviewed changes
Comment threadcontent/docs/components-and-props.md
```

Когда React встречает подобный элемент, он собирает все JSX-атрибуты в один объект и передаёт их нашему компоненту. Такой объект называется «пропсы».
Когда React встречает подобный элемент, он собирает все JSX-атрибуты в один объект и передаёт их нашему компоненту. Этот объект называется «пропсы» (props).

@lex111lex111Feb 9, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

А если выделить его?

Этот объект называется props.

@gaearongaearonFeb 9, 2019

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Я бы хотел тогда сначала поменять в английской наверное, не хочется в форматировании отклоняться.

@lex111lex111Feb 9, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Хорошо, главное — не забыть.

lex111
lex111 reviewed Feb 9, 2019
View reviewed changes
Comment threadcontent/docs/components-and-props.md Outdated
lex111
lex111 reviewed Feb 9, 2019
View reviewed changes
Comment threadcontent/docs/components-and-props.md Outdated
lex111
lex111 reviewed Feb 9, 2019
View reviewed changes
Comment threadcontent/docs/components-and-props.md Outdated
alexkuz
alexkuz reviewed Feb 9, 2019
View reviewed changes
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md
>React принимает компоненты, начинающиеся с маленькой буквы, за DOM-теги. Например, `<div />` представляет собой div-тег из HTML, а `<Welcome />` это уже наш компонент `Welcome`, который должен быть доступен в области видимости.
>
>You can read more about the reasoning behind this convention [here.](/docs/jsx-in-depth.html#user-defined-components-must-be-capitalized)
>Вы можете почитать, почему это так работает [здесь.](/docs/jsx-in-depth.html#user-defined-components-must-be-capitalized)

@alexkuzalexkuzFeb 9, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Общее замечание (касается и оригинала): текст ссылки желательно делать содержательным, в том числе для accessibility. Например, что-нибудь вроде: "Чтобы узнать больше о том, как это работает, почитайте [JSX в подробностях]"

@gaearongaearonFeb 9, 2019

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Да, но это надо фиксить в оригинале, а потом тут поправим. Сделаешь PR?

@alexkuzalexkuzFeb 10, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@gaearon готого reactjs/react.dev#1659

Comment threadcontent/docs/components-and-props.md Outdated
Comment threadcontent/docs/components-and-props.md
Comment threadcontent/docs/components-and-props.md Outdated
@lffglffg mentioned this pull request Feb 9, 2019
Merged
@gaearon

gaearon commented Feb 9, 2019

Copy link
Copy Markdown
MemberAuthor

Поправил.

lex111
lex111 approved these changes Feb 9, 2019
View reviewed changes

@lex111lex111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@gaearon
gaearon merged commit 2571281 into masterFeb 10, 2019
@gaearon
gaearon deleted the comps-props branch February 10, 2019 00:23
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@alexkuzalexkuzalexkuz left review comments
@lex111lex111lex111 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs reviewA pull request ready to be reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gaearon@alexkuz@lex111