Skip to content

Update relative methods signatures when specifying multiple relative views - #48

Merged
lucdion merged 1 commit into
masterfrom
relative_visibles
Jun 18, 2017
Merged

lucdion merged 1 commit into
masterfrom
relative_visibles

Conversation

@lucdion

@lucdion lucdion commented Jun 18, 2017

Copy link
Copy Markdown
Member

Instead of having a variadic parameter, method above(of…), below(of…), left(of…) and right(of…) now takes either a single UIView or an Array of UIViews.
This make it easier to call these methods if the developper has already an array of UIViews. Variadic parameters are quite limited in Swift, they don’t accept an Array, which other language accept.

Plus:

  • Refactor source code to split PinLayout’s implementation in multiple files.

…views.

Instead of having a variadic parameter, method `above(of…)`, `below(of…)`, `left(of…)` and `right(of…)` now takes either a single UIView or an Array of UIViews.
This make it easier to call these methods if the developper has already an array of UIViews. Variadic parameters are quite limited in Swift, they don’t accept an Array, which other language accept.

Plus:
* Refactor source code to split PinLayout’s implementation in multiple files.
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #48 into master will decrease coverage by 0.42%.
The diff coverage is 85.45%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #48      +/-   ##
=========================================
- Coverage   95.93%   95.5%   -0.43%     
=========================================
  Files          15      19       +4     
  Lines        3245    3273      +28     
=========================================
+ Hits         3113    3126      +13     
- Misses        132     147      +15
Impacted Files Coverage Δ
Sources/PinLayout.swift 100% <ø> (ø) ⬆️
Sources/PinLayout+Filters.swift 0% <0%> (ø)
Sources/PinLayoutImpl.swift 86.14% <100%> (-0.41%) ⬇️
Tests/RelativePositionMultipleViewsSpec.swift 100% <100%> (ø) ⬆️
Tests/RelativePositionSpec.swift 100% <100%> (ø) ⬆️
Sources/PinLayoutImpl+Warning.swift 75% <75%> (ø)
Sources/PinLayoutImpl+Coordinates.swift 83.67% <83.67%> (ø)
Sources/PinLayoutImpl+Relative.swift 86.7% <86.7%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88ff81e...fa0612f. Read the comment docs.

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.

2 participants