Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added new settings with functionality #8

Merged
merged 5 commits into from
Apr 18, 2017
Merged

Conversation

bdudarPGS
Copy link
Contributor

Added setting for auto-scrolling down when refreshing message view, a…dded setting for auto-adjusting input toolbar to keyboard

…dded setting for auto-adjusting input toolbar to keyboard
@bdudarPGS bdudarPGS requested a review from pgs-dkanak April 13, 2017 15:23
@@ -72,7 +72,7 @@ extension ViewController: MessagesViewDelegate {
func didTapRightButton() {
let text = messagesView.inputText
TestData.exampleMessageText.append(text)
messagesView.refresh()
messagesView.refresh(scrollDown: true)
Copy link
Contributor

Choose a reason for hiding this comment

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

There is a setting for this feature: MessagesViewSettings::textInputScrollsToRecentMessage, and this one should be used and/or renamed if needed. My proposal is: scrollsToLastMessage.

…fixed keyboard vertical adjustment, fixed collectionView bottom scroll indicator inset, lower Deployment Target to enable iPhone 4s, added public scrollToLastMessage function
Copy link
Contributor

@pgs-dkanak pgs-dkanak left a comment

Choose a reason for hiding this comment

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

screen shot 2017-04-18 at 12 36 37
Message in its default form lost its tail. It should be visible.

public var delegate : MessagesViewDelegate?
public var dataSource: MessagesViewDataSource?

//NARK:-
Copy link
Contributor

Choose a reason for hiding this comment

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

A bit of typo

Copy link
Contributor

@pgs-dkanak pgs-dkanak left a comment

Choose a reason for hiding this comment

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

Typo correction

@pgs-dkanak pgs-dkanak merged commit a7ce188 into develop Apr 18, 2017
@pgs-dkanak pgs-dkanak deleted the feature/scroll_fix branch April 18, 2017 11:38
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