From: Slava P. <sp...@gj...> - 2000-11-10 07:31:07
|
Todd Papaioannou wrote: > > --On Thursday, November 09, 2000 04:59:04 AM -0800 Alan Moore <ala...@op...> wrote: > > > At 04:47 PM 11/8/00 +1100, you wrote: > >> One change was that the caretVisible variable was removed; previously, > >> it was set when the text area gets focus, and unset when focus was lost. > >> The caret would be drawn if this variable was set. Now, it just directly > >> checks if the text area has focus (hasFocus() method). So that might be > >> the problem. > > > > > > Yep, that's it. I commented out all of the hasFocus() checks, and it > > works perfectly. It seems they were superfluous anyway; just having the > > textarea check for 'focusedComponent == this' is sufficient to determine > > whether to show the caret. > > I've made those changes in my local copy and they also work fine. Since > I'm on the other side of the world, and Slava is probably asleep I've made > available a fixed jar file for those of you not comfortable with doing it > yourselves =) > > You can get it from: > > http://www.luckyspin.org/Projects/jEdit/index.html > > I must stress there may be other issues as a result of the changes, but I've > been using it with much more success today and haven't noticed any. This file > will go away as soon as the official download is fixed, but it should serve > to help for a few days anyway. I see you didn't actually test your fix :-) Try changing the focus to some other object (eg, text field). Note how the caret in the text area continues blinking. So the hasFocus() test is important. Slava |