-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add tvOS target #71
Add tvOS target #71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!😍
@@ -22,6 +22,7 @@ | |||
|
|||
import UIKit | |||
|
|||
#if os(iOS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try using @available(tvOS, unavailable)
. Not sure if this works, but looks better this way.
@@ -30,6 +30,7 @@ protocol HeroDebugViewDelegate { | |||
func onDone() | |||
} | |||
|
|||
#if os(iOS) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
@@ -134,7 +134,9 @@ extension HeroContext { | |||
} else if let barView = view as? UINavigationBar, barView.isTranslucent { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure how tvOS's navigation bar works. But i think we should not go in to this else if
statement. The navigationBar snapshot used here is pretty dependent on iOS's navigation bar implementation. Lets limit this else if
to be only available on iOS
I will merge now before I do any works to cause conflicts. Thank you! |
Alright! If you still want any changes let me know :) |
* Minimal working tvOS project * Don't compile HeroDebugPlugin and HeroDebugView for tvOS target
* Minimal working tvOS project * Don't compile HeroDebugPlugin and HeroDebugView for tvOS target
* Minimal working tvOS project * Don't compile HeroDebugPlugin and HeroDebugView for tvOS target
Added tvOS target.
Made HeroDebugPlugin and HeroDebugView unavailable until UISlider functionality is replaced with continuous gesture.