[evolution-patches] patch to fix #335625 (gtkhtml) : keyboard navigation perform the wrong order



Hi,

When user press "tab" to navigate the attached web page, gtkhtml will
omit the 2nd link in the table. With my patch, the order will be
corrected.
Please see bug: http://bugzilla.gnome.org/show_bug.cgi?id=335625

Regards,
Li
Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/gtkhtml/src/ChangeLog,v
retrieving revision 1.2174
diff -u -r1.2174 ChangeLog
--- ChangeLog	5 Apr 2006 10:37:26 -0000	1.2174
+++ ChangeLog	11 Apr 2006 06:29:02 -0000
@@ -1,3 +1,11 @@
+2006-04-11  Li Yuan <li yuan sun com>
+
+	* htmlobject.c: (move_object_downtree_cursor):
+	The second htmlobject in a clue could be a clue too, so we need
+	a recursive function.
+
+	Fixes bug #335625.
+
 2006-04-05  Jeff Cai <jeff cai sun com>
 
 	* htmlengine-edit-cut-and-paste.c: (remove_empty_and_merge):
Index: htmlobject.c
===================================================================
RCS file: /cvs/gnome/gtkhtml/src/htmlobject.c,v
retrieving revision 1.191
diff -u -r1.191 htmlobject.c
--- htmlobject.c	31 Mar 2005 12:44:42 -0000	1.191
+++ htmlobject.c	11 Apr 2006 06:29:02 -0000
@@ -1672,9 +1672,14 @@
 	if (!obj && last_obj) {
 		obj = last_obj;
 
-		while ((obj = (*next_fn) (obj)))
+		while ((obj = (*next_fn) (obj))) {
 			if (html_object_accepts_cursor (obj))
 				break;
+			last_obj = obj;
+			if (obj = move_object_downtree_cursor (obj, down_fn, next_fn))
+				break;
+			obj = last_obj;
+		}
 	}
 
 	return obj;
Title: Test 6

Example 6: Table Stress Test


title linkAuto-width tables with spans nested 8 levels deep


Cell
newCell
Cell Cell, and stuff to test out table cell layout. This should be long enough for testing.Cellnew1


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