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.

Sign up for GitHub

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

Jump to bottom

Терминология: fallback #93

Merged
merged 1 commit into from
Feb 11, 2019
Merged

Терминология: fallback #93

merged 1 commit into from
Feb 11, 2019

Conversation

gcor
Copy link
Collaborator

@gcor gcor commented Feb 11, 2019

@another-guy another-guy merged commit c6da0ad into reactjs:master Feb 11, 2019
@gaearon
Copy link
Member

gaearon commented Feb 12, 2019

Есть ещё вариант "заглушка", но может быть слишком близко к задержке (у которой кстати тоже заглушка бывает)

@another-guy
Copy link
Collaborator

@gaearon заглушку мы тоже рассмотрели как вариант. #92 (comment)

@gaearon
Copy link
Member

gaearon commented Feb 12, 2019

Я в целом за заглушку

@lex111
Copy link
Member

lex111 commented Feb 12, 2019

fallback prop — запасной проп, но fallback UI — заглушка интерфейса, так получается?

@another-guy
Copy link
Collaborator

fallback prop — запасной проп, но fallback UI — заглушка интерфейса, так получается?

Если остановились на этом решении, надо не забыть исправить в #73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lex111 lex111 lex111 approved these changes

@another-guy another-guy another-guy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gcor @gaearon @another-guy @lex111