Use visibility signal as driver for focus stack focus

opengisch · Jan 6, 2025 · 07832ff · 07832ff
1 parent 4a9ab89
commit 07832ff
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 8 deletions.
diff --git a/src/core/focusstack.cpp b/src/core/focusstack.cpp
@@ -17,16 +17,20 @@
 
 void FocusStack::addFocusTaker( QObject *object )
 {
-  QVariant visible = object->property( "visible" );
-  QVariant opened = object->property( "opened" );
+  const QVariant visible = object->property( "visible" );
+  const QVariant opened = object->property( "opened" );
   if ( opened.isValid() )
   {
     connect( object, SIGNAL( opened() ), this, SLOT( popupOpened() ) );
     connect( object, SIGNAL( closed() ), this, SLOT( popupClosed() ) );
   }
   else if ( visible.isValid() )
   {
-    connect( object, SIGNAL( activeFocusChanged( bool ) ), this, SLOT( itemFocusChanged( bool ) ) );
+    connect( object, SIGNAL( visibleChanged() ), this, SLOT( visibleChanged() ) );
+    if ( visible.toBool() )
+    {
+      mStackList.append( object );
+    }
   }
 }
 
@@ -40,9 +44,10 @@ void FocusStack::popupClosed()
   setUnfocused( sender() );
 }
 
-void FocusStack::itemFocusChanged( bool itemActiveFocus )
+void FocusStack::visibleChanged()
 {
-  if ( itemActiveFocus )
+  const QVariant visible = sender()->property( "visible" );
+  if ( visible.toBool() )
   {
     setFocused( sender() );
   }
@@ -60,8 +65,8 @@ void FocusStack::setFocused( QObject *object )
 
 void FocusStack::setUnfocused( QObject *object )
 {
-  QVariant visible = object->property( "visible" );
-  QVariant opened = object->property( "opened" );
+  const QVariant visible = object->property( "visible" );
+  const QVariant opened = object->property( "opened" );
   if ( opened.isValid() )
   {
     if ( !opened.toBool() )

diff --git a/src/core/focusstack.h b/src/core/focusstack.h
@@ -31,7 +31,7 @@ class FocusStack : public QObject
     Q_INVOKABLE void forceActiveFocusOnLastTaker() const;
 
   private slots:
-    void itemFocusChanged( bool itemActiveFocus );
+    void visibleChanged();
     void popupOpened();
     void popupClosed();