code cleanup with auxiliary local variables

This commit is contained in:
sgourdas 2024-07-23 14:03:46 +03:00 committed by Kelson
parent 68a7f4be8b
commit 1969fac0a4
2 changed files with 18 additions and 16 deletions

View File

@ -504,17 +504,18 @@ void KiwixApp::handleItemsState(TabType tabType)
{
auto libraryOrSettingsTab = (tabType == TabType::LibraryTab || tabType == TabType::SettingsTab);
auto notBookmarkableTab = libraryOrSettingsTab || getTabWidget()->currentArticleUrl().isEmpty();
KiwixApp::instance()->getAction(KiwixApp::ToggleReadingListAction)->setDisabled(libraryOrSettingsTab);
KiwixApp::instance()->getAction(KiwixApp::ToggleAddBookmarkAction)->setDisabled(notBookmarkableTab);
KiwixApp::instance()->getAction(KiwixApp::FindInPageAction)->setDisabled(libraryOrSettingsTab);
KiwixApp::instance()->getAction(KiwixApp::ZoomInAction)->setDisabled(libraryOrSettingsTab);
KiwixApp::instance()->getAction(KiwixApp::ZoomOutAction)->setDisabled(libraryOrSettingsTab);
KiwixApp::instance()->getAction(KiwixApp::ZoomResetAction)->setDisabled(libraryOrSettingsTab);
KiwixApp::instance()->getAction(KiwixApp::RandomArticleAction)->setDisabled(libraryOrSettingsTab);
auto app = KiwixApp::instance();
app->getAction(KiwixApp::ToggleReadingListAction)->setDisabled(libraryOrSettingsTab);
app->getAction(KiwixApp::ToggleAddBookmarkAction)->setDisabled(notBookmarkableTab);
app->getAction(KiwixApp::FindInPageAction)->setDisabled(libraryOrSettingsTab);
app->getAction(KiwixApp::ZoomInAction)->setDisabled(libraryOrSettingsTab);
app->getAction(KiwixApp::ZoomOutAction)->setDisabled(libraryOrSettingsTab);
app->getAction(KiwixApp::ZoomResetAction)->setDisabled(libraryOrSettingsTab);
app->getAction(KiwixApp::RandomArticleAction)->setDisabled(libraryOrSettingsTab);
/* Non-Zim tabs are not bookmarkable therefore never in reading list. */
if (notBookmarkableTab)
KiwixApp::instance()->getAction(KiwixApp::ToggleAddBookmarkAction)->setChecked(false);
app->getAction(KiwixApp::ToggleAddBookmarkAction)->setChecked(false);
}
void KiwixApp::updateNameMapper()

View File

@ -16,14 +16,14 @@ TopWidget::TopWidget(QWidget *parent) :
auto app = KiwixApp::instance();
QAction *back = app->getAction(KiwixApp::HistoryBackAction);
connect(back, &QAction::triggered, [](){
KiwixApp::instance()->getTabWidget()->triggerWebPageAction(QWebEnginePage::Back);
connect(back, &QAction::triggered, [app](){
app->getTabWidget()->triggerWebPageAction(QWebEnginePage::Back);
});
addAction(back);
QAction *forward = app->getAction(KiwixApp::HistoryForwardAction);
connect(forward, &QAction::triggered, [](){
KiwixApp::instance()->getTabWidget()->triggerWebPageAction(QWebEnginePage::Forward);
connect(forward, &QAction::triggered, [app](){
app->getTabWidget()->triggerWebPageAction(QWebEnginePage::Forward);
});
addAction(forward);
@ -42,11 +42,12 @@ TopWidget::TopWidget(QWidget *parent) :
addSeparator();
addWidget(&m_searchEntry);
addAction(KiwixApp::instance()->getAction(KiwixApp::ToggleReadingListAction));
addAction(app->getAction(KiwixApp::ToggleReadingListAction));
addSeparator();
addAction(KiwixApp::instance()->getAction(KiwixApp::OpenFileAction));
addAction(app->getAction(KiwixApp::OpenFileAction));
QMenu* menu = new MainMenu();
QToolButton *toolButton = new QToolButton(menu);
@ -61,8 +62,8 @@ TopWidget::TopWidget(QWidget *parent) :
#if !SYSTEMTITLEBAR
addAction(QIcon(":/icons/minimize.svg"), "minimize", parent, SLOT(showMinimized()));
#endif
addAction(KiwixApp::instance()->getAction(KiwixApp::ToggleFullscreenAction));
widgetForAction(KiwixApp::instance()->getAction(KiwixApp::ToggleFullscreenAction))->setObjectName("fullScreenButton");
addAction(app->getAction(KiwixApp::ToggleFullscreenAction));
widgetForAction(app->getAction(KiwixApp::ToggleFullscreenAction))->setObjectName("fullScreenButton");
#if !SYSTEMTITLEBAR
addAction(QIcon(":/icons/close.svg"), "close", parent, SLOT(close()));
#endif