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

[FIX] Add chat message search back to the app #2097

Merged

Conversation

Sameesunkaria
Copy link
Collaborator

@Sameesunkaria Sameesunkaria commented Jul 31, 2018

@RocketChat/ios

Tested on

  • iPhone 5s (10.3.1)
  • iPhone 5s (11.4)
  • iPhone X (11.4)

Screenshots

Closes #2050

@rafaelks rafaelks added this to the 3.1.0 milestone Jul 31, 2018
@rafaelks rafaelks added this to In Progress in Cycle 08: 08/05 ~ 08/27 via automation Jul 31, 2018
@codecov
Copy link

codecov bot commented Jul 31, 2018

Codecov Report

Merging #2097 into develop will decrease coverage by 0.25%.
The diff coverage is 65%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2097      +/-   ##
===========================================
- Coverage    35.72%   35.47%   -0.26%     
===========================================
  Files          382      382              
  Lines        17918    17916       -2     
===========================================
- Hits          6402     6355      -47     
- Misses       11516    11561      +45
Impacted Files Coverage Δ
...ket.Chat/Controllers/Chat/ChatViewController.swift 30.62% <0%> (-8.44%) ⬇️
...rollers/Chat/ChatControllerTitleViewProtocol.swift 0% <0%> (ø) ⬆️
Rocket.Chat/Theme/ThemeableViews.swift 76.53% <100%> (+0.24%) ⬆️
Rocket.Chat/Views/Chat/ChatTitleView.swift 85.71% <76.92%> (+1.93%) ⬆️
Rocket.Chat/Views/Base/BaseNavigationBar.swift 0% <0%> (-100%) ⬇️
...t/Extensions/UISplitViewControllerExtensions.swift 50% <0%> (-50%) ⬇️
Rocket.Chat/Views/Chat/KeyboardFrameView.swift 33.33% <0%> (-7.7%) ⬇️
...t/Views/Subscriptions/SubscriptionsTitleView.swift 87.23% <0%> (-4.26%) ⬇️
...ubscriptionsList/SubscriptionsViewController.swift 39.23% <0%> (-2.46%) ⬇️
... and 12 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 101d5ed...9ef68ac. Read the comment docs.

}

@IBAction func buttonMoreDidPressed(_ sender: Any) {
delegate?.titleViewButtonMoreDidPressed()
@IBAction func buttonSearchDidPressed(_ sender: Any) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do ChatTitleView manages the search button? For me it's supposed to be a UIBarButtonItem. The title view should limit itself to manage everything that goes between the left and the right bar button items.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's how the ellipsis button was implemented 😅. I can add it as a BarButtonItem if you want.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Got it. It would be nice if you could do it :D

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Cycle 08: 08/05 ~ 08/27 automation moved this from In Progress to In Review/QA Jul 31, 2018
Copy link
Collaborator

@filipealva filipealva left a comment

Choose a reason for hiding this comment

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

@Sameesunkaria It's looking great, thank you!

@filipealva filipealva merged commit b028666 into RocketChat:develop Jul 31, 2018
Cycle 08: 08/05 ~ 08/27 automation moved this from In Review/QA to Done Jul 31, 2018
@Sameesunkaria Sameesunkaria deleted the 2050-add-chat-message-search branch August 31, 2018 12:03
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