Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Chat animation #140

Merged
merged 13 commits into from
Jun 22, 2016
Merged

Chat animation #140

merged 13 commits into from
Jun 22, 2016

Conversation

rasom
Copy link
Contributor

@rasom rasom commented Jun 21, 2016

Attempt to simplify the logic of triggering of the chat's animation. Now all animations related to changes of edit-mode are triggered by (dispatch [:command-edit-mode])/(dispatch [:text-edit-mode]).

@rasom rasom added this to the smooth-user-interface milestone Jun 21, 2016
@rasom rasom added ready and removed in progress labels Jun 21, 2016
@rasom
Copy link
Contributor Author

rasom commented Jun 21, 2016

The next step is to add some sugar for views with animation

@rasom rasom mentioned this pull request Jun 21, 2016
@0xc1c4da
Copy link
Contributor

curious what orientation is used for (i mean we need it eventually)

@rasom
Copy link
Contributor Author

rasom commented Jun 21, 2016

It is used for calculation related to suggestion's header "sticking" https://github.com/status-im/status-react/blob/chat-animation/src/status_im/chat/views/response.cljs#L46

@rasom
Copy link
Contributor Author

rasom commented Jun 21, 2016

ehh, i have to add comments there

@0xc1c4da
Copy link
Contributor

0xc1c4da commented Jun 21, 2016

want to fix* conflicts now and add comments and i'll merge

@rasom
Copy link
Contributor Author

rasom commented Jun 21, 2016

ok

rasom added 2 commits June 21, 2016 19:17
 Conflicts:
	.re-natal
	src/status_im/android/core.cljs
	src/status_im/chat/views/message_input.cljs
	src/status_im/chat/views/plain_message.cljs
	src/status_im/db.cljs
@0xc1c4da 0xc1c4da merged commit 31d7161 into develop Jun 22, 2016
@0xc1c4da 0xc1c4da deleted the chat-animation branch June 22, 2016 18:43
@0xc1c4da 0xc1c4da removed the ready label Jun 22, 2016
0xc1c4da added a commit that referenced this pull request Jan 5, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants