commit 223bcc8cb36667eefb601ded9cec6acca2d0f685
author Olivier SOLDANO <olivier.soldano@savoirfairelinux.com> Wed Jul 19 16:05:05 2017 -0400
committer Guillaume Roguez <guillaume.roguez@savoirfairelinux.com> Thu Jul 20 11:59:07 2017 -0400
tree 5dcd3fc617098ae5403c92416e368d1875d2d3dd
parent 8179e996cdc7a1777afd8a8cf8ff8c906ef1d70f

change video overlay buttons if fullscreen

the buttons back and config were shown on the overlay
in full screen. However due the the preemption of the full screen
mode, they seem to have no effect for the user, but changes the page
in background.
Thus when in full screen, these buttons are hidden.

Change-Id: Iba0bd7f5630523a7f56a932bc96cf0eb583c4266
Reviewed-by: Guillaume Roguez <guillaume.roguez@savoirfairelinux.com>

videooverlay.cpp

diff --git a/videooverlay.cpp b/videooverlay.cpp
index 9e30be7..8643b07 100644
--- a/videooverlay.cpp
+++ b/videooverlay.cpp
@@ -19,12 +19,16 @@
 #include "videooverlay.h"
 #include "ui_videooverlay.h"
 
+// Client
 #include "contactpicker.h"
 
+// LRC
 #include "callmodel.h"
 #include "contactmethod.h"
 #include "person.h"
 
+
+
 VideoOverlay::VideoOverlay(QWidget* parent) :
     QWidget(parent),
     ui(new Ui::VideoOverlay),
@@ -101,6 +105,9 @@
     connect(qualityDialog_, &QualityDialog::isVisible, [this] (bool visible) {
         dialogVisible_ = visible;
     });
+
+    // temporary hide
+    ui->addPersonButton->hide();
 }
 
 VideoOverlay::~VideoOverlay()
@@ -122,6 +129,17 @@
     ui->timerLabel->setText(time);
 }
 
+void VideoOverlay::toggleContextButtons(bool visible)
+{
+    if (! visible) {
+        ui->videoCfgBtn->hide();
+        ui->videoBackBtn->hide();
+    } else {
+        ui->videoCfgBtn->show();
+        ui->videoBackBtn->show();
+    }
+}
+
 void
 VideoOverlay::on_hangupButton_clicked()
 {

videooverlay.h

diff --git a/videooverlay.h b/videooverlay.h
index c525a3a..280b13c 100644
--- a/videooverlay.h
+++ b/videooverlay.h
@@ -42,6 +42,7 @@
     void setName(const QString& name);
     void setTime(const QString& time);
     inline bool isDialogVisible(){ return dialogVisible_; };
+    void toggleContextButtons(bool visible);
 
 //UI SLOTS
 private slots:

videoview.cpp

diff --git a/videoview.cpp b/videoview.cpp
index 46d069f..511d0be 100644
--- a/videoview.cpp
+++ b/videoview.cpp
@@ -185,6 +185,7 @@
 void
 VideoView::toggleFullScreen()
 {
+    overlay_->toggleContextButtons(isFullScreen());
     if(isFullScreen()) {
         dynamic_cast<QSplitter*>(oldParent_)->insertWidget(0,this);
         this->resize(oldSize_.width(), oldSize_.height());