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

[NEW] Components interspacing #2269

Merged

Conversation

filipealva
Copy link
Collaborator

@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)

@@ -121,6 +121,18 @@ final class MessagesViewController: RocketChatViewController {

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)

@rafaelks rafaelks added this to the 3.2.0 milestone Oct 31, 2018
@rafaelks rafaelks added this to In Progress in Cycle 10: 10/05 ~ 11/05 via automation 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
Cycle 10: 10/05 ~ 11/05 automation moved this from In Progress to Done 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
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants