Update ActionSheet.java

[brx0826]

Referenced from
http://stackoverflow.com/questions/14177781/java-lang-illegalstateexception-can-not-perform-this-action-after-onsaveinstanc

and

http://stackoverflow.com/questions/7575921/illegalstateexception-can-not-perform-this-action-after-onsaveinstancestate-wit

[brx0826]

Thank you!
Is it possible for me to download new changes via Gradle?

[brx0826]

Thank you!

Would you update gradle so that I can install your control via gradle?

Thanks a lot,
Rixian

On Thu, Dec 10, 2015 at 10:59 AM, Mr.Bao notifications@github.com wrote:

Merged #17 #17.

—
Reply to this email directly or view it on GitHub
#17 (comment)
.

[baoyongzhang]

Hi, please wait.

[baoyongzhang]

It's ok now.

compile 'com.baoyz.actionsheet:library:1.1.4'

[brx0826]

Hello Mr.Bao,

It's much better now, but it is still crashing.

Here are my suggestions;

1. I changed this part of code;
   @OverRide public void onSaveInstanceState(Bundle outState) { super.
   onSaveInstanceState(outState); outState.putBoolean(EXTRA_DISMISSED,
   mDismissed); }
   to

@OverRide public void onSaveInstanceState(Bundle outState) {
outState.putBoolean(EXTRA_DISMISSED, mDismissed); }

And you didn't approve this change, right?
Is this a problem? I saw they wrote on stackoverflow that this
"super.onSaveInstanceState(outState);" is causing bugs from API Level 11.
What is your opinion on this?

2. Can we put the following code inside show function also in new Handler().
   post(new Runnable() { }); ?

FragmentTransaction ft = manager.beginTransaction(); ft.add(this, tag); ft.
addToBackStack(null); ft.commitAllowingStateLoss();

I made those changes and issued a pull request.
Please check.

Best Regards,
Rixian

On Thu, Dec 10, 2015 at 2:45 PM, Mr.Bao notifications@github.com wrote:

It's ok now.

compile 'com.baoyz.actionsheet:library:1.1.4'

—
Reply to this email directly or view it on GitHub
#17 (comment)
.

[brx0826]

Hello Mr.Bao,

I'd really appreciate it if you can check my pull request and reply me.

Thank you so much,
Rixian

On Thu, Dec 10, 2015 at 4:10 PM, rixian bai brx0826@gmail.com wrote:

Hello Mr.Bao,

It's much better now, but it is still crashing.

Here are my suggestions;

1. I changed this part of code;
   @OverRide public void onSaveInstanceState(Bundle outState) { super.
   onSaveInstanceState(outState); outState.putBoolean(EXTRA_DISMISSED,
   mDismissed); }
   to

@OverRide public void onSaveInstanceState(Bundle outState) {
outState.putBoolean(EXTRA_DISMISSED, mDismissed); }

And you didn't approve this change, right?
Is this a problem? I saw they wrote on stackoverflow that this
"super.onSaveInstanceState(outState);" is causing bugs from API Level 11.
What is your opinion on this?

2. Can we put the following code inside show function also in new Handler
   ().post(new Runnable() { }); ?

FragmentTransaction ft = manager.beginTransaction(); ft.add(this, tag); ft
.addToBackStack(null); ft.commitAllowingStateLoss();

I made those changes and issued a pull request.
Please check.

Best Regards,
Rixian

On Thu, Dec 10, 2015 at 2:45 PM, Mr.Bao notifications@github.com wrote:

It's ok now.

compile 'com.baoyz.actionsheet:library:1.1.4'

—
Reply to this email directly or view it on GitHub
#17 (comment)
.

[brx0826]

Once you review, approve my pull request and generate a new Gradle Version, I will mark the issue I created as resolved.
I really need your help in this urgently.
Thank you so much!