Conversation
#import <AppKit/AppKit.h> | ||
#endif | ||
|
||
#ifdef __IPHONE_OS_VERSION_MIN_REQUIRED |
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.
This could be combined with the above block.
🍔 |
|
||
MEDEdgeInsets MEDEdgeInsetsFromString(NSString *string) { | ||
MEDEdgeInsets insets = MEDEdgeInsetsZero; | ||
if (string) { |
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.
Style: explicit comparison vs. nil
💥 |
|
||
MEDEdgeInsets MEDEdgeInsetsFromString(NSString *string) { | ||
double top = 0, left = 0, bottom = 0, right = 0; | ||
if (string != nil) { |
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.
This should be an assertion instead, IMO.
🍟 |
Should I switch all edge insets comparisons to using |
Ehhh… up to you, as long as the tests for that function don't trust it. |
|
Ah, sorry, can you also add this to the framework header? |
I didn't explicitly add it because |
Yes, every public header should be there. |
🍸 |
🍻 |
Fixes #17.
For the
MEDEdgeInsets
geometry functions, I wasn't sure if you wanted me to reuse the UIKit functions in the iOS implementations, but I reimplemented it for both platforms anyways. This can be switched to use UIKit functions if necessary.