Hi,
the following patch implements the above mentioned method.

I found a suitable comment in the JDK's docs that mentions that it should be
done this way. I prepare a mauve test later.

ChangeLog:

2006-04-18  Robert Schuster  <[EMAIL PROTECTED]>

        * javax/swing/plaf/basic/BasicTextUI.java: Implemented.

cya
Robert
Index: javax/swing/plaf/basic/BasicTextUI.java
===================================================================
RCS file: /cvsroot/classpath/classpath/javax/swing/plaf/basic/BasicTextUI.java,v
retrieving revision 1.79
diff -u -r1.79 BasicTextUI.java
--- javax/swing/plaf/basic/BasicTextUI.java	17 Apr 2006 07:41:05 -0000	1.79
+++ javax/swing/plaf/basic/BasicTextUI.java	18 Apr 2006 20:29:05 -0000
@@ -1140,9 +1140,14 @@
   public int getNextVisualPositionFrom(JTextComponent t, int pos,
                                        Position.Bias b, int direction,
                                        Position.Bias[] biasRet)
-    throws BadLocationException, NotImplementedException
+    throws BadLocationException
   {
-    return 0; // TODO: Implement me.
+    // A comment in the spec of NavigationFilter.getNextVisualPositionFrom()
+    // suggests that this method should be implemented by forwarding the call
+    // the root view.
+    return rootView.getNextVisualPositionFrom(pos, b,
+                                              getVisibleEditorRect(),
+                                              direction, biasRet);
   }
 
   /**

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to