Skip to content

Commit cb8e80b

Browse files
committed
Remove unnecessary hover image descriptors
All explicit hover images use the same or equal image descriptors than used for the original image of an action rendering them obsolete. This change removes the unnecessary hover image specifications.
1 parent 6e312ce commit cb8e80b

File tree

9 files changed

+0
-16
lines changed

9 files changed

+0
-16
lines changed

bundles/org.eclipse.ltk.ui.refactoring/src/org/eclipse/ltk/internal/ui/refactoring/PreviewWizardPage.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,6 @@ public NextChange() {
130130
setId(NEXT_CHANGE_ID);
131131
setImageDescriptor(CompareUI.DESC_ETOOL_NEXT);
132132
setDisabledImageDescriptor(CompareUI.DESC_DTOOL_NEXT);
133-
setHoverImageDescriptor(CompareUI.DESC_CTOOL_NEXT);
134133
setToolTipText(RefactoringUIMessages.PreviewWizardPage_next_Change);
135134
PlatformUI.getWorkbench().getHelpSystem().setHelp(this, IRefactoringHelpContextIds.NEXT_CHANGE_ACTION);
136135
}
@@ -145,7 +144,6 @@ public PreviousChange() {
145144
setId(PREVIOUS_CHANGE_ID);
146145
setImageDescriptor(CompareUI.DESC_ETOOL_PREV);
147146
setDisabledImageDescriptor(CompareUI.DESC_DTOOL_PREV);
148-
setHoverImageDescriptor(CompareUI.DESC_CTOOL_PREV);
149147
setToolTipText(RefactoringUIMessages.PreviewWizardPage_previous_Change);
150148
PlatformUI.getWorkbench().getHelpSystem().setHelp(this, IRefactoringHelpContextIds.PREVIOUS_CHANGE_ACTION);
151149
}

bundles/org.eclipse.ltk.ui.refactoring/src/org/eclipse/ltk/internal/ui/refactoring/RefactoringPluginImages.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,6 @@ private static void setImageDescriptors(IAction action, String type, String relP
145145
action.setDisabledImageDescriptor(id);
146146

147147
ImageDescriptor descriptor= create("e" + type, relPath, true); //$NON-NLS-1$
148-
action.setHoverImageDescriptor(descriptor);
149148
action.setImageDescriptor(descriptor);
150149
}
151150

bundles/org.eclipse.ltk.ui.refactoring/src/org/eclipse/ltk/internal/ui/refactoring/RefactoringStatusViewer.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,6 @@ private class NextProblem extends Action {
7474
public NextProblem() {
7575
setImageDescriptor(CompareUI.DESC_ETOOL_NEXT);
7676
setDisabledImageDescriptor(CompareUI.DESC_DTOOL_NEXT);
77-
setHoverImageDescriptor(CompareUI.DESC_CTOOL_NEXT);
7877
setToolTipText(RefactoringUIMessages.ErrorWizardPage_next_Change);
7978
PlatformUI.getWorkbench().getHelpSystem().setHelp(this, IRefactoringHelpContextIds.NEXT_PROBLEM_ACTION);
8079
}
@@ -97,7 +96,6 @@ private class PreviousProblem extends Action {
9796
public PreviousProblem() {
9897
setImageDescriptor(CompareUI.DESC_ETOOL_PREV);
9998
setDisabledImageDescriptor(CompareUI.DESC_DTOOL_PREV);
100-
setHoverImageDescriptor(CompareUI.DESC_CTOOL_PREV);
10199
setToolTipText(RefactoringUIMessages.ErrorWizardPage_previous_Change);
102100
PlatformUI.getWorkbench().getHelpSystem().setHelp(this, IRefactoringHelpContextIds.PREVIOUS_PROBLEM_ACTION);
103101
}

bundles/org.eclipse.search/search/org/eclipse/search/internal/ui/CopyToClipboardAction.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@ public CopyToClipboardAction() {
6161
ISharedImages workbenchImages= PlatformUI.getWorkbench().getSharedImages();
6262
setDisabledImageDescriptor(workbenchImages.getImageDescriptor(ISharedImages.IMG_TOOL_COPY_DISABLED));
6363
setImageDescriptor(workbenchImages.getImageDescriptor(ISharedImages.IMG_TOOL_COPY));
64-
setHoverImageDescriptor(workbenchImages.getImageDescriptor(ISharedImages.IMG_TOOL_COPY));
6564

6665
}
6766

bundles/org.eclipse.search/search/org/eclipse/search/internal/ui/SearchPluginImages.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ public static void setImageDescriptors(IAction action, String type, String relPa
112112
action.setDisabledImageDescriptor(create("d" + type, relPath, false)); //$NON-NLS-1$
113113

114114
ImageDescriptor desc= create("e" + type, relPath, true); //$NON-NLS-1$
115-
action.setHoverImageDescriptor(desc);
116115
action.setImageDescriptor(desc);
117116
}
118117

bundles/org.eclipse.ui.navigator.resources/src/org/eclipse/ui/internal/navigator/resources/ProjectExplorerFilterActionGroup.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ public void makeActions() {
3939
String imageFilePath = "icons/full/elcl16/filter_ps.svg"; //$NON-NLS-1$
4040
ResourceLocator.imageDescriptorFromBundle(getClass(), imageFilePath).ifPresent(d -> {
4141
selectFiltersAction.setImageDescriptor(d);
42-
selectFiltersAction.setHoverImageDescriptor(d);
4342
});
4443
}
4544

bundles/org.eclipse.ui.navigator.resources/src/org/eclipse/ui/internal/navigator/resources/plugin/NavigatorUIPluginImages.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -119,11 +119,6 @@ public static void setLocalImageDescriptors(IAction action, String iconName) {
119119
* - The relative path of the icon.
120120
*/
121121
public static void setImageDescriptors(IAction action, String type, String relPath) {
122-
// /*relPath= relPath.substring(NAVIGATORUI_NAME_PREFIX_LENGTH);*/
123-
// action.setDisabledImageDescriptor(create("d" + type, relPath));
124-
// //$NON-NLS-1$
125-
// action.setHoverImageDescriptor(create("c" + type, relPath));
126-
// //$NON-NLS-1$
127122
action.setImageDescriptor(create("e" + type, relPath)); //$NON-NLS-1$
128123
}
129124

bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/CommonNavigatorActionGroup.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,6 @@ private void makeActions() {
115115
String imageFilePath = "icons/full/elcl16/synced.svg"; //$NON-NLS-1$
116116
ResourceLocator.imageDescriptorFromBundle(getClass(), imageFilePath).ifPresent(d -> {
117117
toggleLinkingAction.setImageDescriptor(d);
118-
toggleLinkingAction.setHoverImageDescriptor(d);
119118
});
120119
service.activateHandler(toggleLinkingAction.getActionDefinitionId(),
121120
new ActionHandler(toggleLinkingAction));
@@ -128,7 +127,6 @@ private void makeActions() {
128127
String imageFilePath = "icons/full/elcl16/collapseall.svg"; //$NON-NLS-1$
129128
ResourceLocator.imageDescriptorFromBundle(getClass(), imageFilePath).ifPresent(d -> {
130129
collapseAllAction.setImageDescriptor(d);
131-
collapseAllAction.setHoverImageDescriptor(d);
132130
});
133131
collapseAllHandler = new CollapseAllHandler(commonViewer);
134132
service.activateHandler(CollapseAllHandler.COMMAND_ID, collapseAllHandler);

bundles/org.eclipse.ui.navigator/src/org/eclipse/ui/internal/navigator/filters/FilterActionGroup.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,6 @@ private void makeActions() {
144144
String imageFilePath = "icons/full/elcl16/filter_ps.svg"; //$NON-NLS-1$
145145
ResourceLocator.imageDescriptorFromBundle(getClass(), imageFilePath).ifPresent(d -> {
146146
selectFiltersAction.setImageDescriptor(d);
147-
selectFiltersAction.setHoverImageDescriptor(d);
148147
});
149148
filtersMenu = new MenuManager(CommonNavigatorMessages.FilterActionGroup_RecentFilters);
150149
menuListener = manager -> {

0 commit comments

Comments
 (0)