From: Dale A. <da...@gr...> - 2008-11-29 14:53:01
|
I went ahead and removed both the commented out code and the associated comment. As you mention, the comment is already in the svn log, which really is good enough. Dale Kazutoshi Satoda wrote: > Hi Dale, > > dal...@us... wrote: >> - cursor = Cursor.getPredefinedCursor(Cursor.TEXT_CURSOR); >> + >> + // removed next line for bug 1968223, "wrong mouse >> pointer in view 1". >> + // Calling visit(SetCursorVisitor) sets the cursor on >> the editPane, >> + // and setting the text cursor on the editPane is >> incorrect since this >> + // will cause the text cursor to be set on the >> bufferswitcher, and the >> + // gutter and scrollbars of the text area. Using the >> default cursor >> + // is the right thing to do, this will set the proper >> cursor on the >> + // subcomponents. >> + ///cursor = Cursor.getPredefinedCursor(Cursor.TEXT_CURSOR); >> + >> visit(new SetCursorVisitor(cursor)); > > Why did you leave the wrong code as a comment? > > I think it should be just removed. The log message looks good enough. |