Description Michael D. Elder 2006-04-10 15:46:01 EDT

This change requires a minor API change:

Additional constructor to WizardActionGroup:

	/**
	 * 
	 * @param aWindow
	 *            The window that will be used to acquire a Shell and a
	 *            Selection Service
	 * @param aWizardRegistry
	 *            The wizard registry will be used to locate the correct wizard
	 *            descriptions.
	 * @param aType
	 *            Indicates the value of the type attribute of the commonWizard
	 *            extension point. Use any of the TYPE_XXX constants defined on
	 *            this class.
	 * @param aContentService 
	 * 			 The content service to use when deciding visibility.         
	 * @see PlatformUI#getWorkbench()
	 * @see IWorkbench#getNewWizardRegistry()
	 * @see IWorkbench#getImportWizardRegistry()
	 * @see IWorkbench#getExportWizardRegistry()
	 */
	public WizardActionGroup(IWorkbenchWindow aWindow,
			IWizardRegistry aWizardRegistry, String aType, INavigatorContentService aContentService) {
		this(aWindow, aWizardRegistry, aType);
		contentService = aContentService;

	}

Existing clients will not be affected by this API addition.

This change also requires the small modification of the schema:

org.eclipse.ui.navigator.navigatorContent:
   navigatorContent:
       + commonWizard (0..*)

Existing extensions will not be broken. The change here allows a client to group commonWizard extensions with a navigatorContent extension. Clients may choose not to take advantage of this feature. 

This is a low risk change that will remove options from the menu when the user has indicated they do not want to see content of a particular type. Existing extensions which take advantage of the current approach will not be broken by this change. I think the improvement in the UI experience merits its approval, even at this stage in the game.