Conversation
This reverts commit 081ca7a.
Merge from upstream
Add customizable accessoryView besides messageContainer
…into development
This change provides better handling of keyboard frames. It copes with different values of UIViewController.edgesForExtendedLayout, UIScrollView.contentInsetAdjumentBehavior, view controller containment scenarios and undocked keyboards, and uses cleaner code. Improvements and fixes in this change: 1. Translates keyboard frame from screen coordinates into view controller coordinates. They aren’t the same thing when containment is involved, or edgesForExtendedLayout does not include .top. 2. Avoids special casing any iPhone models and avoids dealing with safeAreaInsets manually. In iOS 11, UIScrollView may behave differently depending on contentInsetAdjumentBehavior, and the old code incorrectly assumed that the mode is .always (while it is in fact is .automatic) and that no extra safe area insets have been introduced at the bottom edge. Best part, we don’t have to deal with any of this at all, we can offload the logic to UIScrollView itself by looking at the difference between adjustedContentInset and contentInset. 3. Handles view controller containment and undocked keyboards by taking into account an intersection of the scroll view frame and the keyboard’s frame. I think Apple demoes this in one of the WWDC videos, but I couldn’t find which one (or if I’m misremembering). Regardless, this technique makes sure we compute and use a sensible number. There’s also a couple of extra renames that make the intention of the modified code clearer.
…n when compare double
Adjust contentInset for the keyboard frame properly
…with-coder Enhancement/init reusable view with coder
Fix scrollsToBottomOnKeybordBeginsEditing typo (development branch)
* Remove MessageInputBar files and change build settings * Conditional imports to allow for builds without MessageInputBar * Update Cardfile * Specify Version in Cartfile for MessageInputBar * Remove conditional imports in favor of Carthage for local development * Additional `import MessageInputBar` * Remove `MessageInputBar` from Cartfile.private * Bump MessageInputBar version * Add Cartfile * Tweak build settings * Specify import of MessageInputBar for Carthage
Member
Author
|
Blocked by MessageKit/MessageInputBar#10 |
fix intrinsic content size for MessageLabel
…eViewController. Moreover, there is an easier method to find the visible section. It is unnecessary to scroll to the bottom when title view updated.
Example chat screens do not scroll to the bottom on new message insertion.
Accessoryview updates try to fix
Add didTapAccessoryView Delegate method and adjust layout
…ayButton Views, MessageContent, MessageCollectionView Cells, MessagesCollectionViewFlow Layout.
Fix: Allow required initializator methods @ Views.
Member
Author
|
Lets close this in favor of #910 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed
InputTextViewto prevent crashes inparseForComponents().#791 by @nathantannar4
Added
Added support for detection and handling of
NSLinks inside of messages.#815 by @jnic
Added customizable
accessoryView, with a newMessagesDisplayDelegatefunctionconfigureAccessoryView, and corresponding size & padding properties inMessageSizeCalculator.#710 by @hyouuu
Added
additionalBottomInsetproperty that allows to adjust the bottom content inset automatically set on the messages collection view by the view controller.#787 by @andreyvit
Fixed
Fixed bottom content inset adjustment when using an undocked keyboard on iPad, or when
edgesForExtendedLayoutdoes not include.top, or when a parent container view controller adds extra views at the top of the screen.#787 by @andreyvit
Fixed the
MessageData.emojicase to use 2x themessageLabelFontsize by default.#795 by @Vortec4800.
TODO: