-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Swift3 #671
Swift3 #671
Conversation
Fixed Unit Test (2 errors in HiddenRowsTests.swift which I cannot figure out)
…nableDeselection = false and user deselects a row) Added enableDeselection customisation parameter to SelectorViewController
… size of the app. This only happens if the app uses `preferredFont:forTextStyle`. So no need to worry if you were using other fonts.
Operator fixes Wip Build errors fixed Fixed KVO Warning fixes Additional warning fixes Update Core.swift Fixes with kvo for wrong casts Fixed conversion to NSNumber in KVO Removed development team
Swift 3 Beta 6 Support
XCode 8 GM compatibility
Fix predicate evaluation when values include Optional and or nil
Remove PostalAddressRow from project
Change the access control to open for some classes and methods
Declare some more classes, functions and variables as open
…to follow new swift 3 first method named parameter.
c44c588
to
7d68bad
Compare
Add .swift-version file to pass cocoapods's validation.
|
||
Pull requests associated with this milestone can be found in this [filter](https://github.com/xmartlabs/Eureka/issues?utf8=%E2%9C%93&q=milestone%3A2.0.0%20). | ||
|
||
We have made tons of changes in Eureka API to follow the new Swift API design guidelines. |
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.
tons of changes to the
Eureka API
@@ -0,0 +1,53 @@ | |||
# Eureka 2.0 Migration Guide | |||
|
|||
Eureka 2.0.0 includes complete Swift 3 Compatibility and adopts the new [API Design Guidelines](https://swift.org/documentation/api-design-guidelines/). It also includes new validations build-in feature. |
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.
It also includes validations as a build-in features
# Eureka 2.0 Migration Guide | ||
|
||
Eureka 2.0.0 includes complete Swift 3 Compatibility and adopts the new [API Design Guidelines](https://swift.org/documentation/api-design-guidelines/). It also includes new validations build-in feature. | ||
Bring support to swift 3 involves some API updates to follow apple Swift API best practices, we have also changed and deprecated some API. |
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.
Apple's
deprecated some API's
Removed APIs: | ||
|
||
* `PostalAddressRow` and `ImageRow` was deleted. | ||
* row has been deleted. You can find it and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity). |
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.
Remove row has been deleted.
Removed APIs: | ||
|
||
* `PostalAddressRow` and `ImageRow` was deleted. | ||
* row has been deleted. You can find it and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity). |
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.
You can find them and many other custom rows at EurekaCommunity's
|
||
`highlightCell` and `unhighlightCell` callbacks were deleted, now we should use `row.isHighlighted` from cell update to check from highlighted status and make UI modification according its value. | ||
|
||
In case you want to do something when highligth state switches its value you can set up `onCellHighlightChanged` callback. |
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.
highlight
button.setImage(UIImage(named: "checkedDay"), forState: .Selected) | ||
button.setImage(UIImage(named: "uncheckedDay"), forState: .Normal) | ||
button.setImage(UIImage(named: "checkedDay"), for: .selected) | ||
button.setImage(UIImage(named: "uncheckedDay"), for: UIControlState()) |
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.
should be .normal
@@ -35,6 +35,7 @@ | |||
<string>UIInterfaceOrientationPortrait</string> | |||
<string>UIInterfaceOrientationLandscapeLeft</string> | |||
<string>UIInterfaceOrientationLandscapeRight</string> | |||
<string>UIInterfaceOrientationPortraitUpsideDown</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.
This line shouldn't have been added.
* row has been deleted. You can find it and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity). | ||
|
||
`highlightCell` and `unhighlightCell` callbacks were deleted, now we should use `row.isHighlighted` from cell update to check from highlighted status and make UI modification according its value. | ||
* `PostalAddressRow` and `ImageRow` was deleted. You can find them and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity). |
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.
PostalAddressRow
and ImageRow
were deleted
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.
Where can I find these rows in EurekaCommunity?
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.
They are currently in progress. We will soon add them there. I believe the ImageRow has also been added to the Examples project.
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.
Yeah, I just noticed that while looking for examples to make my custom rows working again. Swift 3 really is a pain in the neck!
* validatesOnChangeAfterBlurred - Validates whenever the row value changes after it resigns first responder for the first time. | ||
* validatesOnDemand - We should manually validate the row or form by invoking `validate()` method. | ||
|
||
If you want to validate the entire form (all the rows) you can invoke Form `validate()` method. |
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.
you can invoke form.validate()
is more clear
…Eureka community soon. fixes #674
Changing access level of keyboardWillShow and keyboardWillHide methods
No description provided.