settingsprovider.cpp

@@ -1,6 +1,6 @@
 #include "settingsprovider.h"
-#include <QDir>
 #include <QFileInfo>
+#include <QDir>
 
 SettingsProvider::SettingsProvider(QSettings &settings)
 {
@@ -19,12 +19,9 @@ QStringList SettingsProvider::systemApplicationsEntriesPaths() const
 	return settings->value("sysAppsPaths", "/usr/share/applications/").toStringList();
 }
 
-int SettingsProvider::getMaxCols() const
-{
-	return settings->value("maxColumns", 3).toInt();
-}
-
 bool SettingsProvider::singleInstanceMode() const
 {
 	return settings->value("singleInstance", true).toBool();
 }
+
+

settingsprovider.h

@@ -1,19 +1,17 @@
 #ifndef SETTINGSPROVIDER_H
 #define SETTINGSPROVIDER_H
 
-#include <QSettings>
 #include <stdexcept>
+#include <QSettings>
 
 class SettingsProvider
 {
 private:
 	QSettings *settings;
-
 public:
 	SettingsProvider(QSettings &settings);
 	virtual QStringList userEntriesPaths() const;
 	virtual QStringList systemApplicationsEntriesPaths() const;
-	virtual int getMaxCols() const;
 	virtual bool singleInstanceMode() const;
 };
 

window.cpp

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018-2020 Albert S. <mail at quitesimple dot org>
+* Copyright (c) 2018-2019 Albert S. <mail at quitesimple dot org>
 *
 * Permission to use, copy, modify, and distribute this software for any
 * purpose with or without fee is hereby granted, provided that the above
@@ -13,23 +13,22 @@
 * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
 * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 */
-#include <QClipboard>
+#include <QProcess>
-#include <QDate>
+#include <QProcessEnvironment>
-#include <QDebug>
-#include <QDesktopServices>
 #include <QDirIterator>
-#include <QFileIconProvider>
-#include <QHeaderView>
 #include <QIcon>
 #include <QKeySequence>
 #include <QLabel>
+#include <QDate>
+#include <QHeaderView>
+#include <QDesktopServices>
+#include <QFileIconProvider>
+#include <QDebug>
 #include <QMenu>
-#include <QProcess>
+#include <QClipboard>
-#include <QProcessEnvironment>
 #include <QScrollArea>
-
-#include "entryprovider.h"
 #include "window.h"
+#include "entryprovider.h"
 Window::Window(EntryProvider &entryProvider, SettingsProvider &configProvider)
 {
 	this->entryProvider = &entryProvider;
@@ -44,12 +43,13 @@ Window::Window(EntryProvider &entryProvider, SettingsProvider &configProvider)
 	calculationResultLabel.setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Expanding);
 	calculationResultLabel.setAlignment(Qt::AlignCenter);
 	calculationResultLabel.setContextMenuPolicy(Qt::ContextMenuPolicy::CustomContextMenu);
-	connect(&calculationResultLabel, &QLabel::customContextMenuRequested, this,
+	connect(&calculationResultLabel, &QLabel::customContextMenuRequested, this, &Window::showCalculationResultContextMenu);
-			&Window::showCalculationResultContextMenu);
 }
 
 Window::~Window()
 {
+
+
 }
 
 void Window::initFromConfig()
@@ -198,6 +198,7 @@ void Window::addCalcResult(const QString &expression)
 	calculationResultLabel.setText(labelText);
 	calculationResultLabel.setVisible(true);
 
+
 	QFont currentFont = calculationResultLabel.font();
 	int calculatedPointSize = currentFont.pointSize();
 	QFontMetrics fm(currentFont);
@@ -267,11 +268,11 @@ void Window::keyReleaseEvent(QKeyEvent *event)
 		}
 	}
 	QWidget::keyReleaseEvent(event);
+
 }
 void Window::keyPressEvent(QKeyEvent *event)
 {
-	bool closeWindow =
+	bool closeWindow = ((event->modifiers() & Qt::ControlModifier && event->key() == Qt::Key_Q) || event->key() == Qt::Key_Escape);
-		((event->modifiers() & Qt::ControlModifier && event->key() == Qt::Key_Q) || event->key() == Qt::Key_Escape);
 	if(closeWindow)
 	{
 		this->closeWindow();
@@ -293,8 +294,7 @@ void Window::keyPressEvent(QKeyEvent *event)
 		QKeySequence seq(event->key());
 		QString key = seq.toString().toLower();
 
-		auto it = std::find_if(buttonsInGrid.begin(), buttonsInGrid.end(),
+		auto it = std::find_if(buttonsInGrid.begin(), buttonsInGrid.end(), [&key](const EntryPushButton *y) { return y->getShortcutKey() == key; });
-							   [&key](const EntryPushButton *y) { return y->getShortcutKey() == key; });
 		if(it != buttonsInGrid.end())
 		{
 			buttonClick(**it);
@@ -324,7 +324,6 @@ void Window::filterGridFor(QString filter)
 			int currow = 0;
 			int curcol = 0;
 			int i = 1;
-			const int MAX_COLS = this->settingsProvider->getMaxCols();
 			for(EntryPushButton *button : this->systemEntryButtons)
 			{
 				if(button->getName().contains(filter, Qt::CaseInsensitive))
@@ -336,7 +335,7 @@ void Window::filterGridFor(QString filter)
 					}
 					grid->addWidget(button, currow, curcol++);
 					this->buttonsInGrid.append(button);
-					if(curcol == MAX_COLS)
+					if(curcol == 3)
 					{
 						curcol = 0;
 						++currow;
@@ -344,11 +343,15 @@ void Window::filterGridFor(QString filter)
 				}
 			}
 		}
+
+
+
 	}
 	else
 	{
 		populateGrid(this->userEntryButtons);
 	}
+
 }
 
 EntryPushButton * Window::createEntryButton(const EntryConfig &entry)
@@ -367,6 +370,7 @@ void Window::lineEditReturnPressed()
 		return;
 	}
 
+
 	if(buttonsInGrid.length() > 0 && this->lineEdit->text().length() > 0 )
 	{
 		buttonClick(*buttonsInGrid[0]);
@@ -397,6 +401,7 @@ bool Window::eventFilter(QObject *obj, QEvent *event)
 				return true;
 			}
 		}
+
 	}
 	return QObject::eventFilter(obj, event);
 }