Consider custom middleContentView height

[PhilipDukhov]

This fixes #264

I'm not sure if it's a proper fix for the library, but it works good in my case.

I believe the code above the fix is still needed to manage inputTextView.isScrollEnabled state with the height constraint, so it shouldn't be placed in else statement.

InputBarAccessoryView/Sources/InputBarAccessoryView.swift

Lines 571 to 586 in 093bdc7

	var inputTextViewHeight = requiredInputTextViewHeight
	if inputTextViewHeight >= maxTextViewHeight {
	if !isOverMaxTextViewHeight {
	textViewHeightAnchor?.isActive = true
	inputTextView.isScrollEnabled = true
	isOverMaxTextViewHeight = true
	}
	inputTextViewHeight = maxTextViewHeight
	} else {
	if isOverMaxTextViewHeight {
	textViewHeightAnchor?.isActive = false || shouldForceTextViewMaxHeight
	inputTextView.isScrollEnabled = false
	isOverMaxTextViewHeight = false
	inputTextView.invalidateIntrinsicContentSize()
	}
	}

[nathantannar4]

Yea I think this is right. Replacing the middle content view is def very experimental, and this was a missed edge case.

I think we should optimize this tho, so that we skip calculating inputTextViewHeight if the middleContentView != inputTextView

[PhilipDukhov]

I think we should optimize this tho, so that we skip calculating inputTextViewHeight if the middleContentView != inputTextView

you mean caching middleContentView.intrinsicContentSize operation result?

do you think it should be done in this PR?

Even with your approval I don't have access to merge this in

[PhilipDukhov]

@nathantannar4 please check the last comment!