Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added isIPhone, isIPad, isSimulator methods to UIDevice+Model #62

Merged
merged 2 commits into from
Mar 26, 2016
Merged

Added isIPhone, isIPad, isSimulator methods to UIDevice+Model #62

merged 2 commits into from
Mar 26, 2016

Conversation

aashishdhawan
Copy link
Contributor

  1. Added function isIPhone and isIPad.
    When working on Universal application these are very useful.
  2. Added function isSimulator
    This was already present in Sugar via Simulator. Just added convenient method here.

@@ -33,4 +33,16 @@ public extension UIDevice {

return result
}

public static func isIPhone() -> Bool {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe its just me but iPhone with a capital I kinda makes me sad. Don't get me wrong, I know why you wrote it that way.

Could we do isPhone and isPad instead?

Probably overreacting here 😎

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@zenangst zenangst merged commit 32b26f6 into hyperoslo:master Mar 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants