configdialog: Sets the page layout margin to 0
authorLuís Pereira <luis.artur.pereira@gmail.com>
Tue, 7 Jul 2015 22:38:57 +0000 (15:38 -0700)
committerLuís Pereira <luis.artur.pereira@gmail.com>
Tue, 7 Jul 2015 22:38:57 +0000 (15:38 -0700)
We set the layout margin to 0. In the default configuration, one page
only, it aligns buttons with the page. In multi-page it saves a little
bit of space, without clutter.

It's probably because of this little "bug" that ConfigDialog has little
usage.

configdialog/lxqtconfigdialog.cpp

index 86f3646..da98a5a 100644 (file)
@@ -59,6 +59,15 @@ void ConfigDialog::addPage(QWidget* page, const QString& name, const QString& ic
 
 void ConfigDialog::addPage(QWidget* page, const QString& name, const QStringList& iconNames)
 {
+    /* We set the layout margin to 0. In the default configuration, one page
+     *  only, it aligns buttons with the page. In multi-page it saves a little
+     *  bit of space, without clutter.
+     */
+    if (page->layout())
+    {
+        page->layout()->setMargin(0);
+    }
+
     QStringList icons = QStringList(iconNames) << "application-x-executable";
     new QListWidgetItem(XdgIcon::fromTheme(icons), name, ui->moduleList);
     mIcons.append(icons);