[gnome-shell] a11y: fixing a regression with the accessible name at dash items



commit 4ebf396cf5b34eea72c930e340b47bb539809d0e
Author: Alejandro PiÃeiro <apinheiro igalia com>
Date:   Wed Oct 24 19:29:07 2012 +0200

    a11y: fixing a regression with the accessible name at dash items
    
    After some changes, the tooltip label at the dash is not available
    until it is visually shown. As this is not anymore a reliable
    source of accessible name, we just set the accessible name
    with the string used on that label.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=686583

 js/ui/dash.js |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)
---
diff --git a/js/ui/dash.js b/js/ui/dash.js
index b564aea..e1ad321 100644
--- a/js/ui/dash.js
+++ b/js/ui/dash.js
@@ -142,6 +142,7 @@ const DashItemContainer = new Lang.Class({
 
     setLabelText: function(text) {
         this._labelText = text;
+        this.child.accessible_name = text;
     },
 
     hideLabel: function () {
@@ -259,7 +260,6 @@ const ShowAppsIcon = new Lang.Class({
 
         this.setChild(this.toggleButton);
         this.setDragApp(null);
-        this.toggleButton.label_actor = this.label;
     },
 
     _createIcon: function(size) {
@@ -507,9 +507,11 @@ const Dash = new Lang.Class({
         let item = new DashItemContainer();
         item.setChild(display.actor);
 
+        // Override default AppWellIcon label_actor, now the
+        // accessible_name is set at DashItemContainer.setLabelText
+        display.actor.label_actor = null;
         item.setLabelText(app.get_name());
-        // Override default AppWellIcon label_actor
-        display.actor.label_actor = item.label;
+
         display.icon.setIconSize(this.iconSize);
         this._hookUpLabel(item);
 



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]