Made naming of auto hide config flags explicit

This commit is contained in:
Uwe Kindler 2022-11-01 14:36:09 +01:00
parent bfad80c540
commit 94b80423d7
7 changed files with 15 additions and 19 deletions

View File

@ -85,19 +85,19 @@ void AutoHideTabPrivate::updateOrientation()
switch (area) switch (area)
{ {
case SideBarLocation::Left: case SideBarLocation::Left:
IconOnly = CDockManager::testConfigFlag(CDockManager::LeftSideBarIconOnly); IconOnly = CDockManager::testAutoHideConfigFlag(CDockManager::LeftSideBarIconOnly);
break; break;
case SideBarLocation::Right: case SideBarLocation::Right:
IconOnly = CDockManager::testConfigFlag(CDockManager::RightSideBarIconOnly); IconOnly = CDockManager::testAutoHideConfigFlag(CDockManager::RightSideBarIconOnly);
break; break;
case SideBarLocation::Top: case SideBarLocation::Top:
IconOnly = CDockManager::testConfigFlag(CDockManager::BottomSideBarIconOnly); IconOnly = CDockManager::testAutoHideConfigFlag(CDockManager::BottomSideBarIconOnly);
break; break;
case SideBarLocation::Bottom: case SideBarLocation::Bottom:
IconOnly = CDockManager::testConfigFlag(CDockManager::TopSideBarIconOnly); IconOnly = CDockManager::testAutoHideConfigFlag(CDockManager::TopSideBarIconOnly);
break; break;
} }

View File

