search

freshplanet / ANE-Alert

Air Native Extension (iOS and Android) for Native Alerts
Apache License 2.0
64 stars 21 forks source link

Allow for more buttons. #2

Open Eternalspirit opened 11 years ago

Eternalspirit commented 11 years ago

Hey guys,

Nice ANE. There are a few others out there, but at least on iOS 5.1.1, this is the only one that doesn't crash.

Any chance we'll see support for more than 2 buttons ever?

Thanks!

ataugeron commented 11 years ago

We'll implement this feature if we need it, which is not the case for now. That being said, the ANE is open source and you're more than welcome to contribute ;-) — Sent from Mailbox for iPhone

On Fri, Jun 21, 2013 at 10:36 PM, Eternalspirit notifications@github.com wrote:

Hey guys, Nice ANE. There are a few others out there, but at least on iOS 5.1.1, this is the only one that doesn't crash. Any chance we'll see support for more than 2 buttons ever?

Thanks!

Reply to this email directly or view it on GitHub: https://github.com/freshplanet/ANE-Alert/issues/2

Eternalspirit commented 11 years ago

Sure, I can take a look at doing that, I suppose, though I have very little knowledge or experience of working with iOS/Android native code..

EDIT: The extension actually looks pretty simple. I think I have a handle on how to make it support up to infinite buttons. I'll look at it this weekend and see what I can do.

EDIT2: Looking at Android docs, it appears that the maximum amount of buttons on an AlertDialog is 3.. a positive, negative and neutral button. Although, I don't really care much about the Android version, I just want to get 3 buttons to show up on the iOS version.

EDIT3: So I'm pretty sure I just made it possible to add 3 buttons to an iOS alert, but I do not have any way to compile the ANE right now, nor do I have any devices to test it on. Would you guys be willing to lend a hand with that? If not, I'll have to figure it out on Monday..

EDIT4: By the way, the deadlock issue that you discovered on iOS 5.1.1 is EXACTLY the reason why I could not use the NativeAlert ANE by Mateusz. His ANE otherwise does exactly what I need it to do.