orca r3475 - in trunk: . src/orca test/keystrokes/gtk-demo

[richb svn gnome org]

Author: richb
Date: Fri Jan 18 16:08:00 2008
New Revision: 3475
URL: http://svn.gnome.org/viewvc/orca?rev=3475&view=rev

Log:
        * src/orca/braillegenerator.py:
          test/keystrokes/gtk-demo/role_table.py:
          Fixup for bug #503874 - Read by row in Evolution reads cell 
          information incorrectly. With the fix for bug #508153, we no 
          longer need to put an extra space after the table cell regions 
          for a table cell that has a toggle action (i.e. checkbox). 
          Regression test updated to remove the bogus spaces.


Modified:
   trunk/ChangeLog
   trunk/src/orca/braillegenerator.py
   trunk/test/keystrokes/gtk-demo/role_table.py

Modified: trunk/src/orca/braillegenerator.py
==============================================================================
--- trunk/src/orca/braillegenerator.py	(original)
+++ trunk/src/orca/braillegenerator.py	Fri Jan 18 16:08:00 2008
@@ -1207,7 +1207,6 @@
                     + "looking at action %d" % i)
                 if action.getName(i) == "toggle":
                     regions = self._getBrailleRegionsForCheckBox(obj)
-                    regions[0].append(braille.Region(" "))
 
                     # If this table cell doesn't have any label associated
                     # with it then also braille the table column header.

Modified: trunk/test/keystrokes/gtk-demo/role_table.py
==============================================================================
--- trunk/test/keystrokes/gtk-demo/role_table.py	(original)
+++ trunk/test/keystrokes/gtk-demo/role_table.py	Fri Jan 18 16:08:00 2008
@@ -49,8 +49,8 @@
      "     VISIBLE:  'Editable Cells TREE LEVEL 2', cursor=1",
      "BRAILLE LINE:  'gtk-demo Application Shopping list Frame'",
      "     VISIBLE:  'Shopping list Frame', cursor=1",
-     "BRAILLE LINE:  'gtk-demo Application Shopping list Frame ScrollPane Table Number ColumnHeader 3 bottles of coke '",
-     "     VISIBLE:  '3 bottles of coke ', cursor=1",
+     "BRAILLE LINE:  'gtk-demo Application Shopping list Frame ScrollPane Table Number ColumnHeader 3 bottles of coke'",
+     "     VISIBLE:  '3 bottles of coke', cursor=1",
      "SPEECH OUTPUT: 'Widget (double click for demo) page'",
      "SPEECH OUTPUT: 'Widget (double click for demo) column header'",
      "SPEECH OUTPUT: 'Editable Cells'",
@@ -59,7 +59,7 @@
      "SPEECH OUTPUT: 'Shopping list frame'",
      "SPEECH OUTPUT: ''",
      "SPEECH OUTPUT: 'Number column header'",
-     "SPEECH OUTPUT: '3 bottles of coke '"]))
+     "SPEECH OUTPUT: '3 bottles of coke'"]))
 
 ########################################################################
 # Do a basic "Where Am I" via KP_Enter.
@@ -69,8 +69,8 @@
 sequence.append(PauseAction(3000))
 sequence.append(utils.AssertPresentationAction(
     "Table Where Am I",
-    ["BRAILLE LINE:  'gtk-demo Application Shopping list Frame ScrollPane Table Number ColumnHeader 3 bottles of coke '",
-     "     VISIBLE:  '3 bottles of coke ', cursor=1",
+    ["BRAILLE LINE:  'gtk-demo Application Shopping list Frame ScrollPane Table Number ColumnHeader 3 bottles of coke'",
+     "     VISIBLE:  '3 bottles of coke', cursor=1",
      "SPEECH OUTPUT: ''",
      "SPEECH OUTPUT: 'cell'",
      "SPEECH OUTPUT: '3'",
@@ -90,10 +90,10 @@
                            5000))
 sequence.append(utils.AssertPresentationAction(
     "Table down one line",
-    ["BRAILLE LINE:  'gtk-demo Application Shopping list Frame ScrollPane Table Number ColumnHeader 5 packages of noodles '",
-     "     VISIBLE:  '5 packages of noodles ', cursor=1",
+    ["BRAILLE LINE:  'gtk-demo Application Shopping list Frame ScrollPane Table Number ColumnHeader 5 packages of noodles'",
+     "     VISIBLE:  '5 packages of noodles', cursor=1",
      "SPEECH OUTPUT: ''",
-     "SPEECH OUTPUT: '5 packages of noodles '"]))
+     "SPEECH OUTPUT: '5 packages of noodles'"]))
 
 ########################################################################
 # Do a basic "Where Am I" via KP_Enter.