small UI improvements (#2669)

* small UI improvements

* style improvements

* code style

* remove the padding

Co-authored-by: Adeeb Shihadeh <adeebshihadeh@gmail.com>
old-commit-hash: a5132e44f9b8c7f28dee52ae1d00ed0b849a8879

selfdrive/ui/qt/offroad/settings.cc

@@ -42,17 +42,13 @@ ParamsToggle::ParamsToggle(QString param, QString title, QString description, QS
 
   // toggle switch
   Toggle* toggle_switch = new Toggle(this);
-  QSizePolicy switch_policy(QSizePolicy::Preferred, QSizePolicy::Preferred);
-  switch_policy.setHorizontalStretch(1);
-  toggle_switch->setSizePolicy(switch_policy);
-  toggle_switch->setFixedWidth(120);
-  toggle_switch->setFixedHeight(50);
+  toggle_switch->setFixedSize(150, 100);
 
   // TODO: show descriptions on tap
   hlayout->addWidget(label);
   hlayout->addSpacing(50);
   hlayout->addWidget(toggle_switch);
-  hlayout->addSpacing(50);
+  hlayout->addSpacing(20);
 
   setLayout(hlayout);
   if(Params().read_db_bool(param.toStdString().c_str())){
@@ -73,7 +69,7 @@ ParamsToggle::ParamsToggle(QString param, QString title, QString description, QS
     QCheckBox::indicator:checked {
       image: url(../assets/offroad/circled-checkmark.png);
     }
-    QLabel { font-size: 40px }
+    QLabel { font-size: 50px }
     * {
       background-color: #114265;
     }
@@ -206,16 +202,11 @@ QWidget * developer_panel() {
 }
 
 QWidget * network_panel(QWidget * parent) {
-  QVBoxLayout *main_layout = new QVBoxLayout;
   WifiUI *w = new WifiUI();
-  main_layout->addWidget(w);
-
-  QWidget *widget = new QWidget;
-  widget->setLayout(main_layout);
 
   QObject::connect(w, SIGNAL(openKeyboard()), parent, SLOT(closeSidebar()));
   QObject::connect(w, SIGNAL(closeKeyboard()), parent, SLOT(openSidebar()));
-  return widget;
+  return w;
 }
 
 

selfdrive/ui/qt/offroad/wifi.cc

@@ -25,6 +25,7 @@ WifiUI::WifiUI(QWidget *parent, int page_length) : QWidget(parent), networks_per
   QObject::connect(wifi, SIGNAL(wrongPassword(QString)), this, SLOT(wrongPassword(QString)));
 
   QVBoxLayout * top_layout = new QVBoxLayout;
+  top_layout->setSpacing(0);
   swidget = new QStackedWidget;
 
   // Networks page
@@ -51,6 +52,7 @@ WifiUI::WifiUI(QWidget *parent, int page_length) : QWidget(parent), networks_per
   QLabel *scanning = new QLabel("Scanning for networks");
   scanning->setStyleSheet(R"(font-size: 65px;)");
   vlayout->addWidget(scanning, 0, Qt::AlignCenter);
+  vlayout->setSpacing(25);
 
   wifi->request_scan();
   refresh();
@@ -72,6 +74,7 @@ void WifiUI::refresh() {
 
   int i = 0;
   int countWidgets = 0;
+  int button_height = static_cast<int>(this->height() / (networks_per_page + 1) * 0.6);
   for (Network &network : wifi->seen_networks){
     QHBoxLayout *hlayout = new QHBoxLayout;
     if(page * networks_per_page <= i && i < (page + 1) * networks_per_page){
@@ -91,6 +94,7 @@ void WifiUI::refresh() {
       // connect button
       QPushButton* btn = new QPushButton(network.connected == ConnectedType::CONNECTED ? "Connected" : (network.connected == ConnectedType::CONNECTING ? "Connecting" : "Connect"));
       btn->setFixedWidth(300);
+      btn->setFixedHeight(button_height);
       btn->setDisabled(network.connected == ConnectedType::CONNECTED || network.connected == ConnectedType::CONNECTING || network.security_type == SecurityType::UNSUPPORTED);
       hlayout->addWidget(btn);
       hlayout->addSpacing(20);
@@ -106,7 +110,7 @@ void WifiUI::refresh() {
         }
         QPushButton {
           padding: 0;
-          font-size: 40px;
+          font-size: 50px;
           background-color: #114265;
         }
         QPushButton:disabled {
@@ -116,9 +120,9 @@ void WifiUI::refresh() {
           background-color: #114265;
         }
       )");
-      countWidgets += 1;
+      countWidgets++;
     }
-    i += 1;
+    i++;
   }
 
   // Pad vlayout to prevert oversized network widgets in case of low visible network count
@@ -130,9 +134,9 @@ void WifiUI::refresh() {
   QHBoxLayout *prev_next_buttons = new QHBoxLayout;
   QPushButton* prev = new QPushButton("Previous");
   prev->setEnabled(page);
-  prev->setFixedHeight(100);
+  prev->setFixedHeight(button_height);
   QPushButton* next = new QPushButton("Next");
-  next->setFixedHeight(100);
+  next->setFixedHeight(button_height);
 
   // If there are more visible networks then we can show, enable going to next page
   next->setEnabled(wifi->seen_networks.size() > (page + 1) * networks_per_page);

selfdrive/ui/qt/offroad/wifi.hpp

@@ -15,7 +15,7 @@ class WifiUI : public QWidget {
 
 public:
   int page;
-  explicit WifiUI(QWidget *parent = 0, int page_length = 8);
+  explicit WifiUI(QWidget *parent = 0, int page_length = 6);
 
 private:
   WifiManager* wifi;

selfdrive/ui/qt/widgets/toggle.cc

@@ -1,8 +1,8 @@
 #include "toggle.hpp"
 
 Toggle::Toggle(QWidget *parent) : QAbstractButton(parent),
-_height(60),
-_height_rect(45),
+_height(80),
+_height_rect(60),
 _on(false),
 _anim(new QPropertyAnimation(this, "offset_circle", this))
 {