search

Sumi-Interactive / SIAlertView

An UIAlertView replacement with block syntax and fancy transition styles.
MIT License
2.51k stars 424 forks source link

Add support to iOS 5 and lower #6

Closed mrcrow closed 11 years ago

kyoji2 commented 11 years ago

The problem in iOS5 is: window hierarchy is mess up, the background window is over alert window.

skull-squadron commented 11 years ago

This almost works. :+1: Good work. (We're also stuck supporting iOS 5 for at least 9 more months.)

On a 5.0 device, the alert is appearing below the fade effect. Perhaps it's relying on new 6.0+ window hierarchy behavior for Z-ordering the views? I think this is exactly what @kyoji2 referred to.

img

On a 6.0+ device, everything's fine.

img

ricsantos commented 11 years ago

For what it's worth, I am just checking the OS version and displaying SIAlertView or UIAlertView accordingly.

skull-squadron commented 11 years ago

Cool. @aporat is pull #11 in a testable state?

ricsantos commented 11 years ago

Orientation is not supported properly in iOS 5.

ricsantos commented 11 years ago

Orientation support has now been added in pull #19.