@ -124,9 +124,9 @@ struct DockAreaTitleBarPrivate
* Returns true if the given config flag is set * Returns true if the given config flag is set
* Convenience function to ease config flag testing * Convenience function to ease config flag testing
*/ */
static bool testConfigFlag(CDockManager::eAutoHideFlag Flag) static bool testAutoHideConfigFlag(CDockManager::eAutoHideFlag Flag)
{ {
return CDockManager::testConfigFlag(Flag); return CDockManager::testAutoHideConfigFlag(Flag);
} }
/** /**
@ -191,14 +191,14 @@ void DockAreaTitleBarPrivate::createButtons()
_this->connect(UndockButton, SIGNAL(clicked()), SLOT(onUndockButtonClicked())); _this->connect(UndockButton, SIGNAL(clicked()), SLOT(onUndockButtonClicked()));
// AutoHide Button // AutoHide Button
const auto autoHideEnabled = testConfigFlag(CDockManager::AutoHideFeatureEnabled); const auto autoHideEnabled = testAutoHideConfigFlag(CDockManager::AutoHideFeatureEnabled);
AutoHideButton = new CTitleBarButton(testConfigFlag(CDockManager::DockAreaHasAutoHideButton) && autoHideEnabled); AutoHideButton = new CTitleBarButton(testAutoHideConfigFlag(CDockManager::DockAreaHasAutoHideButton) && autoHideEnabled);
AutoHideButton->setObjectName("dockAreaAutoHideButton"); AutoHideButton->setObjectName("dockAreaAutoHideButton");
AutoHideButton->setAutoRaise(true); AutoHideButton->setAutoRaise(true);
internal::setToolTip(AutoHideButton, QObject::tr("Toggle Auto Hide")); internal::setToolTip(AutoHideButton, QObject::tr("Toggle Auto Hide"));
internal::setButtonIcon(AutoHideButton, QStyle::SP_DialogOkButton, ads::AutoHideIcon); internal::setButtonIcon(AutoHideButton, QStyle::SP_DialogOkButton, ads::AutoHideIcon);
AutoHideButton->setSizePolicy(ButtonSizePolicy); AutoHideButton->setSizePolicy(ButtonSizePolicy);
AutoHideButton->setCheckable(testConfigFlag(CDockManager::AutoHideButtonCheckable)); AutoHideButton->setCheckable(testAutoHideConfigFlag(CDockManager::AutoHideButtonCheckable));
AutoHideButton->setChecked(false); AutoHideButton->setChecked(false);
Layout->addWidget(AutoHideButton, 0); Layout->addWidget(AutoHideButton, 0);
_this->connect(AutoHideButton, SIGNAL(clicked()), SLOT(onAutoHideButtonClicked())); _this->connect(AutoHideButton, SIGNAL(clicked()), SLOT(onAutoHideButtonClicked()));

View File

@ -1176,7 +1176,7 @@ void CDockAreaWidget::toggleAutoHideArea(bool Enable)
const auto area = dockContainer()->calculateSideTabBarArea(this); const auto area = dockContainer()->calculateSideTabBarArea(this);
if (dockManager()->testConfigFlag(CDockManager::AutoHideButtonTogglesArea)) if (CDockManager::testAutoHideConfigFlag(CDockManager::AutoHideButtonTogglesArea))
{ {
for (const auto DockWidget : openedDockWidgets()) for (const auto DockWidget : openedDockWidgets())
{ {

View File

@ -1045,7 +1045,7 @@ bool DockContainerWidgetPrivate::restoreSideBar(CDockingStateReader& s,
Q_UNUSED(CreatedWidget) Q_UNUSED(CreatedWidget)
// Simply ignore side bar auto hide widgets from saved state if // Simply ignore side bar auto hide widgets from saved state if
// auto hide support is disabled // auto hide support is disabled
if (!CDockManager::testConfigFlag(CDockManager::AutoHideFeatureEnabled)) if (!CDockManager::testAutoHideConfigFlag(CDockManager::AutoHideFeatureEnabled))
{ {
return true; return true;
} }
@ -1374,7 +1374,7 @@ CDockAreaWidget* CDockContainerWidget::addDockWidget(DockWidgetArea area, CDockW
CAutoHideDockContainer* CDockContainerWidget::createAndSetupAutoHideContainer( CAutoHideDockContainer* CDockContainerWidget::createAndSetupAutoHideContainer(
SideBarLocation area, CDockWidget* DockWidget, CDockWidget::eAutoHideInsertOrder insertOrder) SideBarLocation area, CDockWidget* DockWidget, CDockWidget::eAutoHideInsertOrder insertOrder)
{ {
if (!CDockManager::testConfigFlag(CDockManager::AutoHideFeatureEnabled)) if (!CDockManager::testAutoHideConfigFlag(CDockManager::AutoHideFeatureEnabled))
{ {
Q_ASSERT_X(false, "CDockContainerWidget::createAndInitializeDockWidgetOverlayContainer", Q_ASSERT_X(false, "CDockContainerWidget::createAndInitializeDockWidgetOverlayContainer",
"Requested area does not exist in config"); "Requested area does not exist in config");
@ -1951,7 +1951,7 @@ void CDockContainerWidget::createRootSplitter()
//============================================================================ //============================================================================
void CDockContainerWidget::createSideTabBarWidgets() void CDockContainerWidget::createSideTabBarWidgets()
{ {
if (!CDockManager::testConfigFlag(CDockManager::AutoHideFeatureEnabled)) if (!CDockManager::testAutoHideConfigFlag(CDockManager::AutoHideFeatureEnabled))
{ {
return; return;
} }

View File

@ -1183,7 +1183,7 @@ bool CDockManager::testConfigFlag(eConfigFlag Flag)
//=========================================================================== //===========================================================================
bool CDockManager::testConfigFlag(eAutoHideFlag Flag) bool CDockManager::testAutoHideConfigFlag(eAutoHideFlag Flag)
{ {
return autoHideConfigFlags().testFlag(Flag); return autoHideConfigFlags().testFlag(Flag);
} }

View File

@ -307,7 +307,7 @@ public:
/** /**
* Returns true if the given overlay config flag is set * Returns true if the given overlay config flag is set
*/ */
static bool testConfigFlag(eAutoHideFlag Flag); static bool testAutoHideConfigFlag(eAutoHideFlag Flag);
/** /**
* Returns the global icon provider. * Returns the global icon provider.

View File

@ -343,10 +343,6 @@ CDockWidgetTab::CDockWidgetTab(CDockWidget* DockWidget, QWidget *parent) :
d->DockWidget = DockWidget; d->DockWidget = DockWidget;
d->createLayout(); d->createLayout();
setFocusPolicy(Qt::NoFocus); setFocusPolicy(Qt::NoFocus);
/*if (CDockManager::testConfigFlag(CDockManager::FocusHighlighting))
{
setFocusPolicy(Qt::ClickFocus);
}*/
} }
//============================================================================ //============================================================================