Attachment #74939 for bug #196878




		super(page, parent, Section.DESCRIPTION, new String[]{
				PDEUIMessages.ManifestEditor_DetailExtension_new, 
				PDEUIMessages.ManifestEditor_DetailExtension_edit,
				null,
				PDEUIMessages.ManifestEditor_DetailExtension_up,
				PDEUIMessages.ManifestEditor_DetailExtension_down});
		fHandleDefaultButton = false;

	protected void selectionChanged(IStructuredSelection selection) {
		getPage().getPDEEditor().setSelection(selection);
		updateButtons(selection.getFirstElement());
		getTreePart().getButton(1).setVisible(isSelectionEditable(selection));
	}
	protected void handleDoubleClick(IStructuredSelection selection) {
		/*

		case 1 :
			handleEdit();
			break;
		case 2 :
			// blank
			break;
		case 3 :
			handleMove(true);
			break;
		case 3 :
			handleMove(false);
			break;
		}

			return;
		boolean sorted = fSortAction != null && fSortAction.isChecked();
		if (sorted) {
			getTreePart().setButtonEnabled(2, false);
			getTreePart().setButtonEnabled(3, false);
			getTreePart().setButtonEnabled(4, false);
			return;
		}
		

			}
		}
		getTreePart().setButtonEnabled(0, addEnabled);
		getTreePart().setButtonEnabled(2, upEnabled);
		getTreePart().setButtonEnabled(3, downEnabled);
	}

	/* (non-Javadoc)

Return to bug 196878

Lines 148-154 Link Here

(-)src/org/eclipse/pde/internal/ui/editor/plugin/ExtensionsSection.java (-10 / +6 lines)
148	super(page, parent, Section.DESCRIPTION, new String[]{	148	super(page, parent, Section.DESCRIPTION, new String[]{
149	PDEUIMessages.ManifestEditor_DetailExtension_new,	149	PDEUIMessages.ManifestEditor_DetailExtension_new,
150	PDEUIMessages.ManifestEditor_DetailExtension_edit,	150	PDEUIMessages.ManifestEditor_DetailExtension_edit,
151	null,
152	PDEUIMessages.ManifestEditor_DetailExtension_up,	151	PDEUIMessages.ManifestEditor_DetailExtension_up,
153	PDEUIMessages.ManifestEditor_DetailExtension_down});	152	PDEUIMessages.ManifestEditor_DetailExtension_down});
154	fHandleDefaultButton = false;	153	fHandleDefaultButton = false;
Lines 286-292 Link Here
286	protected void selectionChanged(IStructuredSelection selection) {	285	protected void selectionChanged(IStructuredSelection selection) {
287	getPage().getPDEEditor().setSelection(selection);	286	getPage().getPDEEditor().setSelection(selection);
288	updateButtons(selection.getFirstElement());	287	updateButtons(selection.getFirstElement());
289	getTreePart().setButtonEnabled(1, isSelectionEditable(selection));	288	getTreePart().getButton(1).setVisible(isSelectionEditable(selection));
290	}	289	}
291	protected void handleDoubleClick(IStructuredSelection selection) {	290	protected void handleDoubleClick(IStructuredSelection selection) {
292	/*	291	/*
Lines 302-314 Link Here
302	case 1 :	301	case 1 :
303	handleEdit();	302	handleEdit();
304	break;	303	break;
305	case 2:	304	case 2 :
306	// blank
307	break;
308	case 3 :
309	handleMove(true);	305	handleMove(true);
310	break;	306	break;
311	case 4 :	307	case 3 :
312	handleMove(false);	308	handleMove(false);
313	break;	309	break;
314	}	310	}
Lines 1062-1069 Link Here
1062	return;	1058	return;
1063	boolean sorted = fSortAction != null && fSortAction.isChecked();	1059	boolean sorted = fSortAction != null && fSortAction.isChecked();
1064	if (sorted) {	1060	if (sorted) {
		1061	getTreePart().setButtonEnabled(2, false);
1065	getTreePart().setButtonEnabled(3, false);	1062	getTreePart().setButtonEnabled(3, false);
1066	getTreePart().setButtonEnabled(4, false);
1067	return;	1063	return;
1068	}	1064	}
1069		1065
Lines 1099-1106 Link Here
1099	}	1095	}
1100	}	1096	}
1101	getTreePart().setButtonEnabled(0, addEnabled);	1097	getTreePart().setButtonEnabled(0, addEnabled);
1102	getTreePart().setButtonEnabled(3, upEnabled);	1098	getTreePart().setButtonEnabled(2, upEnabled);
1103	getTreePart().setButtonEnabled(4, downEnabled);	1099	getTreePart().setButtonEnabled(3, downEnabled);
1104	}	1100	}
1105		1101
1106	/* (non-Javadoc)	1102	/* (non-Javadoc)