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 217091

Summary: Navigator "Toolbar" panel
Product: platform Reporter: Jan Peska <JPESKA>
Component: NavigatorAssignee: apireviews <apireviews>
Severity: normal Keywords: API_REVIEW_FAST
Priority: P3    
Version: 7.2   
Hardware: PC   
OS: All   
Issue Type: DEFECT Exception Reporter:
Attachments: Netbeans UI after the change
Proposed changes
Proposed changes #2

Description Jan Peska 2012-08-20 09:48:47 UTC
Created attachment 123302 [details]
Netbeans UI after the change

Add a new SPI interface. Clients will implement this interface when they need a toolbar for their Navigator view/panel. Toolbar will be displayed on the top of the Navigator window, next to the panel chooser (ComboBox) - see attached screenshot.
Comment 1 Jan Peska 2012-08-20 11:35:12 UTC
See for more info about the reason behind it.
Comment 2 Jan Peska 2012-08-20 11:58:57 UTC
Created attachment 123305 [details]
Proposed changes
Comment 3 Jan Peska 2012-08-20 11:59:28 UTC
Please review attached patch.
Comment 4 Jaroslav Tulach 2012-08-20 12:04:46 UTC
Y01 Don't prefix the name of interface with '@', that resembles annotation type, imho.

Y02 Why NavigatorPanelWithToolbar extends NavigatorPanelWithUndo. Should not it extend only NavigatorPanel?

Y03 There is no test. Write at least one, please.
Comment 5 Jan Peska 2012-08-20 12:48:24 UTC
Created attachment 123308 [details]
Proposed changes #2
Comment 6 Jan Peska 2012-08-20 12:48:56 UTC
Y01 - done

Y02 - Yes it should. Corrected.

Y03 - done. I've added a new test, similar to the UndoRedo feature test.
Comment 7 Tomas Zezula 2012-08-23 05:53:54 UTC
The proposed patch seems good to me.
Please integrate it.
Comment 8 Jaroslav Tulach 2012-08-23 07:25:22 UTC
OK. Just two little comments. Use {@link ClassName} to refer to classes in Javadoc - e.g. in sentence "top of basic NavigatorPanel and NavigatorPanelWithUndo features.". Second, there is left System.out.println in the test which is probably not essential for the test execution and only pollutes the output with unnecessary garbage. If the output is essential for diagnosing failures consider using Logger and override "logLevel" method of NbTestCase.
Comment 9 Jan Peska 2012-08-23 08:08:52 UTC
main fix:

fix two little comments (main fix was already pushed when I realize there is another comment):