From 1d26b879c9b4d356f1bf48216192f5e156d5570f Mon Sep 17 00:00:00 2001
From: Simon Rettberg
Date: Thu, 6 Sep 2018 13:37:34 +0200
Subject: [client] Fix style if checked button; add separator for buttons

---
 src/client/toolbar/toolbar.cpp | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

(limited to 'src/client/toolbar/toolbar.cpp')

diff --git a/src/client/toolbar/toolbar.cpp b/src/client/toolbar/toolbar.cpp
index bec8586..d5e7d08 100644
--- a/src/client/toolbar/toolbar.cpp
+++ b/src/client/toolbar/toolbar.cpp
@@ -138,6 +138,13 @@ void Toolbar::init()
 	connect(&_blinkTimer, SIGNAL(timeout()), this, SLOT(cameraBlink()));
 }
 
+static QFrame* makeVerticalLine() {
+	QFrame *f = new QFrame();
+	f->setFrameShape(QFrame::HLine);
+	f->setFrameShadow(QFrame::Sunken);
+	return f;
+}
+
 /**
  * This function should be called once from the main init() function which in
  * turn should only be called by the constructor.
@@ -149,9 +156,13 @@ void Toolbar::initButtonsAndMenus()
 	auto settings = clientApp->getSettings();
 	AddonManager::loadFromPath(settings->value("addonConfigDir", "/opt/openslx/pvs2/addons").toString(), buttons, menus);
 	// Buttons
-	for (auto i : buttons) {
-		_ui->buttonContainer->addWidget(i);
-		i->setVisible(false);
+	if (!buttons.isEmpty()) {
+		_ui->buttonContainer->addWidget(makeVerticalLine());
+		for (auto i : buttons) {
+			_ui->buttonContainer->addWidget(i);
+			i->setVisible(false);
+		}
+		_ui->buttonContainer->addWidget(makeVerticalLine());
 	}
 	// Menu
 	_menu = new QMenu(this);
@@ -176,7 +187,6 @@ void Toolbar::initButtonsAndMenus()
 	_ui->cmdMenu->setMenu(_menu);
 
 	/* only add a "quit"-button when the configuration allows it. */
-
 	bool allow = settings->value("allowClientQuit").toBool();
 	_acnQuit->setVisible(allow);
 
-- 
cgit v1.2.3-55-g7522