From 7de0834625d03d6ba7c96e4a2c7c64599478938d Mon Sep 17 00:00:00 2001 From: Syarif Fakhri Date: Fri, 23 Jun 2023 14:35:08 +0800 Subject: [PATCH] Make visibleTabCount function consistent with hasVisibleTabs. Add a bit of documentation. --- src/AutoHideSideBar.cpp | 2 +- src/AutoHideSideBar.h | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/AutoHideSideBar.cpp b/src/AutoHideSideBar.cpp index 85e4b99..cae494d 100644 --- a/src/AutoHideSideBar.cpp +++ b/src/AutoHideSideBar.cpp @@ -321,7 +321,7 @@ int CAutoHideSideBar::visibleTabCount() const int count = 0; for (auto i = 0; i < tabCount(); i++) { - if (tabAt(i)->isVisible()) + if (tabAt(i)->isVisibleTo(parentWidget())) { count++; } diff --git a/src/AutoHideSideBar.h b/src/AutoHideSideBar.h index 20a9e6a..1b0cab5 100644 --- a/src/AutoHideSideBar.h +++ b/src/AutoHideSideBar.h @@ -135,14 +135,14 @@ public: int tabCount() const; /** - * Returns the number of visible tabs + * Returns the number of visible tabs to its parent widget. */ int visibleTabCount() const; /** - * Returns true, if the sidebar contains visible tabs. + * Returns true, if the sidebar contains visible tabs to its parent widget. * The function returns as soon as it finds the first visible tab. - * That means, if you just want to find out if theee are visible tabs + * That means, if you just want to find out if there are visible tabs * then this function is quicker than visibleTabCount() */ bool hasVisibleTabs() const;