From c0b1518a9b8719ac6636a7c8a69b628729f19b7b Mon Sep 17 00:00:00 2001
From: Adeeb Shihadeh <adeebshihadeh@gmail.com>
Date: Sun, 20 Dec 2020 21:17:59 -0800
Subject: [PATCH] Qt settings: highlight selected nav button

---
 selfdrive/ui/qt/offroad/settings.cc  | 19 +++++++++++++------
 selfdrive/ui/qt/offroad/settings.hpp |  2 ++
 2 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/selfdrive/ui/qt/offroad/settings.cc b/selfdrive/ui/qt/offroad/settings.cc
index 8f3b2b71b5..f37d0e9309 100644
--- a/selfdrive/ui/qt/offroad/settings.cc
+++ b/selfdrive/ui/qt/offroad/settings.cc
@@ -213,12 +213,12 @@ QWidget * network_panel(QWidget * parent) {
 
 
 void SettingsWindow::setActivePanel() {
-  QPushButton *btn = qobject_cast<QPushButton*>(sender());
+  auto *btn = qobject_cast<QPushButton *>(nav_btns->checkedButton());
   panel_layout->setCurrentWidget(panels[btn->text()]);
 }
 
 SettingsWindow::SettingsWindow(QWidget *parent) : QWidget(parent) {
-  // sidebar
+  // two main layouts
   QVBoxLayout *sidebar_layout = new QVBoxLayout();
   panel_layout = new QStackedLayout();
 
@@ -228,7 +228,7 @@ SettingsWindow::SettingsWindow(QWidget *parent) : QWidget(parent) {
     QPushButton {
       padding: 50px;
       font-weight: bold;
-      font-size: 100px;
+      font-size: 110px;
     }
   )");
   sidebar_layout->addWidget(close_button);
@@ -242,26 +242,33 @@ SettingsWindow::SettingsWindow(QWidget *parent) : QWidget(parent) {
     {"toggles", toggles_panel()},
   };
 
+  nav_btns = new QButtonGroup();
   for (auto &panel : panels) {
     QPushButton *btn = new QPushButton(panel.first);
+    btn->setCheckable(true);
     btn->setStyleSheet(R"(
       QPushButton {
         padding-top: 35px;
         padding-bottom: 35px;
-        font-size: 60px;
-        text-align: right;
         border: none;
         background: none;
+        font-size: 55px;
+      }
+      QPushButton:checked {
+        font-size: 50px;
         font-weight: bold;
       }
     )");
 
-    sidebar_layout->addWidget(btn);
+    nav_btns->addButton(btn);
+    sidebar_layout->addWidget(btn, 0, Qt::AlignRight);
     panel_layout->addWidget(panel.second);
     QObject::connect(btn, SIGNAL(released()), this, SLOT(setActivePanel()));
   }
+  qobject_cast<QPushButton *>(nav_btns->buttons()[0])->setChecked(true);
 
   QHBoxLayout *settings_layout = new QHBoxLayout();
+  settings_layout->setMargin(0);
   settings_layout->addSpacing(45);
 
   sidebar_widget = new QWidget;
diff --git a/selfdrive/ui/qt/offroad/settings.hpp b/selfdrive/ui/qt/offroad/settings.hpp
index b2c0a911d7..2dc375c528 100644
--- a/selfdrive/ui/qt/offroad/settings.hpp
+++ b/selfdrive/ui/qt/offroad/settings.hpp
@@ -4,6 +4,7 @@
 #include <QFrame>
 #include <QTimer>
 #include <QPushButton>
+#include <QButtonGroup>
 #include <QStackedLayout>
 
 #include "wifi.hpp"
@@ -40,6 +41,7 @@ private:
   QPushButton *sidebar_alert_widget;
   QWidget *sidebar_widget;
   std::map<QString, QWidget *> panels;
+  QButtonGroup *nav_btns;
   QStackedLayout *panel_layout;
 
 public slots: