Attachment 186400 Details for Bug 333886 – patch for view.properties.tabbed

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

[patch] patch for view.properties.tabbed

tabbed-properties-rap.patch (text/plain), 47.42 KB, created by Gunnar Wagenknecht

on 2011-01-10 11:38:41 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Gunnar Wagenknecht

Created: 2011-01-10 11:38:41 EST

Size: 47.42 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.rap.ui.views.properties.tabbed
>Index: .project
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/.project,v
>retrieving revision 1.2
>diff -u -r1.2 .project
>--- .project	21 May 2008 20:13:34 -0000	1.2
>+++ .project	10 Jan 2011 16:37:25 -0000
>@@ -1,6 +1,6 @@
> <?xml version="1.0" encoding="UTF-8"?>
> <projectDescription>
>-	<name>org.eclipse.ui.views.properties.tabbed</name>
>+	<name>org.eclipse.rap.ui.views.properties.tabbed</name>
> 	<comment></comment>
> 	<projects>
> 	</projects>
>Index: plugin.xml
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/plugin.xml,v
>retrieving revision 1.2
>diff -u -r1.2 plugin.xml
>--- plugin.xml	1 Feb 2006 00:10:51 -0000	1.2
>+++ plugin.xml	10 Jan 2011 16:37:25 -0000
>@@ -1,9 +1,9 @@
> <?xml version="1.0" encoding="UTF-8"?>
>-<?eclipse version="3.0"?>
>+<?eclipse version="3.2"?>
> <plugin>
> 
>-   <extension-point id="propertyContributor" name="%ExtPoint.propertyContributor" schema="schema/propertyContributor.exsd"/>
>-   <extension-point id="propertyTabs" name="%ExtPoint.propertyTabs" schema="schema/propertyTabs.exsd"/>
>-   <extension-point id="propertySections" name="%ExtPoint.propertySections" schema="schema/propertySections.exsd"/>
>+   <extension-point id="org.eclipse.ui.views.properties.tabbed.propertyContributor" name="%ExtPoint.propertyContributor" schema="schema/propertyContributor.exsd"/>
>+   <extension-point id="org.eclipse.ui.views.properties.tabbed.propertyTabs" name="%ExtPoint.propertyTabs" schema="schema/propertyTabs.exsd"/>
>+   <extension-point id="org.eclipse.ui.views.properties.tabbed.propertySections" name="%ExtPoint.propertySections" schema="schema/propertySections.exsd"/>
> 
> </plugin>
>Index: .settings/org.eclipse.jdt.ui.prefs
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/.settings/org.eclipse.jdt.ui.prefs,v
>retrieving revision 1.6
>diff -u -r1.6 org.eclipse.jdt.ui.prefs
>--- .settings/org.eclipse.jdt.ui.prefs	3 Jul 2008 20:52:08 -0000	1.6
>+++ .settings/org.eclipse.jdt.ui.prefs	10 Jan 2011 16:37:25 -0000
>@@ -1,5 +1,58 @@
>-#Wed Jul 02 15:17:10 EDT 2008
>+#Fri Jan 07 09:15:54 CET 2011
> eclipse.preferences.version=1
>+editor_save_participant_org.eclipse.jdt.ui.postsavelistener.cleanup=false
> internal.default.compliance=user
> org.eclipse.jdt.ui.javadoc=true
> org.eclipse.jdt.ui.text.custom_code_templates=<?xml version\="1.0" encoding\="UTF-8"?><templates><template autoinsert\="true" context\="gettercomment_context" deleted\="false" description\="Comment for getter method" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.gettercomment" name\="gettercomment">/**\r\n * @return Returns the ${bare_field_name}.\r\n */</template><template autoinsert\="true" context\="settercomment_context" deleted\="false" description\="Comment for setter method" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.settercomment" name\="settercomment">/**\r\n * @param ${param} The ${bare_field_name} to set.\r\n */</template><template autoinsert\="true" context\="constructorcomment_context" deleted\="false" description\="Comment for created constructors" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.constructorcomment" name\="constructorcomment">/**\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="filecomment_context" deleted\="false" description\="Comment for created Java files" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.filecomment" name\="filecomment">/*******************************************************************************\r\n * Copyright (c) ${year} IBM Corporation and others.\r\n * All rights reserved. This program and the accompanying materials\r\n * are made available under the terms of the Eclipse Public License v1.0\r\n * which accompanies this distribution, and is available at\r\n * http\://www.eclipse.org/legal/epl-v10.html\r\n *\r\n * Contributors\:\r\n *     IBM Corporation - initial API and implementation\r\n ******************************************************************************/\r\n</template><template autoinsert\="false" context\="typecomment_context" deleted\="false" description\="Comment for created types" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.typecomment" name\="typecomment">/**\r\n * @since 3.5\r\n *\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="fieldcomment_context" deleted\="false" description\="Comment for fields" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.fieldcomment" name\="fieldcomment">/**\r\n * \r\n */</template><template autoinsert\="true" context\="methodcomment_context" deleted\="false" description\="Comment for non-overriding methods" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.methodcomment" name\="methodcomment">/**\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="overridecomment_context" deleted\="false" description\="Comment for overriding methods" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.overridecomment" name\="overridecomment">/* (non-Javadoc)\r\n * ${see_to_overridden}\r\n */</template><template autoinsert\="true" context\="newtype_context" deleted\="false" description\="Newly created files" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.newtype" name\="newtype">${filecomment}\r\n${package_declaration}\r\n\r\n${typecomment}\r\n${type_declaration}</template><template autoinsert\="true" context\="catchblock_context" deleted\="false" description\="Code in new catch blocks" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.catchblock" name\="catchblock">// ${todo} Auto-generated catch block\r\n${exception_var}.printStackTrace();</template><template autoinsert\="true" context\="methodbody_context" deleted\="false" description\="Code in created method stubs" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.methodbody" name\="methodbody">// ${todo} Auto-generated method stub\r\n${body_statement}</template><template autoinsert\="true" context\="constructorbody_context" deleted\="false" description\="Code in created constructor stubs" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.constructorbody" name\="constructorbody">${body_statement}\r\n// ${todo} Auto-generated constructor stub</template><template autoinsert\="true" context\="getterbody_context" deleted\="false" description\="Code in created getters" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.getterbody" name\="getterbody">return ${field};</template><template autoinsert\="true" context\="setterbody_context" deleted\="false" description\="Code in created setters" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.setterbody" name\="setterbody">${field} \= ${param};</template><template autoinsert\="true" context\="delegatecomment_context" deleted\="false" description\="Comment for delegate methods" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.delegatecomment" name\="delegatecomment">/**\r\n * ${tags}\r\n * ${see_to_target}\r\n */</template><template autoinsert\="true" context\="classbody_context" deleted\="false" description\="Code in new class type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.classbody" name\="classbody">\r\n</template><template autoinsert\="true" context\="interfacebody_context" deleted\="false" description\="Code in new interface type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.interfacebody" name\="interfacebody">\r\n</template><template autoinsert\="true" context\="enumbody_context" deleted\="false" description\="Code in new enum type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.enumbody" name\="enumbody">\r\n</template><template autoinsert\="true" context\="annotationbody_context" deleted\="false" description\="Code in new annotation type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.annotationbody" name\="annotationbody">\r\n</template></templates>
>+sp_cleanup.add_default_serial_version_id=true
>+sp_cleanup.add_generated_serial_version_id=false
>+sp_cleanup.add_missing_annotations=true
>+sp_cleanup.add_missing_deprecated_annotations=true
>+sp_cleanup.add_missing_methods=false
>+sp_cleanup.add_missing_nls_tags=false
>+sp_cleanup.add_missing_override_annotations=true
>+sp_cleanup.add_missing_override_annotations_interface_methods=false
>+sp_cleanup.add_serial_version_id=false
>+sp_cleanup.always_use_blocks=true
>+sp_cleanup.always_use_parentheses_in_expressions=true
>+sp_cleanup.always_use_this_for_non_static_field_access=false
>+sp_cleanup.always_use_this_for_non_static_method_access=false
>+sp_cleanup.convert_to_enhanced_for_loop=false
>+sp_cleanup.correct_indentation=false
>+sp_cleanup.format_source_code=true
>+sp_cleanup.format_source_code_changes_only=false
>+sp_cleanup.make_local_variable_final=true
>+sp_cleanup.make_parameters_final=true
>+sp_cleanup.make_private_fields_final=true
>+sp_cleanup.make_type_abstract_if_missing_method=false
>+sp_cleanup.make_variable_declarations_final=true
>+sp_cleanup.never_use_blocks=false
>+sp_cleanup.never_use_parentheses_in_expressions=false
>+sp_cleanup.on_save_use_additional_actions=true
>+sp_cleanup.organize_imports=true
>+sp_cleanup.qualify_static_field_accesses_with_declaring_class=false
>+sp_cleanup.qualify_static_member_accesses_through_instances_with_declaring_class=true
>+sp_cleanup.qualify_static_member_accesses_through_subtypes_with_declaring_class=true
>+sp_cleanup.qualify_static_member_accesses_with_declaring_class=true
>+sp_cleanup.qualify_static_method_accesses_with_declaring_class=false
>+sp_cleanup.remove_private_constructors=true
>+sp_cleanup.remove_trailing_whitespaces=true
>+sp_cleanup.remove_trailing_whitespaces_all=true
>+sp_cleanup.remove_trailing_whitespaces_ignore_empty=false
>+sp_cleanup.remove_unnecessary_casts=true
>+sp_cleanup.remove_unnecessary_nls_tags=true
>+sp_cleanup.remove_unused_imports=true
>+sp_cleanup.remove_unused_local_variables=false
>+sp_cleanup.remove_unused_private_fields=true
>+sp_cleanup.remove_unused_private_members=false
>+sp_cleanup.remove_unused_private_methods=true
>+sp_cleanup.remove_unused_private_types=true
>+sp_cleanup.sort_members=true
>+sp_cleanup.sort_members_all=false
>+sp_cleanup.use_blocks=true
>+sp_cleanup.use_blocks_only_for_return_and_throw=false
>+sp_cleanup.use_parentheses_in_expressions=true
>+sp_cleanup.use_this_for_non_static_field_access=true
>+sp_cleanup.use_this_for_non_static_field_access_only_if_necessary=true
>+sp_cleanup.use_this_for_non_static_method_access=true
>+sp_cleanup.use_this_for_non_static_method_access_only_if_necessary=true
>Index: META-INF/MANIFEST.MF
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/META-INF/MANIFEST.MF,v
>retrieving revision 1.16
>diff -u -r1.16 MANIFEST.MF
>--- META-INF/MANIFEST.MF	6 Jul 2010 14:01:15 -0000	1.16
>+++ META-INF/MANIFEST.MF	10 Jan 2011 16:37:25 -0000
>@@ -1,8 +1,8 @@
> Manifest-Version: 1.0
> Bundle-ManifestVersion: 2
> Bundle-Name: %Plugin.name
>-Bundle-SymbolicName: org.eclipse.ui.views.properties.tabbed;singleton:=true
>-Bundle-Version: 3.5.200.qualifier
>+Bundle-SymbolicName: org.eclipse.rap.ui.views.properties.tabbed;singleton:=true
>+Bundle-Version: 1.4.0.qualifier
> Bundle-Activator: org.eclipse.ui.internal.views.properties.tabbed.TabbedPropertyViewPlugin
> Bundle-Vendor: %Plugin.providerName
> Bundle-Localization: plugin
>@@ -10,10 +10,10 @@
>  org.eclipse.ui.internal.views.properties.tabbed.l10n;x-internal:=true,
>  org.eclipse.ui.internal.views.properties.tabbed.view;x-internal:=true,
>  org.eclipse.ui.views.properties.tabbed
>-Require-Bundle: org.eclipse.ui.forms;bundle-version="[3.3.0,4.0.0)";visibility:=reexport,
>- org.eclipse.ui.views;bundle-version="[3.2.0,4.0.0)";visibility:=reexport,
>- org.eclipse.ui;bundle-version="[3.3.0,4.0.0)";visibility:=reexport,
>- org.eclipse.core.runtime;bundle-version="[3.2.0,4.0.0)"
>+Require-Bundle: org.eclipse.core.runtime;bundle-version="[3.2.0,4.0.0)",
>+ org.eclipse.rap.ui;bundle-version="1.4.0",
>+ org.eclipse.rap.ui.forms;bundle-version="1.4.0",
>+ org.eclipse.rap.ui.views;bundle-version="1.4.0"
> Import-Package: com.ibm.icu.text
> Bundle-RequiredExecutionEnvironment: CDC-1.0/Foundation-1.0,
>  J2SE-1.3
>Index: src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyList.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyList.java,v
>retrieving revision 1.7
>diff -u -r1.7 TabbedPropertyList.java
>--- src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyList.java	20 Jan 2009 01:10:34 -0000	1.7
>+++ src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyList.java	10 Jan 2011 16:37:25 -0000
>@@ -4,7 +4,7 @@
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>  * http://www.eclipse.org/legal/epl-v10.html
>- * 
>+ *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>  *     Mariot Chauvin <mariot.chauvin@obeo.fr> - bug 259553
>@@ -13,24 +13,14 @@
> 
> import org.eclipse.jface.resource.JFaceResources;
> import org.eclipse.swt.SWT;
>-import org.eclipse.swt.accessibility.ACC;
>-import org.eclipse.swt.accessibility.Accessible;
>-import org.eclipse.swt.accessibility.AccessibleAdapter;
>-import org.eclipse.swt.accessibility.AccessibleControlAdapter;
>-import org.eclipse.swt.accessibility.AccessibleControlEvent;
>-import org.eclipse.swt.accessibility.AccessibleEvent;
> import org.eclipse.swt.events.ControlAdapter;
> import org.eclipse.swt.events.ControlEvent;
> import org.eclipse.swt.events.FocusEvent;
> import org.eclipse.swt.events.FocusListener;
> import org.eclipse.swt.events.MouseAdapter;
> import org.eclipse.swt.events.MouseEvent;
>-import org.eclipse.swt.events.MouseMoveListener;
>-import org.eclipse.swt.events.MouseTrackAdapter;
> import org.eclipse.swt.events.PaintEvent;
> import org.eclipse.swt.events.PaintListener;
>-import org.eclipse.swt.events.TraverseEvent;
>-import org.eclipse.swt.events.TraverseListener;
> import org.eclipse.swt.graphics.Color;
> import org.eclipse.swt.graphics.FontMetrics;
> import org.eclipse.swt.graphics.GC;
>@@ -44,7 +34,6 @@
> import org.eclipse.swt.widgets.Composite;
> import org.eclipse.swt.widgets.Display;
> import org.eclipse.swt.widgets.Event;
>-import org.eclipse.swt.widgets.Listener;
> import org.eclipse.ui.forms.FormColors;
> import org.eclipse.ui.internal.views.properties.tabbed.l10n.TabbedPropertyMessages;
> import org.eclipse.ui.views.properties.tabbed.ITabItem;
>@@ -53,7 +42,7 @@
> 
> /**
>  * Shows the list of tabs in the tabbed property sheet page.
>- * 
>+ *
>  * @author Anthony Hunter
>  */
> public class TabbedPropertyList
>@@ -128,7 +117,7 @@
> 
> 		/**
> 		 * Constructor for ListElement.
>-		 * 
>+		 *
> 		 * @param parent
> 		 *            the parent Composite.
> 		 * @param tab
>@@ -168,27 +157,29 @@
> 					}
> 				}
> 			});
>-			addMouseMoveListener(new MouseMoveListener() {
>-
>-				public void mouseMove(MouseEvent e) {
>-					if (!hover) {
>-						hover = true;
>-						redraw();
>-					}
>-				}
>-			});
>-			addMouseTrackListener(new MouseTrackAdapter() {
>-
>-				public void mouseExit(MouseEvent e) {
>-					hover = false;
>-					redraw();
>-				}
>-			});
>+			// RAP does not support this yet
>+//			addMouseMoveListener(new MouseMoveListener() {
>+//
>+//				public void mouseMove(MouseEvent e) {
>+//					if (!hover) {
>+//						hover = true;
>+//						redraw();
>+//					}
>+//				}
>+//			});
>+//			addMouseTrackListener(new MouseTrackAdapter() {
>+//
>+//				public void mouseExit(MouseEvent e) {
>+//					hover = false;
>+//					redraw();
>+//				}
>+//			});
>+			// /RAP
> 		}
> 
> 		/**
> 		 * Set selected value for this element.
>-		 * 
>+		 *
> 		 * @param selected
> 		 *            the selected value.
> 		 */
>@@ -199,7 +190,7 @@
> 
> 		/**
> 		 * Paint the element.
>-		 * 
>+		 *
> 		 * @param e
> 		 *            the paint event.
> 		 */
>@@ -286,7 +277,7 @@
> 
> 		/**
> 		 * Get the tab item.
>-		 * 
>+		 *
> 		 * @return the tab item.
> 		 */
> 		public ITabItem getTabItem() {
>@@ -307,7 +298,7 @@
> 
> 		/**
> 		 * Constructor for TopNavigationElement.
>-		 * 
>+		 *
> 		 * @param parent
> 		 *            the parent Composite.
> 		 */
>@@ -337,7 +328,7 @@
> 
> 		/**
> 		 * Paint the element.
>-		 * 
>+		 *
> 		 * @param e
> 		 *            the paint event.
> 		 */
>@@ -394,7 +385,7 @@
> 
> 		/**
> 		 * Constructor for BottomNavigationElement.
>-		 * 
>+		 *
> 		 * @param parent
> 		 *            the parent Composite.
> 		 */
>@@ -424,7 +415,7 @@
> 
> 		/**
> 		 * Paint the element.
>-		 * 
>+		 *
> 		 * @param e
> 		 *            the paint event.
> 		 */
>@@ -473,7 +464,7 @@
> 
> 	/**
> 	 * Constructor for TabbedPropertyList.
>-	 * 
>+	 *
> 	 * @param parent
> 	 *            the parent widget.
> 	 * @param factory
>@@ -514,27 +505,29 @@
> 				computeTopAndBottomTab();
> 			}
> 		});
>-		this.addTraverseListener(new TraverseListener() {
>-
>-			public void keyTraversed(TraverseEvent e) {
>-				if (e.detail == SWT.TRAVERSE_ARROW_PREVIOUS
>-					|| e.detail == SWT.TRAVERSE_ARROW_NEXT) {
>-					int nMax = elements.length - 1;
>-					int nCurrent = getSelectionIndex();
>-					if (e.detail == SWT.TRAVERSE_ARROW_PREVIOUS) {
>-						nCurrent -= 1;
>-						nCurrent = Math.max(0, nCurrent);
>-					} else if (e.detail == SWT.TRAVERSE_ARROW_NEXT) {
>-						nCurrent += 1;
>-						nCurrent = Math.min(nCurrent, nMax);
>-					}
>-					select(nCurrent);
>-					redraw();
>-				} else {
>-					e.doit = true;
>-				}
>-			}
>-		});
>+		// RAP not supported yet
>+//		this.addTraverseListener(new TraverseListener() {
>+//
>+//			public void keyTraversed(TraverseEvent e) {
>+//				if (e.detail == SWT.TRAVERSE_ARROW_PREVIOUS
>+//					|| e.detail == SWT.TRAVERSE_ARROW_NEXT) {
>+//					int nMax = elements.length - 1;
>+//					int nCurrent = getSelectionIndex();
>+//					if (e.detail == SWT.TRAVERSE_ARROW_PREVIOUS) {
>+//						nCurrent -= 1;
>+//						nCurrent = Math.max(0, nCurrent);
>+//					} else if (e.detail == SWT.TRAVERSE_ARROW_NEXT) {
>+//						nCurrent += 1;
>+//						nCurrent = Math.min(nCurrent, nMax);
>+//					}
>+//					select(nCurrent);
>+//					redraw();
>+//				} else {
>+//					e.doit = true;
>+//				}
>+//			}
>+//		});
>+		// /RAP
> 	}
> 
> 	/**
>@@ -551,7 +544,7 @@
> 	/**
> 	 * Returns the element with the given index from this list viewer. Returns
> 	 * <code>null</code> if the index is out of range.
>-	 * 
>+	 *
> 	 * @param index
> 	 *            the zero-based index
> 	 * @return the element at the given index, or <code>null</code> if the
>@@ -567,7 +560,7 @@
> 	/**
> 	 * Returns the zero-relative index of the item which is currently selected
> 	 * in the receiver, or -1 if no item is selected.
>-	 * 
>+	 *
> 	 * @return the index of the selected item
> 	 */
> 	public int getSelectionIndex() {
>@@ -592,7 +585,7 @@
> 
> 	/**
> 	 * Sets the new list elements.
>-	 * 
>+	 *
> 	 * @param children
> 	 */
> 	public void setElements(Object[] children) {
>@@ -628,7 +621,7 @@
> 
> 	/**
> 	 * Selects one of the elements in the list.
>-	 * 
>+	 *
> 	 * @param index
> 	 *            the index of the element to select.
> 	 */
>@@ -707,7 +700,7 @@
> 
> 	/**
> 	 * Get the dimensions of the provided string.
>-	 * 
>+	 *
> 	 * @param text
> 	 *            the string.
> 	 * @return the dimensions of the provided string.
>@@ -811,11 +804,11 @@
> 	/**
> 	 * Get the height of a tab. The height of the tab is the height of the text
> 	 * plus buffer.
>-	 * 
>+	 *
> 	 * @return the height of a tab.
> 	 */
> 	private int getTabHeight() {
>-		int tabHeight = getTextDimension("").y + INDENT; //$NON-NLS-1$ 
>+		int tabHeight = getTextDimension("").y + INDENT; //$NON-NLS-1$
> 		if (tabsThatFitInComposite == 1) {
> 			/*
> 			 * if only one tab will fix, reduce the size of the tab height so
>@@ -831,7 +824,7 @@
> 
> 	/**
> 	 * Determine if a downward scrolling is required.
>-	 * 
>+	 *
> 	 * @return true if downward scrolling is required.
> 	 */
> 	private boolean isDownScrollRequired() {
>@@ -841,7 +834,7 @@
> 
> 	/**
> 	 * Determine if an upward scrolling is required.
>-	 * 
>+	 *
> 	 * @return true if upward scrolling is required.
> 	 */
> 	private boolean isUpScrollRequired() {
>@@ -973,73 +966,75 @@
> 	 * Initialize the accessibility adapter.
> 	 */
> 	private void initAccessible() {
>-		final Accessible accessible = getAccessible();
>-		accessible.addAccessibleListener(new AccessibleAdapter() {
>-
>-			public void getName(AccessibleEvent e) {
>-				if (getSelectionIndex() != NONE) {
>-					e.result = elements[getSelectionIndex()].getTabItem()
>-							.getText();
>-				}
>-			}
>-
>-			public void getHelp(AccessibleEvent e) {
>-				if (getSelectionIndex() != NONE) {
>-					e.result = elements[getSelectionIndex()].getTabItem()
>-							.getText();
>-				}
>-			}
>-		});
>-
>-		accessible.addAccessibleControlListener(new AccessibleControlAdapter() {
>-
>-			public void getChildAtPoint(AccessibleControlEvent e) {
>-				Point pt = toControl(new Point(e.x, e.y));
>-				e.childID = (getBounds().contains(pt)) ? ACC.CHILDID_SELF
>-					: ACC.CHILDID_NONE;
>-			}
>-
>-			public void getLocation(AccessibleControlEvent e) {
>-				if (getSelectionIndex() != NONE) {
>-					Rectangle location = elements[getSelectionIndex()]
>-						.getBounds();
>-					Point pt = toDisplay(new Point(location.x, location.y));
>-					e.x = pt.x;
>-					e.y = pt.y;
>-					e.width = location.width;
>-					e.height = location.height;
>-				}
>-			}
>-
>-			public void getChildCount(AccessibleControlEvent e) {
>-				e.detail = 0;
>-			}
>-
>-			public void getRole(AccessibleControlEvent e) {
>-				e.detail = ACC.ROLE_TABITEM;
>-			}
>-
>-			public void getState(AccessibleControlEvent e) {
>-				e.detail = ACC.STATE_NORMAL | ACC.STATE_SELECTABLE
>-					| ACC.STATE_SELECTED | ACC.STATE_FOCUSED
>-					| ACC.STATE_FOCUSABLE;
>-			}
>-		});
>-
>-		addListener(SWT.Selection, new Listener() {
>-
>-			public void handleEvent(Event event) {
>-				if (isFocusControl()) {
>-					accessible.setFocus(ACC.CHILDID_SELF);
>-				}
>-			}
>-		});
>-
>-		addListener(SWT.FocusIn, new Listener() {
>-
>-			public void handleEvent(Event event) {
>-				accessible.setFocus(ACC.CHILDID_SELF);
>-			}
>-		});
>+		// RAP does not support this
>+//		final Accessible accessible = getAccessible();
>+//		accessible.addAccessibleListener(new AccessibleAdapter() {
>+//
>+//			public void getName(AccessibleEvent e) {
>+//				if (getSelectionIndex() != NONE) {
>+//					e.result = elements[getSelectionIndex()].getTabItem()
>+//							.getText();
>+//				}
>+//			}
>+//
>+//			public void getHelp(AccessibleEvent e) {
>+//				if (getSelectionIndex() != NONE) {
>+//					e.result = elements[getSelectionIndex()].getTabItem()
>+//							.getText();
>+//				}
>+//			}
>+//		});
>+//
>+//		accessible.addAccessibleControlListener(new AccessibleControlAdapter() {
>+//
>+//			public void getChildAtPoint(AccessibleControlEvent e) {
>+//				Point pt = toControl(new Point(e.x, e.y));
>+//				e.childID = (getBounds().contains(pt)) ? ACC.CHILDID_SELF
>+//					: ACC.CHILDID_NONE;
>+//			}
>+//
>+//			public void getLocation(AccessibleControlEvent e) {
>+//				if (getSelectionIndex() != NONE) {
>+//					Rectangle location = elements[getSelectionIndex()]
>+//						.getBounds();
>+//					Point pt = toDisplay(new Point(location.x, location.y));
>+//					e.x = pt.x;
>+//					e.y = pt.y;
>+//					e.width = location.width;
>+//					e.height = location.height;
>+//				}
>+//			}
>+//
>+//			public void getChildCount(AccessibleControlEvent e) {
>+//				e.detail = 0;
>+//			}
>+//
>+//			public void getRole(AccessibleControlEvent e) {
>+//				e.detail = ACC.ROLE_TABITEM;
>+//			}
>+//
>+//			public void getState(AccessibleControlEvent e) {
>+//				e.detail = ACC.STATE_NORMAL | ACC.STATE_SELECTABLE
>+//					| ACC.STATE_SELECTED | ACC.STATE_FOCUSED
>+//					| ACC.STATE_FOCUSABLE;
>+//			}
>+//		});
>+//
>+//		addListener(SWT.Selection, new Listener() {
>+//
>+//			public void handleEvent(Event event) {
>+//				if (isFocusControl()) {
>+//					accessible.setFocus(ACC.CHILDID_SELF);
>+//				}
>+//			}
>+//		});
>+//
>+//		addListener(SWT.FocusIn, new Listener() {
>+//
>+//			public void handleEvent(Event event) {
>+//				accessible.setFocus(ACC.CHILDID_SELF);
>+//			}
>+//		});
>+		// /RAP
> 	}
> }
>Index: src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyRegistry.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyRegistry.java,v
>retrieving revision 1.6
>diff -u -r1.6 TabbedPropertyRegistry.java
>--- src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyRegistry.java	6 Jul 2010 14:00:42 -0000	1.6
>+++ src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyRegistry.java	10 Jan 2011 16:37:25 -0000
>@@ -4,7 +4,7 @@
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>  * http://www.eclipse.org/legal/epl-v10.html
>- * 
>+ *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>  *******************************************************************************/
>@@ -44,7 +44,7 @@
> /**
>  * Provides information about the tabbed property extension points. Each tabbed
>  * property registry is associated with a unique contributor ID.
>- * 
>+ *
>  * @author Anthony Hunter
>  */
> public class TabbedPropertyRegistry {
>@@ -167,7 +167,7 @@
> 
> 	/**
> 	 * Gets the categories that are valid for this contributor.
>-	 * 
>+	 *
> 	 * @param configurationElement
> 	 *            the configuration element for this contributor.
> 	 */
>@@ -184,7 +184,7 @@
> 	/**
> 	 * Handle the error when an issue is found loading from the configuration
> 	 * element.
>-	 * 
>+	 *
> 	 * @param id
> 	 *            the configuration id.
> 	 * @param exception
>@@ -232,10 +232,10 @@
> 		if (contributorId == null) {
> 			return new IConfigurationElement[0];
> 		}
>+		// RAP modified to use 'org.eclipse.ui.views.properties.tabbed' namespace
> 		IExtensionPoint point = Platform.getExtensionRegistry()
> 				.getExtensionPoint(
>-						TabbedPropertyViewPlugin.getPlugin().getBundle()
>-								.getSymbolicName(), extensionPointId);
>+						"org.eclipse.ui.views.properties.tabbed", extensionPointId); //$NON-NLS-1$
> 		IConfigurationElement[] extensions = point.getConfigurationElements();
> 		List unordered = new ArrayList(extensions.length);
> 		for (int i = 0; i < extensions.length; i++) {
>@@ -267,7 +267,7 @@
> 
> 	/**
> 	 * Returns all section descriptors for the provided selection.
>-	 * 
>+	 *
> 	 * @param part
> 	 *            the workbench part containing the selection
> 	 * @param selection
>@@ -485,7 +485,7 @@
> 
> 	/**
> 	 * Gets the type mapper for the contributor.
>-	 * 
>+	 *
> 	 * @return the type mapper for the contributor.
> 	 */
> 	public ITypeMapper getTypeMapper() {
>@@ -494,7 +494,7 @@
> 
> 	/**
> 	 * Gets the label provider for the contributor.
>-	 * 
>+	 *
> 	 * @return the label provider for the contributor.
> 	 */
> 	public ILabelProvider getLabelProvider() {
>@@ -503,7 +503,7 @@
> 
> 	/**
> 	 * Gets the action provider for the contributor.
>-	 * 
>+	 *
> 	 * @return the action provider for the contributor.
> 	 */
> 	public IActionProvider getActionProvider() {
>@@ -512,7 +512,7 @@
> 
> 	/**
> 	 * Gets the tab list content provider for the contributor.
>-	 * 
>+	 *
> 	 * @return the tab list content provider for the contributor.
> 	 */
> 	public IStructuredContentProvider getTabListContentProvider() {
>@@ -525,7 +525,7 @@
> 	/**
> 	 * Handle the tab error when an issue is found loading from the
> 	 * configuration element.
>-	 * 
>+	 *
> 	 * @param configurationElement
> 	 *            the configuration element
> 	 */
>@@ -542,7 +542,7 @@
> 
> 	/**
> 	 * Disposes this registry.
>-	 * 
>+	 *
> 	 * @since 3.7
> 	 */
> 	public void dispose() {
>Index: src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyTitle.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyTitle.java,v
>retrieving revision 1.3
>diff -u -r1.3 TabbedPropertyTitle.java
>--- src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyTitle.java	16 Apr 2008 02:27:36 -0000	1.3
>+++ src/org/eclipse/ui/internal/views/properties/tabbed/view/TabbedPropertyTitle.java	10 Jan 2011 16:37:25 -0000
>@@ -4,7 +4,7 @@
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>  * http://www.eclipse.org/legal/epl-v10.html
>- * 
>+ *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>  *******************************************************************************/
>@@ -14,7 +14,6 @@
> import org.eclipse.swt.SWT;
> import org.eclipse.swt.custom.CLabel;
> import org.eclipse.swt.events.PaintEvent;
>-import org.eclipse.swt.events.PaintListener;
> import org.eclipse.swt.graphics.Color;
> import org.eclipse.swt.graphics.Font;
> import org.eclipse.swt.graphics.FontData;
>@@ -31,7 +30,7 @@
> 
> /**
>  * The title in the tabbed property sheet page.
>- * 
>+ *
>  * @author Anthony Hunter
>  */
> public class TabbedPropertyTitle
>@@ -39,19 +38,19 @@
> 
> 	private CLabel label;
> 
>-	private Image image = null;
>+//	private Image image = null;
>+
>+//	private String text = null;
> 
>-	private String text = null;
>-	
> 	private static final String BLANK = ""; //$NON-NLS-1$
> 
> 	private static final String TITLE_FONT = "org.eclipse.ui.internal.views.properties.tabbed.view.TabbedPropertyTitle"; //$NON-NLS-1$
>-	
>+
> 	private TabbedPropertySheetWidgetFactory factory;
> 
> 	/**
> 	 * Constructor for TabbedPropertyTitle.
>-	 * 
>+	 *
> 	 * @param parent
> 	 *            the parent composite.
> 	 * @param factory
>@@ -62,17 +61,19 @@
> 		super(parent, SWT.NO_FOCUS);
> 		this.factory = factory;
> 
>-		this.addPaintListener(new PaintListener() {
>-
>-			public void paintControl(PaintEvent e) {
>-				if (image == null && (text == null || text.equals(BLANK))) {
>-					label.setVisible(false);
>-				} else {
>-					label.setVisible(true);
>-					drawTitleBackground(e);
>-				}
>-			}
>-		});
>+		// RAP not suppported
>+//		this.addPaintListener(new PaintListener() {
>+//
>+//			public void paintControl(PaintEvent e) {
>+//				if (image == null && (text == null || text.equals(BLANK))) {
>+//					label.setVisible(false);
>+//				} else {
>+//					label.setVisible(true);
>+//					drawTitleBackground(e);
>+//				}
>+//			}
>+//		});
>+		// /RAP
> 
> 		factory.getColors().initializeSectionToolBarColors();
> 		setBackground(factory.getColors().getBackground());
>@@ -87,17 +88,17 @@
> 		if (! JFaceResources.getFontRegistry().hasValueFor(TITLE_FONT)) {
> 			FontData[] fontData = JFaceResources.getFontRegistry().getBold(
> 					JFaceResources.DEFAULT_FONT).getFontData();
>-			/* title font is 2pt larger than that used in the tabs. */  
>+			/* title font is 2pt larger than that used in the tabs. */
> 			fontData[0].setHeight(fontData[0].getHeight() + 2);
> 			JFaceResources.getFontRegistry().put(TITLE_FONT, fontData);
> 		}
> 		font = JFaceResources.getFont(TITLE_FONT);
>-		
>+
> 		label = factory.createCLabel(this, BLANK);
>-		label.setBackground(new Color[] {
>-				factory.getColors().getColor(IFormColors.H_GRADIENT_END),
>-				factory.getColors().getColor(IFormColors.H_GRADIENT_START) },
>-				new int[] { 100 }, true);
>+//		label.setBackground(new Color[] {
>+//				factory.getColors().getColor(IFormColors.H_GRADIENT_END),
>+//				factory.getColors().getColor(IFormColors.H_GRADIENT_START) },
>+//				new int[] { 100 }, true);
> 		label.setFont(font);
> 		label.setForeground(factory.getColors().getColor(IFormColors.TITLE));
> 		FormData data = new FormData();
>@@ -118,10 +119,10 @@
> 	 */
> 	protected void drawTitleBackground(PaintEvent e) {
> 		Rectangle bounds = getClientArea();
>-		label.setBackground(new Color[] {
>-				factory.getColors().getColor(IFormColors.H_GRADIENT_END),
>-				factory.getColors().getColor(IFormColors.H_GRADIENT_START) },
>-				new int[] { 100 }, true);
>+//		label.setBackground(new Color[] {
>+//				factory.getColors().getColor(IFormColors.H_GRADIENT_END),
>+//				factory.getColors().getColor(IFormColors.H_GRADIENT_START) },
>+//				new int[] { 100 }, true);
> 		Color bg = factory.getColors().getColor(IFormColors.H_GRADIENT_END);
> 		Color gbg = factory.getColors().getColor(IFormColors.H_GRADIENT_START);
> 		GC gc = e.gc;
>@@ -142,19 +143,19 @@
> 
> 	/**
> 	 * Set the text label.
>-	 * 
>+	 *
> 	 * @param text
> 	 *            the text label.
> 	 * @param image
> 	 *            the image for the label.
> 	 */
> 	public void setTitle(String text, Image image) {
>-		this.text = text;
>-		this.image = image;
>+//		this.text = text;
>+//		this.image = image;
> 		if (text != null) {
> 			label.setText(text);
> 		} else {
>-			label.setText(BLANK); 
>+			label.setText(BLANK);
> 		}
> 		label.setImage(image);
> 		redraw();
>Index: src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetPage.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetPage.java,v
>retrieving revision 1.17
>diff -u -r1.17 TabbedPropertySheetPage.java
>--- src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetPage.java	9 Jan 2009 18:14:59 -0000	1.17
>+++ src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetPage.java	10 Jan 2011 16:37:25 -0000
>@@ -4,7 +4,7 @@
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>  * http://www.eclipse.org/legal/epl-v10.html
>- * 
>+ *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>  *******************************************************************************/
>@@ -34,14 +34,12 @@
> import org.eclipse.swt.events.ControlEvent;
> import org.eclipse.swt.graphics.Image;
> import org.eclipse.swt.graphics.Point;
>-import org.eclipse.swt.graphics.Rectangle;
> import org.eclipse.swt.layout.FillLayout;
> import org.eclipse.swt.layout.FormAttachment;
> import org.eclipse.swt.layout.FormData;
> import org.eclipse.swt.layout.FormLayout;
> import org.eclipse.swt.widgets.Composite;
> import org.eclipse.swt.widgets.Control;
>-import org.eclipse.swt.widgets.ScrollBar;
> import org.eclipse.ui.IActionBars;
> import org.eclipse.ui.IEditorPart;
> import org.eclipse.ui.IPartListener;
>@@ -61,7 +59,7 @@
> 
> /**
>  * A property sheet page that provides a tabbed UI.
>- * 
>+ *
>  * @author Anthony Hunter
>  */
> public class TabbedPropertySheetPage
>@@ -138,7 +136,7 @@
> 
> 		/**
> 		 * Constructor that takes in a contributor id taken from a selection.
>-		 * 
>+		 *
> 		 * @param contributorId
> 		 *            the contributor id.
> 		 */
>@@ -265,7 +263,7 @@
> 
> 	/**
> 	 * create a new tabbed property sheet page.
>-	 * 
>+	 *
> 	 * @param tabbedPropertySheetPageContributor
> 	 *            the tabbed property sheet page contributor.
> 	 */
>@@ -273,15 +271,15 @@
> 			ITabbedPropertySheetPageContributor tabbedPropertySheetPageContributor) {
> 		this(tabbedPropertySheetPageContributor, true);
> 	}
>-	
>+
> 	/**
> 	 * create a new tabbed property sheet page.
> 	 *
> 	 * @param tabbedPropertySheetPageContributor
>-	 *            the tabbed property sheet page contributor.	  
>+	 *            the tabbed property sheet page contributor.
> 	 * @param showTitleBar
>-	 *            boolean indicating if the title bar should be shown; 
>-	 *            default value is <code>true</code> 	
>+	 *            boolean indicating if the title bar should be shown;
>+	 *            default value is <code>true</code>
> 	 * @since 3.5
> 	 */
> 	public TabbedPropertySheetPage(
>@@ -297,7 +295,7 @@
> 
> 	/**
> 	 * Handle the part activated event.
>-	 * 
>+	 *
> 	 * @param part
> 	 *            the new activated part.
> 	 */
>@@ -350,7 +348,7 @@
> 		widgetFactory = new TabbedPropertySheetWidgetFactory();
> 		tabbedPropertyComposite = new TabbedPropertyComposite(parent,
> 			widgetFactory, hasTitleBar);
>-		widgetFactory.paintBordersFor(tabbedPropertyComposite);
>+//		widgetFactory.paintBordersFor(tabbedPropertyComposite);
> 		tabbedPropertyComposite.setLayout(new FormLayout());
> 		FormData formData = new FormData();
> 		formData.left = new FormAttachment(0, 0);
>@@ -392,7 +390,7 @@
> 
> 	/**
> 	 * Initialize the contributor with the provided contributor id.
>-	 * 
>+	 *
> 	 * @param contributorId
> 	 *            the contributor id.
> 	 */
>@@ -432,7 +430,7 @@
> 
> 	/**
> 	 * Gets the tab list content provider for the contributor.
>-	 * 
>+	 *
> 	 * @return the tab list content provider for the contributor.
> 	 */
> 	protected IStructuredContentProvider getTabListContentProvider() {
>@@ -499,7 +497,7 @@
> 				contributor);
> 			registry = null;
> 		}
>-        
>+
>         contributor = null;
>         currentSelection = null;
> 	}
>@@ -524,8 +522,8 @@
> 		} else if (contributor instanceof IViewPart) {
>             IViewPart viewPart = (IViewPart) contributor;
>             partActionBars = viewPart.getViewSite().getActionBars();
>-        } 
>-        
>+        }
>+
>         if (partActionBars != null) {
>             IAction action = partActionBars.getGlobalActionHandler(ActionFactory.UNDO
>                 .getId());
>@@ -533,7 +531,7 @@
>                 actionBars.setGlobalActionHandler(ActionFactory.UNDO.getId(), action);
>             }
>             action = partActionBars.getGlobalActionHandler(ActionFactory.REDO
>-                .getId()); 
>+                .getId());
>             if (action != null) {
>                 actionBars.setGlobalActionHandler(ActionFactory.REDO.getId(), action);
>             }
>@@ -570,9 +568,9 @@
> 	}
> 
> 	/**
>-	 * Resize the scrolled composite enclosing the sections, which may result in the 
>+	 * Resize the scrolled composite enclosing the sections, which may result in the
> 	 * addition or removal of scroll bars.
>-	 * 
>+	 *
> 	 * @since 3.5
> 	 */
> 	public void resizeScrolledComposite() {
>@@ -581,29 +579,29 @@
> 			Composite sizeReference = (Composite) tabToComposite
> 				.get(currentTab);
> 			if (sizeReference != null) {
>-				currentTabSize = sizeReference.computeSize(SWT.DEFAULT, SWT.DEFAULT); 
>+				currentTabSize = sizeReference.computeSize(SWT.DEFAULT, SWT.DEFAULT);
> 			}
> 		}
> 		tabbedPropertyComposite.getScrolledComposite().setMinSize(
> 				currentTabSize);
> 
>-		ScrollBar verticalScrollBar = tabbedPropertyComposite
>-				.getScrolledComposite().getVerticalBar();
>-		if (verticalScrollBar != null) {
>-			Rectangle clientArea = tabbedPropertyComposite
>-					.getScrolledComposite().getClientArea();
>-			int increment = clientArea.height - 5;
>-			verticalScrollBar.setPageIncrement(increment);
>-		}
>-
>-		ScrollBar horizontalScrollBar = tabbedPropertyComposite
>-				.getScrolledComposite().getHorizontalBar();
>-		if (horizontalScrollBar != null) {
>-			Rectangle clientArea = tabbedPropertyComposite
>-					.getScrolledComposite().getClientArea();
>-			int increment = clientArea.width - 5;
>-			horizontalScrollBar.setPageIncrement(increment);
>-		}
>+//		ScrollBar verticalScrollBar = tabbedPropertyComposite
>+//				.getScrolledComposite().getVerticalBar();
>+//		if (verticalScrollBar != null) {
>+//			Rectangle clientArea = tabbedPropertyComposite
>+//					.getScrolledComposite().getClientArea();
>+//			int increment = clientArea.height - 5;
>+//			verticalScrollBar.setPageIncrement(increment);
>+//		}
>+
>+//		ScrollBar horizontalScrollBar = tabbedPropertyComposite
>+//				.getScrolledComposite().getHorizontalBar();
>+//		if (horizontalScrollBar != null) {
>+//			Rectangle clientArea = tabbedPropertyComposite
>+//					.getScrolledComposite().getClientArea();
>+//			int increment = clientArea.width - 5;
>+//			horizontalScrollBar.setPageIncrement(increment);
>+//		}
> 	}
> 
> 	private void disposeTabs(Collection tabs) {
>@@ -673,7 +671,7 @@
> 
> 	/**
> 	 * Create the tab contents for the provided tab descriptor.
>-	 * 
>+	 *
> 	 * @param tabDescriptor
> 	 *            the tab descriptor.
> 	 * @return the tab contents.
>@@ -685,7 +683,7 @@
> 
> 	/**
> 	 * Helper method for creating property tab composites.
>-	 * 
>+	 *
> 	 * @return the property tab composite.
> 	 */
> 	private Composite createTabComposite() {
>@@ -751,7 +749,7 @@
> 
> 	/**
> 	 * Get the currently active tab.
>-	 * 
>+	 *
> 	 * @return the currently active tab.
> 	 * @since 3.4
> 	 */
>@@ -761,7 +759,7 @@
> 
> 	/**
> 	 * Handle the tab selected change event.
>-	 * 
>+	 *
> 	 * @param tabDescriptor
> 	 *            the new selected tab.
> 	 */
>@@ -780,7 +778,7 @@
> 
> 	/**
> 	 * Add a tab selection listener.
>-	 * 
>+	 *
> 	 * @param listener
> 	 *            a tab selection listener.
> 	 */
>@@ -790,7 +788,7 @@
> 
> 	/**
> 	 * Remove a tab selection listener.
>-	 * 
>+	 *
> 	 * @param listener
> 	 *            a tab selection listener.
> 	 */
>@@ -813,7 +811,7 @@
> 
> 	/**
> 	 * Get the widget factory.
>-	 * 
>+	 *
> 	 * @return the widget factory.
> 	 */
> 	public TabbedPropertySheetWidgetFactory getWidgetFactory() {
>@@ -851,7 +849,7 @@
> 
>     /**
>      * Retrieve the contributor from the selection.
>-     * 
>+     *
>      * @param object -
>      *            the selected element
>      * @return the TabbedPropertySheetPageContributor or null if not applicable
>@@ -888,7 +886,7 @@
> 	 * It is possible for elements in a selection to implement
> 	 * ITabbedPropertySheetPageContributor to provide a different contributor id
> 	 * and thus a differenent registry.
>-	 * 
>+	 *
> 	 * @param selection
> 	 *            the current selection in the active workbench part.
> 	 */
>@@ -907,14 +905,14 @@
> 		}
> 
>         ITabbedPropertySheetPageContributor newContributor = getTabbedPropertySheetPageContributor(structuredSelection.getFirstElement());
>-        
>+
> 		if (newContributor == null) {
> 			/**
> 			 * selection does not implement or adapt ITabbedPropertySheetPageContributor.
> 			 */
> 			newContributor = contributor;
> 		}
>-		
>+
>         String selectionContributorId = newContributor.getContributorId();
> 		if (selectionContributorId.equals(currentContributorId)) {
> 			/**
>@@ -970,7 +968,7 @@
> 
> 	/**
> 	 * Returns the currently selected tab.
>-	 * 
>+	 *
> 	 * @return the currently selected tab or <code>null</code> if there is no
> 	 *         tab selected.
> 	 * @since 3.5
>@@ -988,7 +986,7 @@
> 
> 	/**
> 	 * Returns the list of currently active tabs.
>-	 * 
>+	 *
> 	 * @return the currently active tabs.
> 	 * @since 3.5
> 	 */
>@@ -1001,10 +999,10 @@
> 		}
> 		return new ITabDescriptor[] {};
> 	}
>-	
>+
> 	/**
> 	 * Set the currently selected tab to be that of the provided tab id.
>-	 * 
>+	 *
> 	 * @param id
> 	 *            The string id of the tab to select.
> 	 * @since 3.5
>@@ -1022,26 +1020,26 @@
> 			}
> 		}
> 	}
>-	
>+
> 	/**
>      * Returns text of the properties title for given selection. If selection is null,
>      * then currentSelection is used
>-     * 
>-	 * @param selection Selection whose properties title text is to be returned 
>+     *
>+	 * @param selection Selection whose properties title text is to be returned
>      * @return String representing title text.
> 	 * @since 3.5
>-     */	
>+     */
>     public String getTitleText(ISelection selection) {
>     	if (selection == null) {
>     		selection = currentSelection;
>     	}
>     	return registry.getLabelProvider().getText(selection);
>     }
>-    
>+
>     /**
>      * Returns the title image for given selection. If selection is null,
>-     * then currentSelection is used. 
>-     * 
>+     * then currentSelection is used.
>+     *
>      * @param selection Selection whose properties title image is to be returned
>      * @return Image that is used as a title image.
>      * @since 3.5
>Index: src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetWidgetFactory.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.ui.views.properties.tabbed/src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetWidgetFactory.java,v
>retrieving revision 1.3
>diff -u -r1.3 TabbedPropertySheetWidgetFactory.java
>--- src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetWidgetFactory.java	29 Aug 2006 13:27:39 -0000	1.3
>+++ src/org/eclipse/ui/views/properties/tabbed/TabbedPropertySheetWidgetFactory.java	10 Jan 2011 16:37:25 -0000
>@@ -4,7 +4,7 @@
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>  * http://www.eclipse.org/legal/epl-v10.html
>- * 
>+ *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>  *******************************************************************************/
>@@ -25,7 +25,7 @@
> 
> /**
>  * A FormToolkit customized for use by tabbed property sheet page.
>- * 
>+ *
>  * @author Anthony Hunter
>  */
> public class TabbedPropertySheetWidgetFactory
>@@ -40,7 +40,7 @@
> 
>     /**
>      * Creates the tab folder as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the composite parent.
>      * @param style
>@@ -54,7 +54,7 @@
> 
>     /**
>      * Creates the tab item as a part of the tab folder.
>-     * 
>+     *
>      * @param tabFolder
>      *            the parent.
>      * @param style
>@@ -68,7 +68,7 @@
> 
>     /**
>      * Creates the list as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the composite parent.
>      * @param style
>@@ -82,7 +82,7 @@
> 
>     public Composite createComposite(Composite parent, int style) {
>         Composite c = super.createComposite(parent, style);
>-        paintBordersFor(c);
>+//        paintBordersFor(c);
>         return c;
>     }
> 
>@@ -93,7 +93,7 @@
> 
>     /**
>      * Creates a plain composite as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the composite parent.
>      * @param style
>@@ -103,13 +103,13 @@
>     public Composite createPlainComposite(Composite parent, int style) {
>         Composite c = super.createComposite(parent, style);
>         c.setBackground(parent.getBackground());
>-        paintBordersFor(c);
>+//        paintBordersFor(c);
>         return c;
>     }
> 
>     /**
>      * Creates a scrolled composite as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the composite parent.
>      * @param style
>@@ -124,7 +124,7 @@
> 
>     /**
>      * Creates a combo box as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the combo box parent.
>      * @param comboStyle
>@@ -143,7 +143,7 @@
> 
>     /**
>      * Creates a combo box as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the combo box parent.
>      * @return the combo box.
>@@ -154,7 +154,7 @@
> 
>     /**
>      * Creates a group as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the group parent.
>      * @param text
>@@ -171,7 +171,7 @@
> 
>     /**
>      * Creates a flat form composite as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the composite parent.
>      * @return the composite.
>@@ -188,7 +188,7 @@
> 
>     /**
>      * Creates a label as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the label parent.
>      * @param text
>@@ -201,7 +201,7 @@
> 
>     /**
>      * Creates a label as a part of the form.
>-     * 
>+     *
>      * @param parent
>      *            the label parent.
>      * @param text

Actions: View | Diff

Attachments on bug 333886: 186400 | 186415