From 12fef4bf8b3ad91de47afc5ce21a0023aae7cd81 Mon Sep 17 00:00:00 2001
From: Angela Mazzurco <giangiasoc17@gmail.com>
Date: Sat, 10 Jun 2017 16:02:09 +0200
Subject: [PATCH] Refactor to add message text area into a flickable

---
 retroshare-qml-app/src/ChatView.qml           | 60 ++++++++++++-------
 .../src/styles/ChatBubblesStyle.qml           |  2 +-
 2 files changed, 38 insertions(+), 24 deletions(-)

diff --git a/retroshare-qml-app/src/ChatView.qml b/retroshare-qml-app/src/ChatView.qml
index e921d0f79..c91b13c14 100644
--- a/retroshare-qml-app/src/ChatView.qml
+++ b/retroshare-qml-app/src/ChatView.qml
@@ -117,6 +117,7 @@ Item
 
 		RowLayout {
 			id: msgComposer
+			property var styles: StyleChat.inferiorPanel.msgComposer
 
 			anchors.verticalCenter: parent.verticalCenter
 			anchors.left: attachButton.right
@@ -126,36 +127,49 @@ Item
 				   (attachButton.width + attachButton.anchors.margins) -
 				   (emojiButton.width + emojiButton.anchors.margins)
 
+			height: (flickable.contentHeight <  styles.maxHeight)? flickable.contentHeight : styles.maxHeight
 
-			TextArea
-			{
-				property var styles: StyleChat.inferiorPanel.msgComposer
-				id: msgField
-
-				height:
-					(contentHeight > font.pixelSize)?
-						contentHeight +font.pixelSize : parent.styles.height
+			Flickable {
+				id: flickable
+				anchors.fill: parent
+				flickableDirection: Flickable.VerticalFlick
 
 				width: parent.width
 
-				Layout.maximumHeight: chatView.height / styles.maxHeight
+				contentWidth: msgField.width
+				contentHeight: msgField.height
+				contentY: contentHeight - height
 
-				placeholderText: styles.placeHolder
-				background: styles.background
+				ScrollBar.vertical: ScrollBar {}
 
-				wrapMode: TextEdit.Wrap
+				clip: true
 
-				onTextChanged: {
-					if (msgComposer.length == 0)
-					{
-						sendButton.state = ""
-					}
-					else if (msgComposer.length > 0)
-					{
-						sendButton.state = "SENDBTN"
+				TextArea
+				{
+					property var styles: StyleChat.inferiorPanel.msgComposer
+					id: msgField
+
+					height: contentHeight + font.pixelSize
+
+					width: parent.width
+
+					placeholderText: styles.placeHolder
+					background: styles.background
+
+					wrapMode: TextEdit.Wrap
+
+					onTextChanged: {
+						if (msgField.length == 0)
+						{
+							sendButton.state = ""
+						}
+						else if (msgField.length > 0)
+						{
+							sendButton.state = "SENDBTN"
+						}
 					}
+
 				}
-
 			}
 
 		}
@@ -199,9 +213,9 @@ Item
 			onClicked:
 			{
 				if (sendButton.state == "SENDBTN" ) {
-					var jsonData = {"chat_id":chatView.chatId, "msg":msgComposer.text}
+					var jsonData = {"chat_id":chatView.chatId, "msg":msgField.text}
 					rsApi.request( "/chat/send_message", JSON.stringify(jsonData),
-								   function(par) { msgComposer.text = ""; } )
+								   function(par) { msgField.text = ""; } )
 				}
 			}
 
diff --git a/retroshare-qml-app/src/styles/ChatBubblesStyle.qml b/retroshare-qml-app/src/styles/ChatBubblesStyle.qml
index dd5246bf7..84216a487 100644
--- a/retroshare-qml-app/src/styles/ChatBubblesStyle.qml
+++ b/retroshare-qml-app/src/styles/ChatBubblesStyle.qml
@@ -34,7 +34,7 @@ QtObject {
 
 		property QtObject msgComposer: QtObject{
 			readonly property string placeHolder: "Send message..."
-			readonly property int maxHeight: 2 // chatListView/maxHeight
+			readonly property int maxHeight: 180
 
 			property QtObject background: Rectangle {
 				color: "transparent"