From 0ea25ebb95da3a7f838d211166206cf04d7c8002 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Tyrychtr?= <ltyrycht@redhat.com>
Date: Thu, 11 May 2023 11:19:36 +0200
Subject: [PATCH] a11y: Don't create a parent loop for quick settings sliders

Rather than returning the parent container of the slider, which presents
itself as the slider, return the parent of the slider container.

https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/6686

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/2762>
(cherry picked from commit a5cf08ac556dc9c339c03bb8d7fb24e5c6b56d38)
---
 js/ui/quickSettings.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/js/ui/quickSettings.js b/js/ui/quickSettings.js
index f35a15c3d..7cfd4f4e7 100644
--- a/js/ui/quickSettings.js
+++ b/js/ui/quickSettings.js
@@ -193,7 +193,10 @@ var QuickSlider = GObject.registerClass({
         });
         box.add_child(sliderBin);
 
-        sliderBin.set_accessible(this.slider.get_accessible());
+        // Make the slider bin transparent for a11y
+        const sliderAccessible = this.slider.get_accessible();
+        sliderAccessible.set_parent(sliderBin.get_parent().get_accessible());
+        sliderBin.set_accessible(sliderAccessible);
         sliderBin.connect('event', (bin, event) => this.slider.event(event, false));
 
         this._menuButton = new St.Button({