search

VISTALL / jsyntaxpane

Automatically exported from code.google.com/p/jsyntaxpane
0 stars 0 forks source link

Up/Down arrows don't work for navigation #14

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. I instantiated a jsyntaxpane with a "sql" editorkit
2. Type 2 or more rows of text
3. Try to move caret up or down with up or down arrows

What is the expected output? What do you see instead?

I expect the caret to move up or down but the caret does not move. Left and 
right arrows work 
OK.

What version of the product are you using? On what operating system?

Version 080717 on Mac OS X (JRE 6)

Please provide any additional information below.

Original issue reported on code.google.com by curtis.s...@gmail.com on 31 Jul 2008 at 4:05

GoogleCodeExporter commented 9 years ago 
Unfortunately, I do not have access to a Mac.  I tried to execute the JAR file
directly, using the supplied tester on Windows and on Ubuntu, change the syntax 
to
SQL from the drop-down list and the arrows work fine.
Is your case specific to SQL syntax?  does the supplied JAR and tester work as 
above?

Original comment by ayman.al...@gmail.com on 1 Aug 2008 at 8:40

GoogleCodeExporter commented 9 years ago 
I'll take this one, as I work exclusively on a Mac. This problem occurs 
regardless of the language being used in 
SyntaxKit. The problem stems from this:

1: public void install(JEditorPane editorPane) {
2:       // clear the bindings and action so we can start fresh
3:       editorPane.getActionMap().clear();
4:       editorPane.getKeymap().removeBindings();
5:       // now use the defaults
 6:      super.install(editorPane);

When I comment out lines 3, the problem  goes away.

Ayman:

What was the rationale behind clearing out the action map?

Original comment by kenneth....@gmail.com on 10 Aug 2008 at 11:17

GoogleCodeExporter commented 9 years ago 
I can't remember the exact reason.  But it had to do with double entries and 
needing
a fresh thing to start with.  This may be tricky to make it work for Mac and 
others.

Original comment by ayman.al...@gmail.com on 11 Aug 2008 at 6:23

GoogleCodeExporter commented 9 years ago 
I finally got to setup an environment to do some testing!
I commented out the lines 3 and 4, so the method starts out with the call to 
super,
and it works on Ubuntu...
Can anybody else test on other platforms as well?

Original comment by ayman.al...@gmail.com on 12 Aug 2008 at 6:20

GoogleCodeExporter commented 9 years ago 
Fix in r28.  Can you verify, please?

Original comment by ayman.al...@gmail.com on 12 Aug 2008 at 6:35

GoogleCodeExporter commented 9 years ago 
ooops.. my mistake.. now the flex files are committed in r29.

Original comment by ayman.al...@gmail.com on 13 Aug 2008 at 3:42

GoogleCodeExporter commented 9 years ago 
Still can't verify...I can build now, but I get no lexers are found when I run 
SyntaxTester.

Original comment by kenneth....@gmail.com on 13 Aug 2008 at 11:20

GoogleCodeExporter commented 9 years ago 
Ken, can you rebuild r36?

Original comment by ayman.al...@gmail.com on 17 Aug 2008 at 5:19

GoogleCodeExporter commented 9 years ago 
Verified on Intel Mac running Mac OS 10.5.4.

Original comment by kenneth....@gmail.com on 17 Aug 2008 at 9:41