From c1fa6b41e45bb9d4442afc2eee10cdded63ef919 Mon Sep 17 00:00:00 2001
From: Ryan Huffman <ryanhuffman@gmail.com>
Date: Thu, 17 Apr 2014 13:19:36 -0700
Subject: [PATCH] Update chat window to use vertical layout

---
 interface/src/ui/ChatWindow.cpp | 55 +++++++++------------------------
 interface/ui/chatWindow.ui      | 10 +++---
 2 files changed, 18 insertions(+), 47 deletions(-)

diff --git a/interface/src/ui/ChatWindow.cpp b/interface/src/ui/ChatWindow.cpp
index 2849d0c748..ae5dd0185c 100644
--- a/interface/src/ui/ChatWindow.cpp
+++ b/interface/src/ui/ChatWindow.cpp
@@ -44,9 +44,6 @@ ChatWindow::ChatWindow(QWidget* parent) :
     FlowLayout* flowLayout = new FlowLayout(0, 4, 4);
     ui->usersWidget->setLayout(flowLayout);
 
-    ui->messagesGridLayout->setColumnStretch(0, 1);
-    ui->messagesGridLayout->setColumnStretch(1, 3);
-
     ui->messagePlainTextEdit->installEventFilter(this);
     ui->messagePlainTextEdit->setWordWrapMode(QTextOption::WrapAtWordBoundaryOrAnywhere);
 
@@ -172,8 +169,8 @@ void ChatWindow::addTimeStamp() {
 
         bool atBottom = isAtBottom();
 
-        ui->messagesGridLayout->addWidget(timeLabel, ui->messagesGridLayout->rowCount(), 0, 1, 2);
-        ui->messagesGridLayout->parentWidget()->updateGeometry();
+        ui->messagesVBoxLayout->addWidget(timeLabel);
+        ui->messagesVBoxLayout->parentWidget()->updateGeometry();
 
         Application::processEvents();
         numMessagesAfterLastTimeStamp = 0;
@@ -251,29 +248,8 @@ void ChatWindow::messageReceived(const QXmppMessage& message) {
         return;
     }
 
-    // Create username label
-    ChatMessageArea* userLabel = new ChatMessageArea(false);
-    userLabel->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Maximum);
-    userLabel->setWordWrapMode(QTextOption::NoWrap);
-    userLabel->setLineWrapMode(QTextEdit::NoWrap);
-    userLabel->setTextInteractionFlags(Qt::NoTextInteraction);
-    userLabel->setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
-    userLabel->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
-    userLabel->setReadOnly(true);
-    userLabel->setSizeAdjustPolicy(QAbstractScrollArea::AdjustToContents);
-
-    userLabel->setStyleSheet("padding: 2px;"
-                             "font-weight: bold;"
-                             "background-color: rgba(0, 0, 0, 0%);"
-                             "border: 0;");
-
-    QTextBlockFormat format;
-    format.setLineHeight(130, QTextBlockFormat::ProportionalHeight);
-    QTextCursor cursor = userLabel->textCursor();
-    cursor.setBlockFormat(format);
-    cursor.insertText(getParticipantName(message.from()));
-
-    userLabel->setAlignment(Qt::AlignRight);
+    // Update background if this is a message from the current user
+    bool fromSelf = getParticipantName(message.from()) == AccountManager::getInstance().getUsername();
 
     // Create message area
     ChatMessageArea* messageArea = new ChatMessageArea(true);
@@ -289,30 +265,27 @@ void ChatWindow::messageReceived(const QXmppMessage& message) {
                                "padding-left: 2px;"
                                "padding-top: 2px;"
                                "padding-right: 20px;"
+                               "margin: 0px;"
+                               "color: #333333;"
+                               "font-size: 14pt;"
                                "background-color: rgba(0, 0, 0, 0%);"
                                "border: 0;");
 
-    // Update background if this is a message from the current user
-    bool fromSelf = getParticipantName(message.from()) == AccountManager::getInstance().getUsername();
+    QString userLabel = getParticipantName(message.from());
     if (fromSelf) {
-        userLabel->setStyleSheet(userLabel->styleSheet() + "background-color: #e1e8ea");
+        userLabel = "<b style=\"color: #4a6f91\">" + userLabel + ": </b>";
         messageArea->setStyleSheet(messageArea->styleSheet() + "background-color: #e1e8ea");
+    } else {
+        userLabel = "<b>" + userLabel + ": </b>";
     }
 
     messageArea->document()->setDefaultStyleSheet("a { text-decoration: none; font-weight: bold; color: #267077;}");
-    messageArea->setHtml(message.body().replace(regexLinks, "<a href=\"\\1\">\\1</a>"));
+    messageArea->setHtml(userLabel + message.body().replace(regexLinks, "<a href=\"\\1\">\\1</a>"));
 
     bool atBottom = isAtBottom();
-    ui->messagesGridLayout->addWidget(userLabel, ui->messagesGridLayout->rowCount(), 0);
-    ui->messagesGridLayout->addWidget(messageArea, ui->messagesGridLayout->rowCount() - 1, 1);
 
-    // Force the height of the username area to match the height of the message area
-    connect(messageArea, &ChatMessageArea::sizeChanged, userLabel, &ChatMessageArea::setSize);
-
-    // Force initial height to match message area
-    userLabel->setFixedHeight(messageArea->size().height());
-
-    ui->messagesGridLayout->parentWidget()->updateGeometry();
+    ui->messagesVBoxLayout->addWidget(messageArea);
+    ui->messagesVBoxLayout->parentWidget()->updateGeometry();
     Application::processEvents();
 
     if (atBottom || fromSelf) {
diff --git a/interface/ui/chatWindow.ui b/interface/ui/chatWindow.ui
index 78757ded38..b516418f8c 100644
--- a/interface/ui/chatWindow.ui
+++ b/interface/ui/chatWindow.ui
@@ -159,7 +159,10 @@ QPushButton:pressed {
         <property name="styleSheet">
          <string notr="true">margin-top: 0px;</string>
         </property>
-        <layout class="QGridLayout" name="messagesGridLayout">
+        <layout class="QVBoxLayout" name="messagesVBoxLayout">
+         <property name="spacing">
+          <number>0</number>
+         </property>
          <property name="leftMargin">
           <number>0</number>
          </property>
@@ -172,9 +175,6 @@ QPushButton:pressed {
          <property name="bottomMargin">
           <number>0</number>
          </property>
-         <property name="spacing">
-          <number>0</number>
-         </property>
         </layout>
        </widget>
       </widget>
@@ -213,8 +213,6 @@ QPushButton:pressed {
     </layout>
    </item>
   </layout>
-  <zorder>messagePlainTextEdit</zorder>
-  <zorder>dockWidgetContents</zorder>
  </widget>
  <tabstops>
   <tabstop>messagePlainTextEdit</tabstop>