This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 92780 - Support for new Diff implementation
Summary: Support for new Diff implementation
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 6.x
Hardware: All All
: P1 blocker (vote)
Assignee: issues@editor
URL:
Keywords:
Depends on:
Blocks: 76681 83749 91517
  Show dependency tree
 
Reported: 2007-01-18 15:09 UTC by Maros Sandor
Modified: 2007-11-05 13:44 UTC (History)
1 user (show)

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Maros Sandor 2007-01-18 15:09:32 UTC 
This is to track needed edtior enhancements that block Diff release to 6.0. To
summarize (Diff creates its own JEditorPane):

1) ability to highlight portions of text
2) put a standard error stripe in the pane
3) have a working Undo in the pane
4) have a working hints (lightbulb)
Comment 1 Vitezslav Stejskal 2007-01-18 22:27:30 UTC 
#1 should be possible with the new Highlighting SPI that we are in the process
of merging in trunk right now.

I know you had probably talked about it with Mila, but could you please
elaborate a bit more on all of those requirements? It would be useful to know
what exactly you need and where you hit the wall with the current
infrastructure. If you have a document describing all this and if it's not too
long you can just link it here. Thanks.
Comment 2 Maros Sandor 2007-01-19 09:38:34 UTC 
I would suggest we meet offline, this issue is here mostly to track the progress.
Comment 3 Maros Sandor 2007-02-26 10:16:48 UTC 
1) is fixed
3) is tracked in Issue #96543.

Remaining issues: 2), 4) and also
5) have a working semantic coloring and code completion
Comment 4 Maros Sandor 2007-03-08 09:23:16 UTC 
Currently the last remaining issue is 
3) error stripe
Comment 5 Miloslav Metelka 2007-03-19 19:03:27 UTC 
Error stripe should now work in trunk (other side bars disabled):
Checking in
diff/src/org/netbeans/modules/diff/builtin/visualizer/editable/DiffContentPanel.java;
/cvs/diff/src/org/netbeans/modules/diff/builtin/visualizer/editable/DiffContentPanel.java,v
 <--  DiffContentPanel.java
new revision: 1.3; previous revision: 1.2
done
Checking in
diff/src/org/netbeans/modules/diff/builtin/visualizer/editable/EditableDiffView.java;
/cvs/diff/src/org/netbeans/modules/diff/builtin/visualizer/editable/EditableDiffView.java,v
 <--  EditableDiffView.java
new revision: 1.7; previous revision: 1.6
done
Checking in editor/arch/arch-editor.xml;
/cvs/editor/arch/arch-editor.xml,v  <--  arch-editor.xml
new revision: 1.36; previous revision: 1.35
done
Checking in
editor/errorstripe/src/org/netbeans/modules/editor/errorstripe/AnnotationView.java;
/cvs/editor/errorstripe/src/org/netbeans/modules/editor/errorstripe/AnnotationView.java,v
 <--  AnnotationView.java
new revision: 1.31; previous revision: 1.30
done
Checking in editor/src/org/netbeans/modules/editor/impl/CustomizableSideBar.java;
/cvs/editor/src/org/netbeans/modules/editor/impl/CustomizableSideBar.java,v  <--
 CustomizableSideBar.java
new revision: 1.2; previous revision: 1.1