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

try state nesting #49

Merged
merged 18 commits into from
Nov 14, 2023
Merged

try state nesting #49

merged 18 commits into from
Nov 14, 2023

Conversation

bryzZz
Copy link
Collaborator

@bryzZz bryzZz commented Oct 9, 2023

Вариант того как реализовать UX для вложения состояний друг в друга. Тут мне не нравится что на каждый фрейм во время перемещения состояния нужно пробегаться по всем состояниям, не знаю как сделать по другому :(.

@bryzZz bryzZz mentioned this pull request Oct 10, 2023
Merged
@bryzZz bryzZz self-assigned this Oct 11, 2023
@chekoopa chekoopa added the hacktoberfest-accepted For the particular time of a year label Oct 11, 2023
@chekoopa
Copy link
Collaborator

Проблема ещё в том, что при двойном клике внутри дочернего контейнера, пользователь будет ожидать, что состояние попадёт в него. linkStateByPoint отсюда родился.

@bryzZz
Copy link
Collaborator Author

bryzZz commented Oct 13, 2023

Проблема ещё в том, что при двойном клике внутри дочернего контейнера, пользователь будет ожидать, что состояние попадёт в него. linkStateByPoint отсюда родился.

АА, тогда вернул

@chekoopa chekoopa added the test-me Needs to be checked out and tested label Oct 23, 2023
@chekoopa chekoopa merged commit 8ca0566 into kruzhok-team:main Nov 14, 2023
1 check passed
@bryzZz bryzZz deleted the state-nesting-ux branch November 17, 2023 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted For the particular time of a year test-me Needs to be checked out and tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants