Bug 58696 Override/Implement Methods dialog: better labels for insertion position [code manipulation]
The Override/Implement Methods dialog says: Insertion Point: - First Method - Last Method - a() - b() ... I *never* know whether that means before or after the selected method. Better labels would be: Insert after: - Class Declaration Header - a() - b() ... or Insert Before: - a() - b() ... - End of Class
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/dialogs/SourceActionDialog.java
commentButton.setLayoutData(gd);
		
		return commentComposite;
	}
	protected Composite addVisibilityAndModifiersChoices(Composite buttonComposite) {
		
		
		IVisibilityChangeListener visibilityChangeListener= new IVisibilityChangeListener(){
			public void visibilityChanged(int newVisibility) {
				setVisibility(newVisibility);
			}
			public void modifierChanged(int modifier, boolean isChecked) {	
				switch (modifier) {
					case Modifier.FINAL:  {
						setFinal(isChecked);
						return; 
					}
					case Modifier.SYNCHRONIZED:  {
						setSynchronized(isChecked);
						return;
					}
					default: return;
				}
			}
		};
			
		int initialVisibility= getVisibilityModifier();
		int[] availableVisibilities= new int[]{Modifier.PUBLIC, Modifier.PROTECTED, Modifier.PRIVATE, Modifier.NONE};
			
		Composite visibilityComposite= createVisibilityControlAndModifiers(buttonComposite, visibilityChangeListener, availableVisibilities, initialVisibility);