[LyX/master] Remove Qt < 5.2 conditionals and special code
Juergen Spitzmueller
spitz at lyx.org
Fri Aug 18 05:49:52 UTC 2023
commit 5c8e3a5f0d637974a0ac321d1f729c5ead2c1717
Author: Juergen Spitzmueller <spitz at lyx.org>
Date: Fri Aug 18 09:04:12 2023 +0200
Remove Qt < 5.2 conditionals and special code
We require Qt 5.2 at least for LyX 2.4
---
src/frontends/qt/FancyLineEdit.cpp | 4 -
src/frontends/qt/GuiApplication.cpp | 17 +-----
src/frontends/qt/GuiApplication.h | 2 -
src/frontends/qt/GuiFontMetrics.cpp | 4 -
src/frontends/qt/GuiProgress.cpp | 32 ----------
src/frontends/qt/GuiView.cpp | 2 -
src/frontends/qt/GuiWorkArea.cpp | 2 -
src/frontends/qt/Makefile.am | 1 -
src/frontends/qt/ui/ToggleWarningUi.ui | 103 --------------------------------
9 files changed, 2 insertions(+), 165 deletions(-)
diff --git a/src/frontends/qt/FancyLineEdit.cpp b/src/frontends/qt/FancyLineEdit.cpp
index 570d6cc..269a717 100644
--- a/src/frontends/qt/FancyLineEdit.cpp
+++ b/src/frontends/qt/FancyLineEdit.cpp
@@ -15,10 +15,6 @@
#include "FancyLineEdit.h"
-#if QT_VERSION < 0x050200
-#include "GuiApplication.h"
-#endif
-
#include <QEvent>
#include <QDebug>
#include <QString>
diff --git a/src/frontends/qt/GuiApplication.cpp b/src/frontends/qt/GuiApplication.cpp
index 165d140..a237242 100644
--- a/src/frontends/qt/GuiApplication.cpp
+++ b/src/frontends/qt/GuiApplication.cpp
@@ -1028,10 +1028,7 @@ public:
struct GuiApplication::Private
{
Private(): language_model_(nullptr), meta_fake_bit(NoModifier),
- global_menubar_(nullptr)
- #if (QT_VERSION >= QT_VERSION_CHECK(5, 1, 0))
- , last_state_(Qt::ApplicationInactive)
- #endif
+ global_menubar_(nullptr), last_state_(Qt::ApplicationInactive)
{
#if (QT_VERSION >= 0x050400)
#if defined(Q_OS_WIN) || defined(Q_CYGWIN_WIN)
@@ -1112,10 +1109,8 @@ struct GuiApplication::Private
/// Only used on mac.
QMenuBar * global_menubar_;
-#if (QT_VERSION >= QT_VERSION_CHECK(5, 1, 0))
/// Holds previous application state on Mac
Qt::ApplicationState last_state_;
-#endif
#if defined(Q_OS_MAC) && (QT_VERSION < 0x060000)
/// Linkback mime handler for MacOSX.
@@ -1156,7 +1151,7 @@ GuiApplication::GuiApplication(int & argc, char ** argv)
QCoreApplication::setOrganizationName(app_name);
QCoreApplication::setOrganizationDomain("lyx.org");
QCoreApplication::setApplicationName(lyx_package);
-#if QT_VERSION >= 0x050100 && QT_VERSION < 0x060000
+#if QT_VERSION < 0x060000
QCoreApplication::setAttribute(Qt::AA_UseHighDpiPixmaps);
#endif
@@ -1195,11 +1190,9 @@ GuiApplication::GuiApplication(int & argc, char ** argv)
setupApplescript();
appleCleanupEditMenu();
appleCleanupViewMenu();
-#if (QT_VERSION >= QT_VERSION_CHECK(5, 1, 0))
connect(this, SIGNAL(applicationStateChanged(Qt::ApplicationState)),
this, SLOT(onApplicationStateChanged(Qt::ApplicationState)));
#endif
-#endif
#if defined(QPA_XCB)
// doubleClickInterval() is 400 ms on X11 which is just too long.
@@ -2954,11 +2947,7 @@ namespace {
QFont const GuiApplication::typewriterSystemFont()
{
-#if QT_VERSION >= 0x050200
QFont font = QFontDatabase::systemFont(QFontDatabase::FixedFont);
-#else
- QFont font("monospace");
-#endif
if (!isFixedPitch(font)) {
// try to enforce a real monospaced font
font.setStyleHint(QFont::Monospace);
@@ -3452,7 +3441,6 @@ void GuiApplication::onLastWindowClosed()
}
-#if (QT_VERSION >= QT_VERSION_CHECK(5, 1, 0))
void GuiApplication::onApplicationStateChanged(Qt::ApplicationState state)
{
std::string name = "unknown";
@@ -3479,7 +3467,6 @@ void GuiApplication::onApplicationStateChanged(Qt::ApplicationState state)
LYXERR(Debug::GUI, "onApplicationStateChanged..." << name);
d->last_state_ = state;
}
-#endif
void GuiApplication::startLongOperation() {
diff --git a/src/frontends/qt/GuiApplication.h b/src/frontends/qt/GuiApplication.h
index 3eab5ec..a77b0c7 100644
--- a/src/frontends/qt/GuiApplication.h
+++ b/src/frontends/qt/GuiApplication.h
@@ -226,9 +226,7 @@ private Q_SLOTS:
///
void slotProcessFuncRequestQueue() { processFuncRequestQueue(); }
///
-#if (QT_VERSION >= 0x050100)
void onApplicationStateChanged(Qt::ApplicationState state);
-#endif
private:
///
diff --git a/src/frontends/qt/GuiFontMetrics.cpp b/src/frontends/qt/GuiFontMetrics.cpp
index 0e69ffd..631aecc 100644
--- a/src/frontends/qt/GuiFontMetrics.cpp
+++ b/src/frontends/qt/GuiFontMetrics.cpp
@@ -30,11 +30,7 @@
#include <QRawFont>
#include <QtEndian>
-#if QT_VERSION >= 0x050100
#include <QtMath>
-#else
-#define qDegreesToRadians(degree) (degree) * (M_PI / 180)
-#endif
using namespace std;
using namespace lyx::support;
diff --git a/src/frontends/qt/GuiProgress.cpp b/src/frontends/qt/GuiProgress.cpp
index a21baf3..80ba8ae 100644
--- a/src/frontends/qt/GuiProgress.cpp
+++ b/src/frontends/qt/GuiProgress.cpp
@@ -32,22 +32,6 @@ namespace lyx {
namespace frontend {
-// This dialog is only a fallback for Qt < 5.2, which does not feature
-// QMessageBox::setCheckBox() yet. Note that it has issues with line
-// breaking and size, in particular with html.
-#if QT_VERSION < 0x050200
-class GuiToggleWarningDialog : public QDialog, public Ui::ToggleWarningUi
-{
-public:
- GuiToggleWarningDialog(QWidget * parent) : QDialog(parent)
- {
- Ui::ToggleWarningUi::setupUi(this);
- QDialog::setModal(true);
- }
-};
-#endif
-
-
GuiProgress::GuiProgress()
{
connect(this, SIGNAL(processStarted(QString const &)), SLOT(doProcessStarted(QString const &)));
@@ -174,21 +158,6 @@ void GuiProgress::doToggleWarning(QString const & title, QString const & msg, QS
if (settings.value("hidden_warnings/" + msg, false).toBool())
return;
-// Qt < 5.2 does not feature QMessageBox::setCheckBox() yet,
-// so we roll our own dialog.
-#if QT_VERSION < 0x050200
- GuiToggleWarningDialog * dlg =
- new GuiToggleWarningDialog(qApp->focusWidget());
-
- dlg->setWindowTitle(title);
- dlg->messageLA->setText(formatted);
- dlg->dontShowAgainCB->setChecked(false);
-
- if (dlg->exec() == QDialog::Accepted)
- if (dlg->dontShowAgainCB->isChecked())
- settings.setValue("hidden_warnings/"
- + msg, true);
-#else
QCheckBox * dontShowAgainCB = new QCheckBox();
dontShowAgainCB->setText(qt_("&Do not show this warning again!"));
dontShowAgainCB->setToolTip(qt_("If you check this, LyX will not warn you again in the given case."));
@@ -199,7 +168,6 @@ void GuiProgress::doToggleWarning(QString const & title, QString const & msg, QS
if (dontShowAgainCB->isChecked())
settings.setValue("hidden_warnings/"
+ msg, true);
-#endif
}
diff --git a/src/frontends/qt/GuiView.cpp b/src/frontends/qt/GuiView.cpp
index ddf7e8d..2d8847e 100644
--- a/src/frontends/qt/GuiView.cpp
+++ b/src/frontends/qt/GuiView.cpp
@@ -1122,11 +1122,9 @@ void GuiView::updateLockToolbars()
if (tb && tb->isMovable())
toolbarsMovable_ = true;
}
-#if QT_VERSION >= 0x050200
// set unified mac toolbars only when not movable as recommended:
// https://doc.qt.io/qt-5/qmainwindow.html#unifiedTitleAndToolBarOnMac-prop
setUnifiedTitleAndToolBarOnMac(!toolbarsMovable_);
-#endif
}
diff --git a/src/frontends/qt/GuiWorkArea.cpp b/src/frontends/qt/GuiWorkArea.cpp
index ef91e0b..099b129 100644
--- a/src/frontends/qt/GuiWorkArea.cpp
+++ b/src/frontends/qt/GuiWorkArea.cpp
@@ -1503,11 +1503,9 @@ TabWorkArea::TabWorkArea(QWidget * parent)
QObject::connect(this, SIGNAL(currentChanged(int)),
this, SLOT(on_currentTabChanged(int)));
-#if QT_VERSION >= 0x050200
// Fix for #11835
QObject::connect(this, SIGNAL(tabBarClicked(int)),
this, SLOT(on_currentTabChanged(int)));
-#endif
closeBufferButton = new QToolButton(this);
closeBufferButton->setPalette(pal);
diff --git a/src/frontends/qt/Makefile.am b/src/frontends/qt/Makefile.am
index 9ca258d..c038e3f 100644
--- a/src/frontends/qt/Makefile.am
+++ b/src/frontends/qt/Makefile.am
@@ -351,7 +351,6 @@ UIFILES = \
TextLayoutUi.ui \
ThesaurusUi.ui \
TocUi.ui \
- ToggleWarningUi.ui \
ViewSourceUi.ui \
VSpaceUi.ui \
WorkAreaUi.ui \
diff --git a/src/frontends/qt/ui/ToggleWarningUi.ui b/src/frontends/qt/ui/ToggleWarningUi.ui
deleted file mode 100644
index 1b8b746..0000000
--- a/src/frontends/qt/ui/ToggleWarningUi.ui
+++ /dev/null
@@ -1,103 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<ui version="4.0">
- <class>ToggleWarningUi</class>
- <widget class="QDialog" name="ToggleWarningUi">
- <property name="geometry">
- <rect>
- <x>0</x>
- <y>0</y>
- <width>269</width>
- <height>140</height>
- </rect>
- </property>
- <property name="windowTitle">
- <string>Enter text</string>
- </property>
- <property name="sizeGripEnabled">
- <bool>true</bool>
- </property>
- <layout class="QGridLayout" name="gridLayout">
- <item row="0" column="0" colspan="2">
- <spacer>
- <property name="orientation">
- <enum>Qt::Vertical</enum>
- </property>
- <property name="sizeType">
- <enum>QSizePolicy::Expanding</enum>
- </property>
- <property name="sizeHint" stdset="0">
- <size>
- <width>247</width>
- <height>16</height>
- </size>
- </property>
- </spacer>
- </item>
- <item row="1" column="0">
- <widget class="QLabel" name="messageLA">
- <property name="text">
- <string/>
- </property>
- </widget>
- </item>
- <item row="2" column="0" colspan="2">
- <spacer>
- <property name="orientation">
- <enum>Qt::Vertical</enum>
- </property>
- <property name="sizeType">
- <enum>QSizePolicy::Expanding</enum>
- </property>
- <property name="sizeHint" stdset="0">
- <size>
- <width>247</width>
- <height>16</height>
- </size>
- </property>
- </spacer>
- </item>
- <item row="3" column="0" colspan="2">
- <widget class="QCheckBox" name="dontShowAgainCB">
- <property name="toolTip">
- <string>If you check this, LyX will not warn you again in the given case.</string>
- </property>
- <property name="text">
- <string>&Do not show this warning again!</string>
- </property>
- </widget>
- </item>
- <item row="4" column="0" colspan="2">
- <widget class="QDialogButtonBox" name="buttonBox">
- <property name="standardButtons">
- <set>QDialogButtonBox::Ok</set>
- </property>
- <property name="centerButtons">
- <bool>true</bool>
- </property>
- </widget>
- </item>
- </layout>
- </widget>
- <includes>
- <include location="local">qt_i18n.h</include>
- </includes>
- <resources/>
- <connections>
- <connection>
- <sender>buttonBox</sender>
- <signal>accepted()</signal>
- <receiver>ToggleWarningUi</receiver>
- <slot>accept()</slot>
- <hints>
- <hint type="sourcelabel">
- <x>98</x>
- <y>114</y>
- </hint>
- <hint type="destinationlabel">
- <x>80</x>
- <y>805</y>
- </hint>
- </hints>
- </connection>
- </connections>
-</ui>
More information about the lyx-cvs
mailing list