Skip to content

Терминология: функциональный/классовый компонент - #23

Merged
lex111 merged 1 commit into
masterfrom
gaearon-patch-1
Feb 9, 2019
Merged

Терминология: функциональный/классовый компонент#23
lex111 merged 1 commit into
masterfrom
gaearon-patch-1

Conversation

@gaearon

Copy link
Copy Markdown
Member

На английском мы отошли от "functional" в сторону "function component". То есть компонент-класс и компонент-функция. Но это как-то совсем странно по-русски.

Думаю ок будет оставить функциональный/классовый компонент. Кажется относительно популярно так уже.

В будущем это будет не так релевантно по мере того как классы уходят на второй план.

@lex111

Copy link
Copy Markdown
Contributor

Функция-компонент - вполне ок, и оставить классовый компонент, как тебе?

@gaearon

Copy link
Copy Markdown
MemberAuthor

Немножко коробит несоответствие.

Может тогда просто компонент-функция и компонент-класс?

В первую очередь они таки компоненты с точки зрения реакта.

@lex111

lex111 commented Feb 9, 2019

Copy link
Copy Markdown
Contributor

На английском мы отошли от "functional" в сторону "function component".

Глупый, наверно, вопрос - а зачем вы так сделали?

@gaearon

Copy link
Copy Markdown
MemberAuthor

Были всякие споры на тему, «функциональны» ли с точки зрения чистого ФП компоненты. Хотелось избежать повторения когда введём хуки, тк строго говоря они не совсем ФП, по крайней мере не в чистом безэффектном понимании.

С другой стороны, это как бы промежуточный шаг, потому что в будущем компоненты-функции просто станут «компонентами», а компоненты-классы будут документированы на отдельной странице.

@gaearon

Copy link
Copy Markdown
MemberAuthor

Так что, который вариант выбираем? )

@lex111

Copy link
Copy Markdown
Contributor

Который ты предложил :)

@lex111
lex111 merged commit b5416b6 into masterFeb 9, 2019
@lex111
lex111 deleted the gaearon-patch-1 branch February 9, 2019 18:59
@another-guyanother-guy added the translation-agreements Propose new agreement or change to an existing one label Feb 10, 2019
@vvscodevvscode mentioned this pull request Mar 19, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translation-agreementsPropose new agreement or change to an existing one

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@gaearon@lex111@another-guy