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

Added BIG size, 72dp size for fab buttons, Fix MOVE feedback, update gitignore #16

Merged
merged 3 commits into from
Mar 6, 2015

Conversation

Aracem
Copy link
Contributor

@Aracem Aracem commented Mar 6, 2015

1 Added BIG size for fab buttons. Big size shows a 72dp fab button. Could be useful for some projects.

I have a little problems with your code style. I use Android AOSP code style. If you have problem whith this please comment and cancel the pull request. I will make another one with your code style.

2 Resolve the problem with the OnTouchEvent - MOVE that causes the ActionButton stay Pressed when the user clicks and move the finger out of the ActionButton.

3 Improve .gitignore to not add unneccesary files

Thank you so much

@Aracem
Copy link
Contributor Author

Aracem commented Mar 6, 2015

@shell-software Probably this has conflicts due not well formed .gitignore that add gradle files to the repository

@vbaidak vbaidak merged commit 27c6cda into Scalified:1.1.0 Mar 6, 2015
vbaidak pushed a commit that referenced this pull request Mar 6, 2015
@vbaidak
Copy link
Member

vbaidak commented Mar 6, 2015

Hi Aracem,

The pull has been accepted with modifications:

  1. Changed the way of converting density-dependent values into density-independent ones (project uses new logic, starting from version 1.1.0)
  2. Reworked onTouchEvent(MotionEvent) overridden method as described
  3. Removed excessive ignores from .gitignore (see comments)

vbaidak pushed a commit that referenced this pull request Mar 6, 2015
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.

2 participants