Attachment #213980 for bug #355763

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Lines 28-33 Link Here

(-)a/bundles/org.eclipse.e4.ui.workbench.addons.swt/src/org/eclipse/e4/ui/workbench/addons/cleanupaddon/CleanupAddon.java (-1 / +8 lines)
28	import org.eclipse.e4.ui.model.application.ui.basic.MWindow;	28	import org.eclipse.e4.ui.model.application.ui.basic.MWindow;
29	import org.eclipse.e4.ui.model.application.ui.menu.MMenuElement;	29	import org.eclipse.e4.ui.model.application.ui.menu.MMenuElement;
30	import org.eclipse.e4.ui.model.application.ui.menu.MToolBar;	30	import org.eclipse.e4.ui.model.application.ui.menu.MToolBar;
		31	import org.eclipse.e4.ui.workbench.IPresentationEngine;
31	import org.eclipse.e4.ui.workbench.UIEvents;	32	import org.eclipse.e4.ui.workbench.UIEvents;
32	import org.eclipse.e4.ui.workbench.modeling.EModelService;	33	import org.eclipse.e4.ui.workbench.modeling.EModelService;
33	import org.eclipse.e4.ui.workbench.renderers.swt.SashLayout;	34	import org.eclipse.e4.ui.workbench.renderers.swt.SashLayout;
Lines 62-68 Link Here
62	// Determine the elements that should not ever be auto-destroyed	63	// Determine the elements that should not ever be auto-destroyed
63	if (container instanceof MApplication \|\| container instanceof MPerspectiveStack	64	if (container instanceof MApplication \|\| container instanceof MPerspectiveStack
64	\|\| container instanceof MMenuElement \|\| container instanceof MTrimBar	65	\|\| container instanceof MMenuElement \|\| container instanceof MTrimBar
65	\|\| container instanceof MToolBar \|\| container instanceof MArea) {	66	\|\| container instanceof MToolBar \|\| container instanceof MArea
		67	\|\| container.getTags().contains(IPresentationEngine.NO_CLOSE)) {
66	return;	68	return;
67	}	69	}
68		70
Lines 262-267 Link Here
262	if (!container.isToBeRendered())	264	if (!container.isToBeRendered())
263	container.setToBeRendered(true);	265	container.setToBeRendered(true);
264	} else {	266	} else {
		267	// Never hide the container marked as no_close
		268	if (container.getTags().contains(IPresentationEngine.NO_CLOSE)) {
		269	return;
		270	}
		271
265	int visCount = modelService.countRenderableChildren(container);	272	int visCount = modelService.countRenderableChildren(container);
266		273
267	// Remove stacks with no visible children from the display (but not the	274	// Remove stacks with no visible children from the display (but not the




	public static final String NO_TITLE = "NoTitle"; //$NON-NLS-1$

	/**
	 * Applying this tag to a container will ensure that the container remains visible in the UI
	 * even if it has no visible content. Applying the tag to a UI 'leaf' element (MPart /
	 * MPlaceholder) will cause the renderer to not provide a 'close' affordance.
	 * 
	 * <b>NOTE:</b> If you are not using perspectives then use the MPart's 'isCloseable' attribute
	 * to control the affordance.

	public static String MINIMIZED_BY_ZOOM = "MinimizedByZoom"; //$NON-NLS-1$

	/**
	 * This key should be used to add an optional String to an element that is a URI to the elements
	 * disabled icon. This is used, for example, by Toolbar Items which, in Eclipse SDK, provide a
	 * unique icon for disabled tool items that look better than the OS default graying on the
	 * default icon.
	 * 
	 * There is a strong argument to be made that this disabledIconURI actually be part of the model
	 */

Return to bug 355763

Lines 34-41 Link Here

(-)a/bundles/org.eclipse.e4.ui.workbench/src/org/eclipse/e4/ui/workbench/IPresentationEngine.java (-6 / +7 lines)
34	public static final String NO_TITLE = "NoTitle"; //$NON-NLS-1$	34	public static final String NO_TITLE = "NoTitle"; //$NON-NLS-1$
35		35
36	/**	36	/**
37	* When applied as a tag to an MPlaceholder inhibits the display of the close affordance. This	37	* Applying this tag to a container will ensure that the container remains visible in the UI
38	* allows a part to be closeable in one perspective but not in a different one.	38	* even if it has no visible content. Applying the tag to a UI 'leaf' element (MPart /
		39	* MPlaceholder) will cause the renderer to not provide a 'close' affordance.
39	*	40	*
40	* <b>NOTE:</b> If you are not using perspectives then use the MPart's 'isCloseable' attribute	41	* <b>NOTE:</b> If you are not using perspectives then use the MPart's 'isCloseable' attribute
41	* to control the affordance.	42	* to control the affordance.
Lines 69-78 Link Here
69	public static String MINIMIZED_BY_ZOOM = "MinimizedByZoom"; //$NON-NLS-1$	70	public static String MINIMIZED_BY_ZOOM = "MinimizedByZoom"; //$NON-NLS-1$
70		71
71	/**	72	/**
72	* This key should be used to add an optional String to an element that is a	73	* This key should be used to add an optional String to an element that is a URI to the elements
73	* URI to the elements disabled icon. This is used, for example, by Toolbar Items	74	* disabled icon. This is used, for example, by Toolbar Items which, in Eclipse SDK, provide a
74	* which, in Eclipse SDK, provide a unique icon for disabled tool items that look better	75	* unique icon for disabled tool items that look better than the OS default graying on the
75	* than the OS default graying on the default icon.	76	* default icon.
76	*	77	*
77	* There is a strong argument to be made that this disabledIconURI actually be part of the model	78	* There is a strong argument to be made that this disabledIconURI actually be part of the model
78	*/	79	*/