X Tutup
Skip to content
This repository was archived by the owner on Jun 7, 2020. It is now read-only.

[NEW] Components interspacing#2269

Merged
rafaelks merged 17 commits intofeature/integrate-rc-view-controllerfrom
feature/components-spacing
Oct 31, 2018
Merged

[NEW] Components interspacing#2269
rafaelks merged 17 commits intofeature/integrate-rc-view-controllerfrom
feature/components-spacing

Conversation

@filipealva
Copy link
Contributor

@RocketChat/ios

return 0
}

func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, insetForSectionAt section: Int) -> UIEdgeInsets {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 162 characters (line_length)

return 0
}

func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, minimumInteritemSpacingForSectionAt section: Int) -> CGFloat {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 175 characters (line_length)


extension MessagesViewController {

func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, minimumLineSpacingForSectionAt section: Int) -> CGFloat {

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 170 characters (line_length)

@filipealva filipealva requested a review from rafaelks October 31, 2018 02:33
@rafaelks rafaelks added this to the 3.2.0 milestone Oct 31, 2018
@codecov
Copy link

codecov bot commented Oct 31, 2018

Codecov Report

Merging #2269 into feature/integrate-rc-view-controller will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@                           Coverage Diff                           @@
##           feature/integrate-rc-view-controller   #2269      +/-   ##
=======================================================================
- Coverage                                 25.61%   25.6%   -0.02%     
=======================================================================
  Files                                       437     437              
  Lines                                     16515   16522       +7     
=======================================================================
  Hits                                       4231    4231              
- Misses                                    12284   12291       +7
Impacted Files Coverage Δ
...t/Views/Chat/New Chat/Cells/ImageMessageCell.swift 0% <0%> (ø) ⬆️
...cket.Chat/Views/Chat/New Chat/Cells/FileCell.swift 0% <0%> (ø) ⬆️
...at/Views/Chat/New Chat/Cells/FileMessageCell.swift 0% <0%> (ø) ⬆️
...Chat/Controllers/Chat/MessagesViewController.swift 42.16% <0%> (-3.29%) ⬇️
...t/Views/Chat/New Chat/Cells/VideoMessageCell.swift 0% <0%> (ø) ⬆️
...cket.Chat/Controllers/Chat/MessagesViewModel.swift 33.33% <0%> (-0.42%) ⬇️
...ket.Chat/Views/Chat/New Chat/Cells/ImageCell.swift 0% <0%> (ø) ⬆️
...ket.Chat/Views/Chat/New Chat/Cells/VideoCell.swift 0% <0%> (ø) ⬆️

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 f81a2d7...85d906a. Read the comment docs.

@rafaelks rafaelks merged commit 99737bb into feature/integrate-rc-view-controller Oct 31, 2018
@rafaelks rafaelks deleted the feature/components-spacing branch October 31, 2018 16:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup