Remove custom behavior from chat window

interface/src/ui/ChatWindow.cpp

@@ -30,16 +30,14 @@
 
 const int NUM_MESSAGES_TO_TIME_STAMP = 20;
 
-const float OPACITY_ACTIVE = 1.0f;
-const float OPACITY_INACTIVE = 0.8f;
-
 const QRegularExpression regexLinks("((?:(?:ftp)|(?:https?)|(?:hifi))://\\S+)");
 const QRegularExpression regexHifiLinks("([#@]\\S+)");
 const QString mentionSoundsPath("/mention-sounds/");
 const QString mentionRegex("@(\\b%1\\b)");
 
 ChatWindow::ChatWindow(QWidget* parent) :
-    FramelessDialog(parent, 0, POSITION_RIGHT),
+    QWidget(parent, Qt::Window | Qt::CustomizeWindowHint | Qt::WindowTitleHint | Qt::WindowMinMaxButtonsHint |
+            Qt::WindowCloseButtonHint),
     ui(new Ui::ChatWindow),
     numMessagesAfterLastTimeStamp(0),
     _mousePressed(false),
@@ -82,7 +80,6 @@ ChatWindow::ChatWindow(QWidget* parent) :
         startTimerForTimeStamps();
     } else {
         ui->numOnlineLabel->hide();
-        ui->closeButton->hide();
         ui->usersArea->hide();
         ui->messagesScrollArea->hide();
         ui->messagePlainTextEdit->hide();
@@ -113,17 +110,21 @@ void ChatWindow::keyPressEvent(QKeyEvent* event) {
     if (event->key() == Qt::Key_Escape) {
         Application::getInstance()->getWindow()->activateWindow();
     } else {
-        FramelessDialog::keyPressEvent(event);
+        QWidget::keyPressEvent(event);
     }
 }
 
 void ChatWindow::showEvent(QShowEvent* event) {
-    FramelessDialog::showEvent(event);
+    QWidget::showEvent(event);
 
     if (!event->spontaneous()) {
         ui->messagePlainTextEdit->setFocus();
     }
 
+    const QRect parentGeometry = parentWidget()->geometry();
+    setGeometry(parentGeometry.topRight().x() - size().width(), parentGeometry.topRight().y(),
+                size().width(), parentWidget()->height());
+
 #ifdef HAVE_QXMPP
     const QXmppClient& xmppClient = XmppClient::getInstance().getXMPPClient();
     if (xmppClient.isConnected()) {
@@ -163,7 +164,7 @@ bool ChatWindow::eventFilter(QObject* sender, QEvent* event) {
             return true;
         }
     }
-    return FramelessDialog::eventFilter(sender, event);
+    return QWidget::eventFilter(sender, event);
 }
 
 void ChatWindow::addTimeStamp() {
@@ -210,7 +211,6 @@ void ChatWindow::startTimerForTimeStamps() {
 void ChatWindow::connected() {
     ui->connectingToXMPPLabel->hide();
     ui->numOnlineLabel->show();
-    ui->closeButton->show();
     ui->usersArea->show();
     ui->messagesScrollArea->show();
     ui->messagePlainTextEdit->show();
@@ -389,9 +389,7 @@ void ChatWindow::scrollToBottom() {
 
 bool ChatWindow::event(QEvent* event) {
     if (event->type() == QEvent::WindowActivate) {
-        setWindowOpacity(OPACITY_ACTIVE);
+        ui->messagePlainTextEdit->setFocus();
-    } else if (event->type() == QEvent::WindowDeactivate) {
-        setWindowOpacity(OPACITY_INACTIVE);
     }
-    return FramelessDialog::event(event);
+    return QWidget::event(event);
 }

interface/src/ui/ChatWindow.h

@@ -38,7 +38,7 @@ const int AUTO_SCROLL_THRESHOLD = 20;
 class ChatWindow;
 }
 
-class ChatWindow : public FramelessDialog {
+class ChatWindow : public QWidget {
     Q_OBJECT
 
 public:

interface/ui/chatWindow.ui

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <ui version="4.0">
  <class>ChatWindow</class>
- <widget class="QDialog" name="ChatWindow">
+ <widget class="QWidget" name="ChatWindow">
   <property name="geometry">
    <rect>
     <x>0</x>
@@ -86,45 +86,6 @@
          </property>
         </widget>
        </item>
-       <item>
-        <widget class="QPushButton" name="closeButton">
-         <property name="sizePolicy">
-          <sizepolicy hsizetype="Fixed" vsizetype="Fixed">
-           <horstretch>0</horstretch>
-           <verstretch>0</verstretch>
-          </sizepolicy>
-         </property>
-         <property name="maximumSize">
-          <size>
-           <width>16</width>
-           <height>16</height>
-          </size>
-         </property>
-         <property name="focusPolicy">
-          <enum>Qt::NoFocus</enum>
-         </property>
-         <property name="styleSheet">
-          <string notr="true">QPushButton {
-    background-color: rgba( 0, 0, 0, 0% );
-	border: none;
-    image: url(../resources/images/close.svg)
-}
-
-
-QPushButton:pressed {
-    background-color: rgba( 0, 0, 0, 0% );
-	border: none;
-    image: url(../resources/images/close_down.svg)
-}</string>
-         </property>
-         <property name="text">
-          <string/>
-         </property>
-         <property name="flat">
-          <bool>true</bool>
-         </property>
-        </widget>
-       </item>
       </layout>
      </item>
      <item>
@@ -283,7 +244,7 @@ border-color: palette(dark); border-style: solid;  border-left-width: 1px; borde
            <enum>QAbstractScrollArea::AdjustToContents</enum>
           </property>
           <property name="tabChangesFocus">
-            <bool>true</bool>
+           <bool>true</bool>
           </property>
           <property name="acceptRichText">
            <bool>false</bool>
@@ -308,22 +269,5 @@ border-color: palette(dark); border-style: solid;  border-left-width: 1px; borde
   <tabstop>messagesScrollArea</tabstop>
  </tabstops>
  <resources/>
- <connections>
+ <connections/>
-  <connection>
-   <sender>closeButton</sender>
-   <signal>clicked()</signal>
-   <receiver>ChatWindow</receiver>
-   <slot>hide()</slot>
-   <hints>
-    <hint type="sourcelabel">
-     <x>390</x>
-     <y>42</y>
-    </hint>
-    <hint type="destinationlabel">
-     <x>550</x>
-     <y>42</y>
-    </hint>
-   </hints>
-  </connection>
- </connections>
 </ui>