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.
When using graphical diff editor shows 2 toolbars above the text area. IMHO it's wasting space. See screenshots, one showing the problem and second possible solution.
Created attachment 13767 [details] screenshot
Created attachment 13768 [details] screenshot
Blank space shown on second screenshot should be removed of course.
I agree that the second approach would be better, but it would be harder to implement, since the "Previous" and "Next" icons are part of the "Graphical Diff Viewer". They disappear when "Textual Diff Viewer" is selected. We would have to add a possibility to viewers to provide a toolbar. That toolbar would be then displayed separately from the viewer component.
Attaching another screenshots that show how much space is wasted comparing to java editor.
Created attachment 18900 [details] screenshot1
Created attachment 18901 [details] screenshot2
Hmm, we can remove the extra spaces around the Engine a Visualizer. Then we would have to create some mechanism how to put the toolbar next to Visualizer...
Not a functional defect -> P4. More an enhancement then a defect.
Too late for 4.1. But it might be necessary to have a mechanism to plug other components into diff window in promotion F.
Looks promising, I'm able to hack in.
Not award winning UI but IMHO better then before: Checking in DiffPresenter.form; /shared/data/ccvs/repository/diff/src/org/netbeans/modules/diff/builtin/DiffPresenter.form,v <-- DiffPresenter.form new revision: 1.5; previous revision: 1.4 done Checking in DiffPresenter.java; /shared/data/ccvs/repository/diff/src/org/netbeans/modules/diff/builtin/DiffPresenter.java,v <-- DiffPresenter.java new revision: 1.21; previous revision: 1.20 done Checking in visualizer/DiffPanel.form; /shared/data/ccvs/repository/diff/src/org/netbeans/modules/diff/builtin/visualizer/DiffPanel.form,v <-- DiffPanel.form new revision: 1.6; previous revision: 1.5 done Checking in visualizer/DiffPanel.java; /shared/data/ccvs/repository/diff/src/org/netbeans/modules/diff/builtin/visualizer/DiffPanel.java,v <-- DiffPanel.java new revision: 1.29; previous revision: 1.28 done Checking in visualizer/DiffComponent.java; /shared/data/ccvs/repository/diff/src/org/netbeans/modules/diff/builtin/visualizer/DiffComponent.java,v <-- DiffComponent.java new revision: 1.18; previous revision: 1.17 done
Verified in dev-200512061900.