Attachment 190345 Details for Bug 338879 – extend pde editor menus

[patch] extend pde editor menus

pde-extensible-menus.txt (text/plain), 5.08 KB, created by Jeff McAffer

on 2011-03-03 21:35:20 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Jeff McAffer

Created: 2011-03-03 21:35:20 EST

Size: 5.08 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.pde.ui
>Index: src/org/eclipse/pde/internal/ui/editor/StructuredViewerSection.java
>===================================================================
>RCS file: /cvsroot/eclipse/pde/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/StructuredViewerSection.java,v
>retrieving revision 1.25
>diff -u -r1.25 StructuredViewerSection.java
>--- src/org/eclipse/pde/internal/ui/editor/StructuredViewerSection.java	27 May 2009 20:38:16 -0000	1.25
>+++ src/org/eclipse/pde/internal/ui/editor/StructuredViewerSection.java	4 Mar 2011 02:29:39 -0000
>@@ -61,6 +61,7 @@
> 		Control control = fViewerPart.getControl();
> 		Menu menu = popupMenuManager.createContextMenu(control);
> 		control.setMenu(menu);
>+		registerPopupMenu(popupMenuManager);
> 		// Initialize drag and drop
> 		if (isDragAndDropEnabled()) {
> 			initializeDragAndDrop();
>@@ -70,6 +71,15 @@
> 		}
> 	}
> 
>+	/**
>+	 * If the context menu for this section should be registered, do it here
>+	 * with the appropriate id etc.  By default do nothing.
>+	 * @param popupMenuManager the menu manager to be registered
>+	 */
>+	protected void registerPopupMenu(MenuManager popupMenuManager) {
>+		// do nothing by default
>+	}
>+
> 	protected Composite createClientContainer(Composite parent, int span, FormToolkit toolkit) {
> 		Composite container = toolkit.createComposite(parent);
> 		container.setLayout(FormLayoutFactory.createSectionClientGridLayout(false, span));
>Index: src/org/eclipse/pde/internal/ui/editor/feature/RequiresSection.java
>===================================================================
>RCS file: /cvsroot/eclipse/pde/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/feature/RequiresSection.java,v
>retrieving revision 1.51
>diff -u -r1.51 RequiresSection.java
>--- src/org/eclipse/pde/internal/ui/editor/feature/RequiresSection.java	27 May 2009 20:38:18 -0000	1.51
>+++ src/org/eclipse/pde/internal/ui/editor/feature/RequiresSection.java	4 Mar 2011 02:29:39 -0000
>@@ -10,10 +10,6 @@
>  *******************************************************************************/
> package org.eclipse.pde.internal.ui.editor.feature;
> 
>-import org.eclipse.pde.internal.ui.dialogs.PluginSelectionDialog;
>-
>-import org.eclipse.pde.internal.ui.dialogs.FeatureSelectionDialog;
>-
> import java.util.ArrayList;
> import java.util.Iterator;
> import org.eclipse.core.runtime.CoreException;
>@@ -27,11 +23,13 @@
> import org.eclipse.pde.internal.core.ifeature.*;
> import org.eclipse.pde.internal.ui.PDEPlugin;
> import org.eclipse.pde.internal.ui.PDEUIMessages;
>+import org.eclipse.pde.internal.ui.dialogs.FeatureSelectionDialog;
>+import org.eclipse.pde.internal.ui.dialogs.PluginSelectionDialog;
> import org.eclipse.pde.internal.ui.editor.*;
> import org.eclipse.pde.internal.ui.editor.actions.SortAction;
> import org.eclipse.pde.internal.ui.elements.DefaultContentProvider;
> import org.eclipse.pde.internal.ui.parts.TablePart;
>-import org.eclipse.pde.internal.ui.wizards.*;
>+import org.eclipse.pde.internal.ui.wizards.ListUtil;
> import org.eclipse.swt.SWT;
> import org.eclipse.swt.custom.BusyIndicator;
> import org.eclipse.swt.dnd.Clipboard;
>@@ -351,6 +349,10 @@
> 		getPage().getPDEEditor().getContributor().contextMenuAboutToShow(manager);
> 	}
> 
>+	protected void registerPopupMenu(MenuManager popupMenuManager) {
>+		getPage().getSite().registerContextMenu(popupMenuManager, fViewerPart.getViewer());
>+	}
>+
> 	protected void selectionChanged(IStructuredSelection selection) {
> 		getPage().getPDEEditor().setSelection(selection);
> 		getPage().getManagedForm().fireSelectionChanged(this, selection);
>Index: src/org/eclipse/pde/internal/ui/editor/plugin/RequiresSection.java
>===================================================================
>RCS file: /cvsroot/eclipse/pde/ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/RequiresSection.java,v
>retrieving revision 1.54
>diff -u -r1.54 RequiresSection.java
>--- src/org/eclipse/pde/internal/ui/editor/plugin/RequiresSection.java	19 Oct 2008 19:43:02 -0000	1.54
>+++ src/org/eclipse/pde/internal/ui/editor/plugin/RequiresSection.java	4 Mar 2011 02:29:39 -0000
>@@ -10,8 +10,6 @@
>  *******************************************************************************/
> package org.eclipse.pde.internal.ui.editor.plugin;
> 
>-import org.eclipse.pde.internal.ui.dialogs.PluginSelectionDialog;
>-
> import java.io.*;
> import java.util.*;
> import org.eclipse.core.runtime.CoreException;
>@@ -30,6 +28,7 @@
> import org.eclipse.pde.internal.core.text.plugin.PluginDocumentNodeFactory;
> import org.eclipse.pde.internal.core.util.VersionUtil;
> import org.eclipse.pde.internal.ui.*;
>+import org.eclipse.pde.internal.ui.dialogs.PluginSelectionDialog;
> import org.eclipse.pde.internal.ui.editor.FormLayoutFactory;
> import org.eclipse.pde.internal.ui.editor.TableSection;
> import org.eclipse.pde.internal.ui.editor.actions.SortAction;
>@@ -338,6 +337,10 @@
> 		}
> 	}
> 
>+	protected void registerPopupMenu(MenuManager popupMenuManager) {
>+		getPage().getSite().registerContextMenu(popupMenuManager, fViewerPart.getViewer());
>+	}
>+
> 	private void handleOpen(ISelection sel) {
> 		if (sel instanceof IStructuredSelection) {
> 			IStructuredSelection ssel = (IStructuredSelection) sel;

Actions: View | Diff

Attachments on bug 338879: 190345 | 190346 | 191251 | 191331