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

circle reveal animation doesn't work for the first attempt. #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

shinil7
Copy link

@shinil7 shinil7 commented Jul 4, 2017

fixed : changed the search toolbar visibility from gone to invisible.
AND close the search view toolbar with animation when the back button is pressed.

circle reveal animation doesn't work for the first attempt. fix : changed the search toolbar visibility from gone to invisible.
circle reveal animation doesn't work for the first attempt. fix : changed the search toolbar visibility from gone to invisible.
close the search view toolbar with animation when the back button is pressed.
@meierjan
Copy link

meierjan commented Jul 18, 2017

Hey @shinilms12 I just created an issue for that an seen that you already created an PR. Good idea to just make it invisible! Really elegant.

@shinil7
Copy link
Author

shinil7 commented Jul 19, 2017

@meierjan Thank you. You can close your issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants