pingpongboss / StandOut

StandOut lets you easily create floating windows in your Android app.
http://pingpongboss.github.com/StandOut
MIT License
1.24k stars 379 forks source link

Update core functionality to higher APIs #6

Closed jesdga95 closed 11 years ago

jesdga95 commented 11 years ago

Give it a bit more of "Holo" look and usability.

jesdga95 commented 11 years ago

I'm not sure if this should be merged on another branch other than "master", if you want to keep retro compatibility with Older apis as froyo or gingerbread, but atleast give it a look. This looks really solid on jellybean

pingpongboss commented 11 years ago

The minSdk must not change due to backwards compatibility. I will check out your changes and see if anything can be done to keep backwards compatibility. Thanks for your contribution.

pingpongboss commented 11 years ago

Oops that close button is too easy to click

pingpongboss commented 11 years ago

did you forget to git add your drawables? You've removed close.png and hide.png, but have not added your own.

pingpongboss commented 11 years ago

Hi D4rKn3sSyS, looking at this further, I've determined that it is too unwieldily to merge. I am going to refactor the code base so that not everything is inside of StandOutWindow.java. I encourage you to apply your changes again after that. But I urge you strongly to make multiple logical commits. IE: Each commit is for a different bug/feature (you can use commit --amend to avoid spurious commits). That will be much easier to merge.

I can't merge this because EVERY LINE of StandOutWindow.java is reported to have been changed, so I can't see what has happened.

I am closing this now, but I'll ping you later when I've refactored this. Thanks.

jesdga95 commented 11 years ago

Changes are basically made on show() method, documentation, some drawables removed (used now from frameworks to fit holo style), and finally popup menu instead of the inflating method of dropdown. The rest is purely fix indenting, that's why git reports like if line was rewritten. I'll wait your ping then ;)

jesdga95 commented 11 years ago

Oh, and now there's a new flag, wich adds Vibration to ACTION_DOWN event, just like commit msg says :). About SDK, well you want to keep retrocompatibility and that's fine, but API has changed a lot since eclair, you're using those drawables, and they don't fit at all on jellybean design, that's why I said, if this some day gets merged, I guess you should make a new branch, something like "API15+"

pingpongboss commented 11 years ago

I have refactored the code. You may submit another patch now. Try to keep the commits small and in in logical chunks. Thanks for your contribution

pingpongboss commented 11 years ago

what exactly needs API15+? Just the drawables?

jesdga95 commented 11 years ago

No, drawables can be copied, PopupMenu changed its look and added new methods on API15