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

Enable\disable alert click animation #235

Merged
merged 3 commits into from
Sep 3, 2020
Merged

Conversation

TalbotGooday
Copy link
Contributor

Oddly enough, this functionality turned out to be very important when it became necessary to create an alert whose width is less than the width of the screen.

Issue: #226

@kpmmmurphy
Copy link
Collaborator

Looks all good @TalbotGooday, before we merge, please update the version in the build.gradle, and the change log - and we're good to go! Thanks again.

@TalbotGooday
Copy link
Contributor Author

@kpmmmurphy Do you want to release a new version with this request? I thought it would just end up in the code with some larger update)

And another question, if you don't mind: why is the Alert.hide() method is private? Are there any bugs with the hide() method?

@kpmmmurphy
Copy link
Collaborator

@TalbotGooday I think so, otherwise it may site here for some time! Please bump it to 6.2.0!

That method is private, as we want to force the use of Alerter.hide(), which is a singular place to handle hiding & clean up. We could make the method public, but this may require some refactoring!

@TalbotGooday
Copy link
Contributor Author

@kpmmmurphy I got it. Thank you)

@kpmmmurphy kpmmmurphy merged commit ebd9069 into Tapadoo:master Sep 3, 2020
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

2 participants