From d424e69563609eb19a3571ed39bdf649272f2bac Mon Sep 17 00:00:00 2001 From: RickyRister Date: Sun, 2 Mar 2025 02:05:42 -0800 Subject: [PATCH] use isHidden Otherwise, it doesn't work correctly if the BannerWidget is offscreen --- .../src/client/ui/widgets/general/display/banner_widget.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cockatrice/src/client/ui/widgets/general/display/banner_widget.cpp b/cockatrice/src/client/ui/widgets/general/display/banner_widget.cpp index 1fc7ddeda..eda7aebc7 100644 --- a/cockatrice/src/client/ui/widgets/general/display/banner_widget.cpp +++ b/cockatrice/src/client/ui/widgets/general/display/banner_widget.cpp @@ -68,7 +68,7 @@ void BannerWidget::toggleBuddyVisibility() const void BannerWidget::updateDropdownIconState() const { if (clickable) { - bool expanded = buddy && buddy->isVisible(); + bool expanded = buddy && !buddy->isHidden(); iconLabel->setPixmap(DropdownIconPixmapGenerator::generatePixmap(24, expanded)); } else { // we cannot directly hide the iconLabel, since it's needed to center the text; set an empty image instead