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

[BUG][RCVC] Left and right edges on iPad need to increase to match the other elements in the controller #2295

Merged
merged 10 commits into from
Nov 9, 2018

Conversation

filipealva
Copy link
Collaborator

@filipealva filipealva commented Nov 9, 2018

@RocketChat/ios

Closes #2276

@filipealva filipealva added this to the 3.2.0 milestone Nov 9, 2018
@filipealva filipealva changed the title [BUG][RCVC] Left and right edges on iPad need to increase to match the other elements in the controller [WIP][BUG][RCVC] Left and right edges on iPad need to increase to match the other elements in the controller Nov 9, 2018
@codecov
Copy link

codecov bot commented Nov 9, 2018

Codecov Report

Merging #2295 into develop will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2295      +/-   ##
===========================================
- Coverage    26.02%   26.02%   -0.01%     
===========================================
  Files          433      433              
  Lines        15846    15849       +3     
===========================================
  Hits          4124     4124              
- Misses       11722    11725       +3
Impacted Files Coverage Δ
...Views/Chat/New Chat/Cells/MessageActionsCell.swift 0% <ø> (ø) ⬆️
...at/Views/Chat/New Chat/Cells/BaseMessageCell.swift 0% <ø> (ø) ⬆️
.../Views/Chat/New Chat/Cells/DateSeparatorCell.swift 0% <ø> (ø) ⬆️
...t/Views/Chat/New Chat/Cells/UnreadMarkerCell.swift 0% <ø> (ø) ⬆️
...Chat/Views/Chat/New Chat/Cells/ReactionsCell.swift 0% <ø> (ø) ⬆️
...ews/Chat/New Chat/Cells/BaseQuoteMessageCell.swift 0% <0%> (ø) ⬆️
...t/Views/Chat/New Chat/Cells/BasicMessageCell.swift 0% <0%> (ø) ⬆️
...hat/Views/Chat/New Chat/Cells/MessageURLCell.swift 0% <0%> (ø) ⬆️
...Controllers/Chat/ChatSections/MessageSection.swift 4.71% <0%> (+0.04%) ⬆️
...New Chat/Cells/BaseTextAttachmentMessageCell.swift 0% <0%> (ø) ⬆️
... 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 3151726...352ed8e. Read the comment docs.

@rafaelks
Copy link
Contributor

rafaelks commented Nov 9, 2018

@filipealva Referenced PR is merged already! 👍

extension ComposerView: UITextViewDelegate {
@objc public func textViewDidChangeSelection(_ textView: UITextView) {
currentDelegate.composerViewDidChangeSelection(self)
return
}

public func textView(_ textView: UITextView, shouldChangeTextIn range: NSRange, replacementText text: String) -> Bool {

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 123 characters (line_length)

@filipealva filipealva changed the title [WIP][BUG][RCVC] Left and right edges on iPad need to increase to match the other elements in the controller [BUG][RCVC] Left and right edges on iPad need to increase to match the other elements in the controller Nov 9, 2018
@rafaelks rafaelks merged commit 834237e into develop Nov 9, 2018
@rafaelks rafaelks deleted the fix/message-width branch November 9, 2018 20:23
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants