Custom alert View to iOS applications

Dear @vitormesquita,

Following this issue, it looks like it will be sufficient to add the following condition to the if statement in order to fix the problem:

if UIDevice.current.orientation.isLandscape && !preferredInterfaceOrientationForPresentation.isPortrait {
    ...
}

According to Apple documentation for preferredInterfaceOrientationForPresentation.isPortrait: A Boolean value indicating whether the user interface is currently presented in a portrait orientation.

You're very welcome to check other branch I created to showcase the issue and the fix (also added examples to test it.) All the examples are already configured for you.

Way to replicate the issue:

1. Clone my Malert fork from orientation-fix-test branch
2. Run the Malert Example application
3. Rotate the device to Landscape
4. While in landscape orientation, click on "Example Orientation Issue" example
5. While in landscape orientation, click on "Example Orientation Issue (fixed)" to showcase the fix

I've also done some tests to determine whether this affects users using the application in Landscape mode by adding the landscape orientation in info.plist

Tested on:

• IPhone 8 (physical device)
• IPhone 12 Pro Max (simulator)
• IPad (simulator)

Kindly let me know if you have any questions or concerns. If you think there might be a better solution for this issue I'm very open to discuss it.

Thank You!

P.S. I forgot to remove the Landscape orientation from Info.plist on orientation-fix-test branch so please remove it before testing the issue.

Added overlay color configuration

You can now set the color of overlay on Malert object through overlayColor property. Color is then set on background color of visibleView. This can be useful for some use cases like when it's required by the design.

Renamed round method of UIView extension to roundView

I've renamed 'round' method of UIView extension since it hides Swift default 'round' method and creates issues when working with classes extending UIView. Although I couldn't find where this method is used. Kindly check if I'm missing something.

MalertActionCallbackProtocol now extends AnyObject

According to Apple deprecation and requirement.

Hello,

There's an issue with custom view height when application "Supported interface orientations" is set to portrait but the device is in landscape orientation.

Looks like the issue is following code block of Malert constrains extension (Malert+Constraints.swift):

if UIDevice.current.orientation.isLandscape {
         let topContraint = malertView.topAnchor.constraint(equalTo: visibleView.topAnchor, constant: 16)
         topContraint.priority = UILayoutPriority(900)
         malertConstraints.append(topContraint)

         let bottomConstraint = malertView.bottomAnchor.constraint(equalTo: visibleView.bottomAnchor, constant: -16)
         bottomConstraint.priority = UILayoutPriority(900)
         malertConstraints.append(bottomConstraint)
}

Above if condition doesn't sufficiently checks if landscape orientation is enabled for the application. I suggest to change above condition statement to:

if UIDevice.current.orientation.isLandscape && supportedInterfaceOrientations.contains(.landscape) {
    ...
}

This way the issue can be avoided by following extension:

extension Malert {
	public override var supportedInterfaceOrientations: UIInterfaceOrientationMask {
		return .portrait
	}
}

Kindly note that this is just something I came up with on the fly to patch this issue so please feel free to implement a better solution if necessary.

Thank you!

CocoaPods and Carthage are awesome tools and make our life really easier, but there are some devs who still don't know how to use them.

It would be cool to add the Manual installation guide in your README.md. You can take a look at my iOS Readme Template to see how you can do it.

I have a following issue when my app goes to landscape mode. Do you have any suggestions?

I would like alert to be a fixed width instead of going almost entire width of the screen. Thanks