projects/dwt-addons: dwtx/jface/action/ActionContributionItem.d annotate

annotate dwtx/jface/action/ActionContributionItem.d @ 192:c3583c6ec027

Added missing default cases for switch statements

author	Frank Benoit <benoit@tionex.de>
date	Mon, 03 Nov 2008 22:52:26 +0100
parents	862b05e0334a
children

rev	line source
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1 /*******************************************************************************
90 7ffeace6c47f Update 3.4M7 to 3.4 Frank Benoit <benoit@tionex.de> parents: 72 diff changeset	2 * Copyright (c) 2000, 2008 IBM Corporation and others.
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	3 * All rights reserved. This program and the accompanying materials
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	4 * are made available under the terms of the Eclipse Public License v1.0
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	5 * which accompanies this distribution, and is available at
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	6 * http://www.eclipse.org/legal/epl-v10.html
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	7 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	8 * Contributors:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	9 * IBM Corporation - initial API and implementation
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	10 * Port to the D programming language:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	11 * Frank Benoit <benoit@tionex.de>
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	12 *******************************************************************************/
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	13 module dwtx.jface.action.ActionContributionItem;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	14
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	15 import dwtx.jface.action.ContributionItem;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	16 import dwtx.jface.action.IAction;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	17 import dwtx.jface.action.LegacyActionTools;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	18 import dwtx.jface.action.Action;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	19 import dwtx.jface.action.IMenuCreator;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	20 import dwtx.jface.action.IContributionManagerOverrides;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	21
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	22 import dwt.DWT;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	23 import dwt.graphics.GC;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	24 import dwt.graphics.Point;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	25 import dwt.widgets.Button;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	26 import dwt.widgets.Composite;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	27 import dwt.widgets.Display;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	28 import dwt.widgets.Event;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	29 import dwt.widgets.Item;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	30 import dwt.widgets.Listener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	31 import dwt.widgets.Menu;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	32 import dwt.widgets.MenuItem;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	33 import dwt.widgets.ToolBar;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	34 import dwt.widgets.ToolItem;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	35 import dwt.widgets.Widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	36 import dwtx.jface.action.ExternalActionManager;
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	37 import dwtx.core.commands.ExecutionException;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	38 import dwtx.core.commands.NotEnabledException;
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	39 import dwtx.jface.bindings.Trigger;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	40 import dwtx.jface.bindings.TriggerSequence;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	41 import dwtx.jface.bindings.keys.IKeyLookup;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	42 import dwtx.jface.bindings.keys.KeyLookupFactory;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	43 import dwtx.jface.bindings.keys.KeyStroke;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	44 import dwtx.jface.resource.ImageDescriptor;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	45 import dwtx.jface.resource.JFaceResources;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	46 import dwtx.jface.resource.LocalResourceManager;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	47 import dwtx.jface.resource.ResourceManager;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	48 import dwtx.jface.util.IPropertyChangeListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	49 import dwtx.jface.util.Policy;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	50 import dwtx.jface.util.PropertyChangeEvent;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	51
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	52 import dwt.dwthelper.utils;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	53 import dwt.dwthelper.Runnable;
167 862b05e0334a Add a wrapper for Thread Frank Benoit <benoit@tionex.de> parents: 90 diff changeset	54 import dwtx.dwtxhelper.JThread;
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	55 import tango.io.Stdout;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	56
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	57 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	58 * A contribution item which delegates to an action.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	59 * <p>
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	60 * This class may be instantiated; it is not intended to be subclassed.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	61 * </p>
90 7ffeace6c47f Update 3.4M7 to 3.4 Frank Benoit <benoit@tionex.de> parents: 72 diff changeset	62 * @noextend This class is not intended to be subclassed by clients.
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	63 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	64 public class ActionContributionItem : ContributionItem {
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	65 alias ContributionItem.fill fill;
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	66
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	67 /**
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	68 * Mode bit: Show text on tool items or buttons, even if an image is
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	69 * present. If this mode bit is not set, text is only shown on tool items if
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	70 * there is no image present.
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	71 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	72 * @since 3.0
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	73 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	74 public static int MODE_FORCE_TEXT = 1;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	75
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	76 /** a string inserted in the middle of text that has been shortened */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	77 private static const String ellipsis = "..."; //$NON-NLS-1$
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	78
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	79 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	80 * Stores the result of the action. False when the action returned failure.
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	81 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	82 private Boolean result = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	83
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	84 private static bool USE_COLOR_ICONS = true;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	85
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	86 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	87 * Returns whether color icons should be used in toolbars.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	88 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	89 * @return <code>true</code> if color icons should be used in toolbars,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	90 * <code>false</code> otherwise
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	91 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	92 public static bool getUseColorIconsInToolbars() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	93 return USE_COLOR_ICONS;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	94 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	95
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	96 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	97 * Sets whether color icons should be used in toolbars.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	98 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	99 * @param useColorIcons
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	100 * <code>true</code> if color icons should be used in toolbars,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	101 * <code>false</code> otherwise
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	102 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	103 public static void setUseColorIconsInToolbars(bool useColorIcons) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	104 USE_COLOR_ICONS = useColorIcons;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	105 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	106
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	107 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	108 * The presentation mode.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	109 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	110 private int mode = 0;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	111
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	112 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	113 * The action.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	114 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	115 private IAction action;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	116
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	117 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	118 * The listener for changes to the text of the action contributed by an
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	119 * external source.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	120 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	121 private const IPropertyChangeListener actionTextListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	122
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	123 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	124 * Remembers all images in use by this contribution item
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	125 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	126 private LocalResourceManager imageManager;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	127
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	128 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	129 * Listener for DWT button widget events.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	130 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	131 private Listener buttonListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	132
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	133 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	134 * Listener for DWT menu item widget events.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	135 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	136 private Listener menuItemListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	137
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	138 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	139 * Listener for action property change notifications.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	140 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	141 private const IPropertyChangeListener propertyListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	142
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	143 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	144 * Listener for DWT tool item widget events.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	145 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	146 private Listener toolItemListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	147
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	148 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	149 * The widget created for this item; <code>null</code> before creation and
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	150 * after disposal.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	151 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	152 private Widget widget = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	153
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	154 private Listener menuCreatorListener;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	155
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	156 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	157 * Creates a new contribution item from the given action. The id of the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	158 * action is used as the id of the item.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	159 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	160 * @param action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	161 * the action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	162 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	163 public this(IAction action) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	164 super(action.getId());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	165 this.action = action;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	166 actionTextListener = new class IPropertyChangeListener {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	167 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	168 * @see IPropertyChangeListener#propertyChange(PropertyChangeEvent)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	169 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	170 public void propertyChange(PropertyChangeEvent event) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	171 update(event.getProperty());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	172 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	173 };
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	174 propertyListener = new class IPropertyChangeListener {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	175 public void propertyChange(PropertyChangeEvent event) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	176 actionPropertyChange(event);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	177 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	178 };
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	179 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	180
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	181 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	182 * Handles a property change event on the action (forwarded by nested
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	183 * listener).
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	184 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	185 private void actionPropertyChange(PropertyChangeEvent e) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	186 // This code should be removed. Avoid using free asyncExec
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	187
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	188 if (isVisible() && widget !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	189 Display display = widget.getDisplay();
167 862b05e0334a Add a wrapper for Thread Frank Benoit <benoit@tionex.de> parents: 90 diff changeset	190 if (display.getThread() is JThread.currentThread().nativeThread()) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	191 update(e.getProperty());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	192 } else {
167 862b05e0334a Add a wrapper for Thread Frank Benoit <benoit@tionex.de> parents: 90 diff changeset	193 display.asyncExec(dgRunnable( (PropertyChangeEvent e_) {
862b05e0334a Add a wrapper for Thread Frank Benoit <benoit@tionex.de> parents: 90 diff changeset	194 update(e_.getProperty());
862b05e0334a Add a wrapper for Thread Frank Benoit <benoit@tionex.de> parents: 90 diff changeset	195 }, e));
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	196 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	197
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	198 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	199 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	200
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	201 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	202 * Compares this action contribution item with another object. Two action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	203 * contribution items are equal if they refer to the identical Action.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	204 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	205 public override int opEquals(Object o) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	206 if (!(cast(ActionContributionItem)o )) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	207 return false;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	208 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	209 return (cast(Object)action).opEquals(cast(Object)(cast(ActionContributionItem) o).action);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	210 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	211
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	212 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	213 * The <code>ActionContributionItem</code> implementation of this
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	214 * <code>IContributionItem</code> method creates an DWT
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	215 * <code>Button</code> for the action using the action's style. If the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	216 * action's checked property has been set, the button is created and primed
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	217 * to the value of the checked property.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	218 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	219 public override void fill(Composite parent) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	220 if (widget is null && parent !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	221 int flags = DWT.PUSH;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	222 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	223 if (action.getStyle() is IAction.AS_CHECK_BOX) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	224 flags = DWT.TOGGLE;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	225 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	226 if (action.getStyle() is IAction.AS_RADIO_BUTTON) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	227 flags = DWT.RADIO;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	228 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	229 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	230
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	231 Button b = new Button(parent, flags);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	232 b.setData(this);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	233 b.addListener(DWT.Dispose, getButtonListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	234 // Don't hook a dispose listener on the parent
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	235 b.addListener(DWT.Selection, getButtonListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	236 if (action.getHelpListener() !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	237 b.addHelpListener(action.getHelpListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	238 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	239 widget = b;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	240
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	241 update(null);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	242
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	243 // Attach some extra listeners.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	244 action.addPropertyChangeListener(propertyListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	245 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	246 String commandId = action.getActionDefinitionId();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	247 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	248 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	249
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	250 if ((callback !is null) && (commandId !is null)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	251 callback.addPropertyChangeListener(commandId,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	252 actionTextListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	253 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	254 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	255 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	256 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	257
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	258 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	259 * The <code>ActionContributionItem</code> implementation of this
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	260 * <code>IContributionItem</code> method creates an DWT
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	261 * <code>MenuItem</code> for the action using the action's style. If the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	262 * action's checked property has been set, a button is created and primed to
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	263 * the value of the checked property. If the action's menu creator property
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	264 * has been set, a cascading submenu is created.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	265 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	266 public override void fill(Menu parent, int index) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	267 if (widget is null && parent !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	268 int flags = DWT.PUSH;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	269 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	270 int style = action.getStyle();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	271 if (style is IAction.AS_CHECK_BOX) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	272 flags = DWT.CHECK;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	273 } else if (style is IAction.AS_RADIO_BUTTON) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	274 flags = DWT.RADIO;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	275 } else if (style is IAction.AS_DROP_DOWN_MENU) {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	276 flags = DWT.CASCADE;
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	277 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	278 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	279
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	280 MenuItem mi = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	281 if (index >= 0) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	282 mi = new MenuItem(parent, flags, index);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	283 } else {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	284 mi = new MenuItem(parent, flags);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	285 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	286 widget = mi;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	287
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	288 mi.setData(this);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	289 mi.addListener(DWT.Dispose, getMenuItemListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	290 mi.addListener(DWT.Selection, getMenuItemListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	291 if (action.getHelpListener() !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	292 mi.addHelpListener(action.getHelpListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	293 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	294
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	295 if (flags is DWT.CASCADE) {
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	296 // just create a proxy for now, if the user shows it then
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	297 // fill it in
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	298 Menu subMenu = new Menu(parent);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	299 subMenu.addListener(DWT.Show, getMenuCreatorListener());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	300 subMenu.addListener(DWT.Hide, getMenuCreatorListener());
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	301 mi.setMenu(subMenu);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	302 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	303
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	304 update(null);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	305
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	306 // Attach some extra listeners.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	307 action.addPropertyChangeListener(propertyListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	308 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	309 String commandId = action.getActionDefinitionId();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	310 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	311 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	312
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	313 if ((callback !is null) && (commandId !is null)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	314 callback.addPropertyChangeListener(commandId,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	315 actionTextListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	316 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	317 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	318 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	319 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	320
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	321 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	322 * The <code>ActionContributionItem</code> implementation of this ,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	323 * <code>IContributionItem</code> method creates an DWT
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	324 * <code>ToolItem</code> for the action using the action's style. If the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	325 * action's checked property has been set, a button is created and primed to
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	326 * the value of the checked property. If the action's menu creator property
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	327 * has been set, a drop-down tool item is created.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	328 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	329 public override void fill(ToolBar parent, int index) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	330 if (widget is null && parent !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	331 int flags = DWT.PUSH;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	332 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	333 int style = action.getStyle();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	334 if (style is IAction.AS_CHECK_BOX) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	335 flags = DWT.CHECK;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	336 } else if (style is IAction.AS_RADIO_BUTTON) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	337 flags = DWT.RADIO;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	338 } else if (style is IAction.AS_DROP_DOWN_MENU) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	339 flags = DWT.DROP_DOWN;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	340 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	341 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	342
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	343 ToolItem ti = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	344 if (index >= 0) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	345 ti = new ToolItem(parent, flags, index);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	346 } else {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	347 ti = new ToolItem(parent, flags);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	348 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	349 ti.setData(this);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	350 ti.addListener(DWT.Selection, getToolItemListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	351 ti.addListener(DWT.Dispose, getToolItemListener());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	352
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	353 widget = ti;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	354
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	355 update(null);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	356
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	357 // Attach some extra listeners.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	358 action.addPropertyChangeListener(propertyListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	359 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	360 String commandId = action.getActionDefinitionId();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	361 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	362 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	363
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	364 if ((callback !is null) && (commandId !is null)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	365 callback.addPropertyChangeListener(commandId,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	366 actionTextListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	367 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	368 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	369 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	370 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	371
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	372 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	373 * Returns the action associated with this contribution item.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	374 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	375 * @return the action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	376 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	377 public IAction getAction() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	378 return action;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	379 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	380
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	381 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	382 * Returns the listener for DWT button widget events.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	383 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	384 * @return a listener for button events
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	385 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	386 private Listener getButtonListener() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	387 if (buttonListener is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	388 buttonListener = new class Listener {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	389 public void handleEvent(Event event) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	390 switch (event.type) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	391 case DWT.Dispose:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	392 handleWidgetDispose(event);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	393 break;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	394 case DWT.Selection:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	395 Widget ew = event.widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	396 if (ew !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	397 handleWidgetSelection(event, (cast(Button) ew)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	398 .getSelection());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	399 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	400 break;
192 c3583c6ec027 Added missing default cases for switch statements Frank Benoit <benoit@tionex.de> parents: 167 diff changeset	401 default:
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	402 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	403 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	404 };
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	405 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	406 return buttonListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	407 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	408
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	409 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	410 * Returns the listener for DWT menu item widget events.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	411 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	412 * @return a listener for menu item events
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	413 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	414 private Listener getMenuItemListener() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	415 if (menuItemListener is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	416 menuItemListener = new class Listener {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	417 public void handleEvent(Event event) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	418 switch (event.type) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	419 case DWT.Dispose:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	420 handleWidgetDispose(event);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	421 break;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	422 case DWT.Selection:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	423 Widget ew = event.widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	424 if (ew !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	425 handleWidgetSelection(event, (cast(MenuItem) ew)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	426 .getSelection());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	427 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	428 break;
192 c3583c6ec027 Added missing default cases for switch statements Frank Benoit <benoit@tionex.de> parents: 167 diff changeset	429 default:
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	430 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	431 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	432 };
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	433 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	434 return menuItemListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	435 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	436
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	437 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	438 * Returns the presentation mode, which is the bitwise-or of the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	439 * <code>MODE_*</code> constants. The default mode setting is 0, meaning
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	440 * that for menu items, both text and image are shown (if present), but for
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	441 * tool items, the text is shown only if there is no image.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	442 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	443 * @return the presentation mode settings
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	444 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	445 * @since 3.0
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	446 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	447 public int getMode() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	448 return mode;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	449 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	450
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	451 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	452 * Returns the listener for DWT tool item widget events.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	453 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	454 * @return a listener for tool item events
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	455 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	456 private Listener getToolItemListener() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	457 if (toolItemListener is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	458 toolItemListener = new class Listener {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	459 public void handleEvent(Event event) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	460 switch (event.type) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	461 case DWT.Dispose:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	462 handleWidgetDispose(event);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	463 break;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	464 case DWT.Selection:
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	465 Widget ew = event.widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	466 if (ew !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	467 handleWidgetSelection(event, (cast(ToolItem) ew)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	468 .getSelection());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	469 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	470 break;
192 c3583c6ec027 Added missing default cases for switch statements Frank Benoit <benoit@tionex.de> parents: 167 diff changeset	471 default:
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	472 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	473 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	474 };
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	475 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	476 return toolItemListener;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	477 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	478
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	479 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	480 * Handles a widget dispose event for the widget corresponding to this item.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	481 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	482 private void handleWidgetDispose(Event e) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	483 // Check if our widget is the one being disposed.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	484 if (e.widget is widget) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	485 // Dispose of the menu creator.
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	486 if (action.getStyle() is IAction.AS_DROP_DOWN_MENU
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	487 && menuCreatorCalled) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	488 IMenuCreator mc = action.getMenuCreator();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	489 if (mc !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	490 mc.dispose();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	491 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	492 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	493
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	494 // Unhook all of the listeners.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	495 action.removePropertyChangeListener(propertyListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	496 if (action !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	497 String commandId = action.getActionDefinitionId();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	498 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	499 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	500
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	501 if ((callback !is null) && (commandId !is null)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	502 callback.removePropertyChangeListener(commandId,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	503 actionTextListener);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	504 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	505 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	506
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	507 // Clear the widget field.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	508 widget = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	509
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	510 disposeOldImages();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	511 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	512 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	513
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	514 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	515 * Handles a widget selection event.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	516 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	517 private void handleWidgetSelection(Event e, bool selection) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	518
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	519 Widget item = e.widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	520 if (item !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	521 int style = item.getStyle();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	522
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	523 if ((style & (DWT.TOGGLE \| DWT.CHECK)) !is 0) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	524 if (action.getStyle() is IAction.AS_CHECK_BOX) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	525 action.setChecked(selection);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	526 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	527 } else if ((style & DWT.RADIO) !is 0) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	528 if (action.getStyle() is IAction.AS_RADIO_BUTTON) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	529 action.setChecked(selection);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	530 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	531 } else if ((style & DWT.DROP_DOWN) !is 0) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	532 if (e.detail is 4) { // on drop-down button
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	533 if (action.getStyle() is IAction.AS_DROP_DOWN_MENU) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	534 IMenuCreator mc = action.getMenuCreator();
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	535 menuCreatorCalled = true;
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	536 ToolItem ti = cast(ToolItem) item;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	537 // we create the menu as a sub-menu of "dummy" so that
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	538 // we can use
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	539 // it in a cascading menu too.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	540 // If created on a DWT control we would get an DWT
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	541 // error...
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	542 // Menu dummy= new Menu(ti.getParent());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	543 // Menu m= mc.getMenu(dummy);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	544 // dummy.dispose();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	545 if (mc !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	546 Menu m = mc.getMenu(ti.getParent());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	547 if (m !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	548 // position the menu below the drop down item
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	549 Point point = ti.getParent().toDisplay(
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	550 new Point(e.x, e.y));
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	551 m.setLocation(point.x, point.y); // waiting
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	552 // for DWT
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	553 // 0.42
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	554 m.setVisible(true);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	555 return; // we don't fire the action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	556 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	557 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	558 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	559 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	560 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	561
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	562 ExternalActionManager.IExecuteCallback callback = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	563 String actionDefinitionId = action.getActionDefinitionId();
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	564 if (actionDefinitionId !is null) {
72 5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	565 Object obj = cast(Object) ExternalActionManager.getInstance()
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	566 .getCallback();
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	567 if (null !is cast(ExternalActionManager.IExecuteCallback)obj ) {
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	568 callback = cast(ExternalActionManager.IExecuteCallback) obj;
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	569 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	570 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	571
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	572 // Ensure action is enabled first.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	573 // See 1GAN3M6: ITPUI:WINNT - Any IAction in the workbench can be
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	574 // executed while disabled.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	575 if (action.isEnabled()) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	576 bool trace = Policy.TRACE_ACTIONS;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	577
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	578 long ms = 0L;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	579 if (trace) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	580 ms = System.currentTimeMillis();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	581 Stdout.formatln("Running action: {}", action.getText()); //$NON-NLS-1$
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	582 }
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	583
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	584 IPropertyChangeListener resultListener = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	585 if (callback !is null) {
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	586 resultListener = new class IPropertyChangeListener {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	587 public void propertyChange(PropertyChangeEvent event) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	588 // Check on result
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	589 if (event.getProperty().equals(IAction.RESULT)) {
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	590 if (null !is cast(Boolean)event.getNewValue() ) {
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	591 result = cast(Boolean) event.getNewValue();
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	592 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	593 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	594 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	595 };
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	596 action.addPropertyChangeListener(resultListener);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	597 callback.preExecute(action, e);
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	598 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	599
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	600 action.runWithEvent(e);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	601
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	602 if (callback !is null) {
72 5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	603 if (result is null \|\| result.opEquals(Boolean.TRUE)) {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	604 callback.postExecuteSuccess(action, Boolean.TRUE);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	605 } else {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	606 callback.postExecuteFailure(action,
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	607 new ExecutionException(action.getText()
72 5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	608 ~ " returned failure.")); //$NON-NLS-1$
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	609 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	610 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	611
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	612 if (resultListener !is null) {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	613 result = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	614 action.removePropertyChangeListener(resultListener);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	615 }
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	616 if (trace) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	617 Stdout.formatln("{} ms to run action: {}",(System.currentTimeMillis() - ms), action.getText()); //$NON-NLS-1$
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	618 }
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	619 } else {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	620 if (callback !is null) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	621 callback.notEnabled(action, new NotEnabledException(action
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	622 .getText()
72 5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	623 ~ " is not enabled.")); //$NON-NLS-1$
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	624 }
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	625 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	626 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	627 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	628
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	629 /*
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	630 * (non-Javadoc) Method declared on Object.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	631 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	632 public override hash_t toHash() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	633 return (cast(Object)action).toHash();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	634 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	635
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	636 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	637 * Returns whether the given action has any images.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	638 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	639 * @param actionToCheck
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	640 * the action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	641 * @return <code>true</code> if the action has any images,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	642 * <code>false</code> if not
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	643 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	644 private bool hasImages(IAction actionToCheck) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	645 return actionToCheck.getImageDescriptor() !is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	646 \|\| actionToCheck.getHoverImageDescriptor() !is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	647 \|\| actionToCheck.getDisabledImageDescriptor() !is null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	648 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	649
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	650 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	651 * Returns whether the command corresponding to this action is active.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	652 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	653 private bool isCommandActive() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	654 IAction actionToCheck = getAction();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	655
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	656 if (actionToCheck !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	657 String commandId = actionToCheck.getActionDefinitionId();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	658 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	659 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	660
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	661 if (callback !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	662 return callback.isActive(commandId);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	663 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	664 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	665 return true;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	666 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	667
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	668 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	669 * The action item implementation of this <code>IContributionItem</code>
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	670 * method returns <code>true</code> for menu items and <code>false</code>
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	671 * for everything else.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	672 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	673 public override bool isDynamic() {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	674 if (cast(MenuItem)widget ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	675 // Optimization. Only recreate the item is the check or radio style
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	676 // has changed.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	677 bool itemIsCheck = (widget.getStyle() & DWT.CHECK) !is 0;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	678 bool actionIsCheck = getAction() !is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	679 && getAction().getStyle() is IAction.AS_CHECK_BOX;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	680 bool itemIsRadio = (widget.getStyle() & DWT.RADIO) !is 0;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	681 bool actionIsRadio = getAction() !is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	682 && getAction().getStyle() is IAction.AS_RADIO_BUTTON;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	683 return (itemIsCheck !is actionIsCheck)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	684 \|\| (itemIsRadio !is actionIsRadio);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	685 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	686 return false;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	687 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	688
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	689 /*
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	690 * (non-Javadoc) Method declared on IContributionItem.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	691 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	692 public override bool isEnabled() {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	693 return action !is null && action.isEnabled();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	694 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	695
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	696 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	697 * Returns <code>true</code> if this item is allowed to enable,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	698 * <code>false</code> otherwise.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	699 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	700 * @return if this item is allowed to be enabled
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	701 * @since 2.0
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	702 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	703 protected bool isEnabledAllowed() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	704 if (getParent() is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	705 return true;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	706 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	707 auto value = getParent().getOverrides().getEnabled(this);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	708 return (value is null) ? true : value.value;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	709 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	710
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	711 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	712 * The <code>ActionContributionItem</code> implementation of this
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	713 * <code>ContributionItem</code> method extends the super implementation
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	714 * by also checking whether the command corresponding to this action is
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	715 * active.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	716 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	717 public override bool isVisible() {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	718 return super.isVisible() && isCommandActive();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	719 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	720
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	721 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	722 * Sets the presentation mode, which is the bitwise-or of the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	723 * <code>MODE_*</code> constants.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	724 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	725 * @param mode
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	726 * the presentation mode settings
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	727 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	728 * @since 3.0
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	729 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	730 public void setMode(int mode) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	731 this.mode = mode;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	732 update();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	733 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	734
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	735 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	736 * The action item implementation of this <code>IContributionItem</code>
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	737 * method calls <code>update(null)</code>.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	738 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	739 public override final void update() {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	740 update(null);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	741 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	742
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	743 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	744 * Synchronizes the UI with the given property.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	745 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	746 * @param propertyName
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	747 * the name of the property, or <code>null</code> meaning all
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	748 * applicable properties
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	749 */
43 ea8ff534f622 Fix override and super aliases Frank Benoit <benoit@tionex.de> parents: 39 diff changeset	750 public override void update(String propertyName) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	751 if (widget !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	752 // determine what to do
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	753 bool textChanged = propertyName is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	754 \|\| propertyName.equals(IAction.TEXT);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	755 bool imageChanged = propertyName is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	756 \|\| propertyName.equals(IAction.IMAGE);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	757 bool tooltipTextChanged = propertyName is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	758 \|\| propertyName.equals(IAction.TOOL_TIP_TEXT);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	759 bool enableStateChanged = propertyName is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	760 \|\| propertyName.equals(IAction.ENABLED)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	761 \|\| propertyName
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	762 .equals(IContributionManagerOverrides.P_ENABLED);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	763 bool checkChanged = (action.getStyle() is IAction.AS_CHECK_BOX \|\| action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	764 .getStyle() is IAction.AS_RADIO_BUTTON)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	765 && (propertyName is null \|\| propertyName
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	766 .equals(IAction.CHECKED));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	767
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	768 if (cast(ToolItem)widget ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	769 ToolItem ti = cast(ToolItem) widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	770 String text = action.getText();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	771 // the set text is shown only if there is no image or if forced
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	772 // by MODE_FORCE_TEXT
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	773 bool showText = text !is null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	774 && ((getMode() & MODE_FORCE_TEXT) !is 0 \|\| !hasImages(action));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	775
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	776 // only do the trimming if the text will be used
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	777 if (showText && text !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	778 text = Action.removeAcceleratorText(text);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	779 text = Action.removeMnemonics(text);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	780 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	781
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	782 if (textChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	783 String textToSet = showText ? text : ""; //$NON-NLS-1$
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	784 bool rightStyle = (ti.getParent().getStyle() & DWT.RIGHT) !is 0;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	785 if (rightStyle \|\| !ti.getText().equals(textToSet)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	786 // In addition to being required to update the text if
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	787 // it
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	788 // gets nulled out in the action, this is also a
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	789 // workaround
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	790 // for bug 50151: Using DWT.RIGHT on a ToolBar leaves
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	791 // blank space
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	792 ti.setText(textToSet);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	793 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	794 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	795
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	796 if (imageChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	797 // only substitute a missing image if it has no text
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	798 updateImages(!showText);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	799 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	800
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	801 if (tooltipTextChanged \|\| textChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	802 String toolTip = action.getToolTipText();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	803 if ((toolTip is null) \|\| (toolTip.length is 0)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	804 toolTip = text;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	805 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	806
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	807 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	808 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	809 String commandId = action.getActionDefinitionId();
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	810 if ((callback !is null) && (commandId !is null)
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	811 && (toolTip !is null)) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	812 String acceleratorText = callback
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	813 .getAcceleratorText(commandId);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	814 if (acceleratorText !is null
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	815 && acceleratorText.length !is 0) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	816 toolTip = JFaceResources.format(
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	817 "Toolbar_Tooltip_Accelerator", //$NON-NLS-1$
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	818 [ toolTip, acceleratorText ]);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	819 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	820 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	821
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	822 // if the text is showing, then only set the tooltip if
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	823 // different
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	824 if (!showText \|\| toolTip !is null && !toolTip.equals(text)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	825 ti.setToolTipText(toolTip);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	826 } else {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	827 ti.setToolTipText(null);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	828 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	829 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	830
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	831 if (enableStateChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	832 bool shouldBeEnabled = action.isEnabled()
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	833 && isEnabledAllowed();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	834
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	835 if (ti.getEnabled() !is shouldBeEnabled) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	836 ti.setEnabled(shouldBeEnabled);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	837 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	838 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	839
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	840 if (checkChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	841 bool bv = action.isChecked();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	842
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	843 if (ti.getSelection() !is bv) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	844 ti.setSelection(bv);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	845 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	846 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	847 return;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	848 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	849
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	850 if (cast(MenuItem)widget ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	851 MenuItem mi = cast(MenuItem) widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	852
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	853 if (textChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	854 int accelerator = 0;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	855 String acceleratorText = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	856 IAction updatedAction = getAction();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	857 String text = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	858 accelerator = updatedAction.getAccelerator();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	859 ExternalActionManager.ICallback callback = ExternalActionManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	860 .getInstance().getCallback();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	861
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	862 // Block accelerators that are already in use.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	863 if ((accelerator !is 0) && (callback !is null)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	864 && (callback.isAcceleratorInUse(accelerator))) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	865 accelerator = 0;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	866 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	867
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	868 /*
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	869 * Process accelerators on GTK in a special way to avoid Bug
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	870 * 42009. We will override the native input method by
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	871 * allowing these reserved accelerators to be placed on the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	872 * menu. We will only do this for "Ctrl+Shift+[0-9A-FU]".
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	873 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	874 String commandId = updatedAction
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	875 .getActionDefinitionId();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	876 if (("gtk".equals(DWT.getPlatform())) && (cast(ExternalActionManager.IBindingManagerCallback)callback ) //$NON-NLS-1$
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	877 && (commandId !is null)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	878 ExternalActionManager.IBindingManagerCallback bindingManagerCallback = cast(ExternalActionManager.IBindingManagerCallback) callback;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	879 IKeyLookup lookup = KeyLookupFactory.getDefault();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	880 TriggerSequence[] triggerSequences = bindingManagerCallback
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	881 .getActiveBindingsFor(commandId);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	882 for (int i = 0; i < triggerSequences.length; i++) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	883 TriggerSequence triggerSequence = triggerSequences[i];
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	884 Trigger[] triggers = triggerSequence
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	885 .getTriggers();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	886 if (triggers.length is 1) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	887 Trigger trigger = triggers[0];
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	888 if (cast(KeyStroke)trigger ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	889 KeyStroke currentKeyStroke = cast(KeyStroke) trigger;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	890 int currentNaturalKey = currentKeyStroke
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	891 .getNaturalKey();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	892 if ((currentKeyStroke.getModifierKeys() is (lookup
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	893 .getCtrl() \| lookup.getShift()))
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	894 && ((currentNaturalKey >= '0' && currentNaturalKey <= '9')
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	895 \|\| (currentNaturalKey >= 'A' && currentNaturalKey <= 'F') \|\| (currentNaturalKey is 'U'))) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	896 accelerator = currentKeyStroke
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	897 .getModifierKeys()
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	898 \| currentNaturalKey;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	899 acceleratorText = triggerSequence
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	900 .format();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	901 break;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	902 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	903 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	904 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	905 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	906 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	907
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	908 if (accelerator is 0) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	909 if ((callback !is null) && (commandId !is null)) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	910 acceleratorText = callback
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	911 .getAcceleratorText(commandId);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	912 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	913 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	914
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	915 IContributionManagerOverrides overrides = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	916
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	917 if (getParent() !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	918 overrides = getParent().getOverrides();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	919 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	920
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	921 if (overrides !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	922 text = getParent().getOverrides().getText(this);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	923 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	924
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	925 mi.setAccelerator(accelerator);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	926
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	927 if (text is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	928 text = updatedAction.getText();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	929 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	930
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	931 if (text !is null && acceleratorText is null) {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	932 // use extracted accelerator text in case accelerator
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	933 // cannot be fully represented in one int (e.g.
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	934 // multi-stroke keys)
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	935 acceleratorText = LegacyActionTools
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	936 .extractAcceleratorText(text);
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	937 if (acceleratorText is null && accelerator !is 0) {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	938 acceleratorText = Action
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	939 .convertAccelerator(accelerator);
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	940 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	941 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	942
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	943 if (text is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	944 text = ""; //$NON-NLS-1$
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	945 } else {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	946 text = Action.removeAcceleratorText(text);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	947 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	948
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	949 if (acceleratorText is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	950 mi.setText(text);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	951 } else {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	952 mi.setText(text ~ '\t' ~ acceleratorText);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	953 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	954 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	955
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	956 if (imageChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	957 updateImages(false);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	958 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	959
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	960 if (enableStateChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	961 bool shouldBeEnabled = action.isEnabled()
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	962 && isEnabledAllowed();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	963
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	964 if (mi.getEnabled() !is shouldBeEnabled) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	965 mi.setEnabled(shouldBeEnabled);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	966 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	967 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	968
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	969 if (checkChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	970 bool bv = action.isChecked();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	971
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	972 if (mi.getSelection() !is bv) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	973 mi.setSelection(bv);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	974 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	975 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	976
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	977 return;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	978 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	979
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	980 if (cast(Button)widget ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	981 Button button = cast(Button) widget;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	982
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	983 if (imageChanged) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	984 updateImages(false);
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	985 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	986
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	987 if (textChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	988 String text = action.getText();
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	989 bool showText = text !is null && ((getMode() & MODE_FORCE_TEXT) !is 0 \|\| !hasImages(action));
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	990 // only do the trimming if the text will be used
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	991 if (showText) {
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	992 text = Action.removeAcceleratorText(text);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	993 }
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	994 String textToSet = showText ? text : ""; //$NON-NLS-1$
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	995 button.setText(textToSet);
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	996 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	997
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	998 if (tooltipTextChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	999 button.setToolTipText(action.getToolTipText());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1000 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1001
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1002 if (enableStateChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1003 bool shouldBeEnabled = action.isEnabled()
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1004 && isEnabledAllowed();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1005
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1006 if (button.getEnabled() !is shouldBeEnabled) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1007 button.setEnabled(shouldBeEnabled);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1008 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1009 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1010
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1011 if (checkChanged) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1012 bool bv = action.isChecked();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1013
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1014 if (button.getSelection() !is bv) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1015 button.setSelection(bv);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1016 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1017 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1018 return;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1019 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1020 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1021 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1022
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1023 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1024 * Updates the images for this action.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1025 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1026 * @param forceImage
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1027 * <code>true</code> if some form of image is compulsory, and
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1028 * <code>false</code> if it is acceptable for this item to have
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1029 * no image
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1030 * @return <code>true</code> if there are images for this action,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1031 * <code>false</code> if not
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1032 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1033 private bool updateImages(bool forceImage) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1034
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1035 ResourceManager parentResourceManager = JFaceResources.getResources();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1036
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1037 if (cast(ToolItem)widget ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1038 if (USE_COLOR_ICONS) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1039 ImageDescriptor image = action.getHoverImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1040 if (image is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1041 image = action.getImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1042 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1043 ImageDescriptor disabledImage = action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1044 .getDisabledImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1045
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1046 // Make sure there is a valid image.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1047 if (image is null && forceImage) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1048 image = ImageDescriptor.getMissingImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1049 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1050
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1051 LocalResourceManager localManager = new LocalResourceManager(
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1052 parentResourceManager);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1053
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1054 // performance: more efficient in DWT to set disabled and hot
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1055 // image before regular image
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1056 (cast(ToolItem) widget)
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1057 .setDisabledImage(disabledImage is null ? null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1058 : localManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1059 .createImageWithDefault(disabledImage));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1060 (cast(ToolItem) widget).setImage(image is null ? null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1061 : localManager.createImageWithDefault(image));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1062
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1063 disposeOldImages();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1064 imageManager = localManager;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1065
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1066 return image !is null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1067 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1068 ImageDescriptor image = action.getImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1069 ImageDescriptor hoverImage = action.getHoverImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1070 ImageDescriptor disabledImage = action.getDisabledImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1071
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1072 // If there is no regular image, but there is a hover image,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1073 // convert the hover image to gray and use it as the regular image.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1074 if (image is null && hoverImage !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1075 image = ImageDescriptor.createWithFlags(action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1076 .getHoverImageDescriptor(), DWT.IMAGE_GRAY);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1077 } else {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1078 // If there is no hover image, use the regular image as the
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1079 // hover image,
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1080 // and convert the regular image to gray
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1081 if (hoverImage is null && image !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1082 hoverImage = image;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1083 image = ImageDescriptor.createWithFlags(action
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1084 .getImageDescriptor(), DWT.IMAGE_GRAY);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1085 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1086 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1087
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1088 // Make sure there is a valid image.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1089 if (hoverImage is null && image is null && forceImage) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1090 image = ImageDescriptor.getMissingImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1091 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1092
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1093 // Create a local resource manager to remember the images we've
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1094 // allocated for this tool item
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1095 LocalResourceManager localManager = new LocalResourceManager(
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1096 parentResourceManager);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1097
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1098 // performance: more efficient in DWT to set disabled and hot image
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1099 // before regular image
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1100 (cast(ToolItem) widget).setDisabledImage(disabledImage is null ? null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1101 : localManager.createImageWithDefault(disabledImage));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1102 (cast(ToolItem) widget).setHotImage(hoverImage is null ? null
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1103 : localManager.createImageWithDefault(hoverImage));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1104 (cast(ToolItem) widget).setImage(image is null ? null : localManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1105 .createImageWithDefault(image));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1106
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1107 // Now that we're no longer referencing the old images, clear them
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1108 // out.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1109 disposeOldImages();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1110 imageManager = localManager;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1111
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1112 return image !is null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1113 } else if (cast(Item)widget \|\| cast(Button)widget ) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1114
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1115 // Use hover image if there is one, otherwise use regular image.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1116 ImageDescriptor image = action.getHoverImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1117 if (image is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1118 image = action.getImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1119 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1120 // Make sure there is a valid image.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1121 if (image is null && forceImage) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1122 image = ImageDescriptor.getMissingImageDescriptor();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1123 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1124
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1125 // Create a local resource manager to remember the images we've
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1126 // allocated for this widget
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1127 LocalResourceManager localManager = new LocalResourceManager(
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1128 parentResourceManager);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1129
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1130 if (cast(Item)widget) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1131 (cast(Item) widget).setImage(image is null ? null : localManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1132 .createImageWithDefault(image));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1133 } else if (cast(Button)widget) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1134 (cast(Button) widget).setImage(image is null ? null : localManager
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1135 .createImageWithDefault(image));
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1136 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1137
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1138 // Now that we're no longer referencing the old images, clear them
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1139 // out.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1140 disposeOldImages();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1141 imageManager = localManager;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1142
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1143 return image !is null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1144 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1145 return false;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1146 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1147
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1148 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1149 * Dispose any images allocated for this contribution item
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1150 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1151 private void disposeOldImages() {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1152 if (imageManager !is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1153 imageManager.dispose();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1154 imageManager = null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1155 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1156 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1157
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1158 /**
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1159 * Shorten the given text <code>t</code> so that its length doesn't exceed
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1160 * the width of the given ToolItem.The default implementation replaces
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1161 * characters in the center of the original string with an ellipsis ("...").
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1162 * Override if you need a different strategy.
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1163 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1164 * @param textValue
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1165 * the text to shorten
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1166 * @param item
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1167 * the tool item the text belongs to
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1168 * @return the shortened string
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1169 *
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1170 */
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1171 protected String shortenText(String textValue, ToolItem item) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1172 if (textValue is null) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1173 return null;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1174 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1175
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1176 GC gc = new GC(item.getParent());
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1177
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1178 int maxWidth = item.getImage().getBounds().width * 4;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1179
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1180 if (gc.textExtent(textValue).x < maxWidth) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1181 gc.dispose();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1182 return textValue;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1183 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1184
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1185 for (int i = textValue.length; i > 0; i--) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1186 String test = textValue.substring(0, i);
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1187 test = test ~ ellipsis;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1188 if (gc.textExtent(test).x < maxWidth) {
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1189 gc.dispose();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1190 return test;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1191 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1192
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1193 }
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1194 gc.dispose();
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1195 // If for some reason we fall through abort
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1196 return textValue;
e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1197 }
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1198
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1199 /*
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1200 * (non-Javadoc)
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1201 *
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1202 * @see dwtx.jface.action.ContributionItem#dispose()
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1203 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1204 public void dispose() {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1205 if (widget !is null) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1206 widget.dispose();
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1207 widget = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1208 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1209 holdMenu = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1210 }
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1211
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1212 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1213 * Handle show and hide on the proxy menu for IAction.AS_DROP_DOWN_MENU
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1214 * actions.
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1215 *
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1216 * @return the appropriate listener
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1217 * @since 3.4
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1218 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1219 private Listener getMenuCreatorListener() {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1220 if (menuCreatorListener is null) {
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1221 menuCreatorListener = new class Listener {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1222 public void handleEvent(Event event) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1223 switch (event.type) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1224 case DWT.Show:
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1225 handleShowProxy(cast(Menu) event.widget);
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1226 break;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1227 case DWT.Hide:
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1228 handleHideProxy(cast(Menu) event.widget);
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1229 break;
192 c3583c6ec027 Added missing default cases for switch statements Frank Benoit <benoit@tionex.de> parents: 167 diff changeset	1230 default:
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1231 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1232 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1233 };
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1234 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1235 return menuCreatorListener;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1236 }
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1237
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1238 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1239 * This is the easiest way to hold the menu until we can swap it in to the
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1240 * proxy.
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1241 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1242 private Menu holdMenu = null;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1243
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1244 private bool menuCreatorCalled = false;
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1245
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1246 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1247 * The proxy menu is being shown, we better get the real menu.
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1248 *
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1249 * @param proxy
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1250 * the proxy menu
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1251 * @since 3.4
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1252 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1253 private void handleShowProxy(Menu proxy) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1254 proxy.removeListener(DWT.Show, getMenuCreatorListener());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1255 IMenuCreator mc = action.getMenuCreator();
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1256 menuCreatorCalled = true;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1257 if (mc is null) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1258 return;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1259 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1260 holdMenu = mc.getMenu(proxy.getParentMenu());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1261 if (holdMenu is null) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1262 return;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1263 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1264 copyMenu(holdMenu, proxy);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1265 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1266
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1267 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1268 * Create MenuItems in the proxy menu that can execute the real menu items
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1269 * if selected. Create proxy menus for any real item submenus.
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1270 *
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1271 * @param realMenu
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1272 * the real menu to copy from
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1273 * @param proxy
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1274 * the proxy menu to populate
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1275 * @since 3.4
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1276 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1277 private void copyMenu(Menu realMenu, Menu proxy) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1278 if (realMenu.isDisposed() \|\| proxy.isDisposed()) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1279 return;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1280 }
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1281
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1282 // we notify the real menu so it can populate itself if it was
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1283 // listening for DWT.Show
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1284 realMenu.notifyListeners(DWT.Show, null);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1285
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1286 final Listener passThrough = new class Listener {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1287 public void handleEvent(Event event) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1288 if (!event.widget.isDisposed()) {
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1289 Widget realItem = cast(Widget) event.widget.getData();
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1290 if (!realItem.isDisposed()) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1291 int style = event.widget.getStyle();
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1292 if (event.type is DWT.Selection
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1293 && ((style & (DWT.TOGGLE \| DWT.CHECK)) !is 0)
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1294 && (null !is cast(MenuItem)realItem )) {
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1295 (cast(MenuItem) realItem)
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1296 .setSelection((cast(MenuItem) event.widget)
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1297 .getSelection());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1298 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1299 event.widget = realItem;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1300 realItem.notifyListeners(event.type, event);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1301 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1302 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1303 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1304 };
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1305
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1306 MenuItem[] items = realMenu.getItems();
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1307 for (int i = 0; i < items.length; i++) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1308 final MenuItem realItem = items[i];
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1309 final MenuItem proxyItem = new MenuItem(proxy, realItem.getStyle());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1310 proxyItem.setData(realItem);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1311 proxyItem.setAccelerator(realItem.getAccelerator());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1312 proxyItem.setEnabled(realItem.getEnabled());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1313 proxyItem.setImage(realItem.getImage());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1314 proxyItem.setSelection(realItem.getSelection());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1315 proxyItem.setText(realItem.getText());
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1316
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1317 // pass through any events
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1318 proxyItem.addListener(DWT.Selection, passThrough);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1319 proxyItem.addListener(DWT.Arm, passThrough);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1320 proxyItem.addListener(DWT.Help, passThrough);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1321
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1322 final Menu itemMenu = realItem.getMenu();
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1323 if (itemMenu !is null) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1324 // create a proxy for any sub menu items
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1325 final Menu subMenu = new Menu(proxy);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1326 subMenu.setData(itemMenu);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1327 proxyItem.setMenu(subMenu);
72 5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1328 subMenu.addListener(DWT.Show, new class(subMenu, itemMenu) Listener {
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1329 Menu subMenu_;
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1330 Menu itemMenu_;
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1331 this(Menu a,Menu b){
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1332 subMenu_=a;
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1333 itemMenu_=b;
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1334 }
72 5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1335 void handleEvent(Event event){
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1336 event.widget.removeListener(DWT.Show, this);
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1337 if (event.type is DWT.Show) {
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1338 copyMenu(itemMenu_, subMenu_);
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1339 }
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1340 }
5df4896124c7 JFace and its examples do compile Frank Benoit <benoit@tionex.de> parents: 71 diff changeset	1341 });
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1342 subMenu.addListener(DWT.Help, passThrough);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1343 subMenu.addListener(DWT.Hide, passThrough);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1344 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1345 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1346 }
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1347
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1348 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1349 * The proxy menu is being hidden, so we need to make it go away.
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1350 *
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1351 * @param proxy
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1352 * the proxy menu
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1353 * @since 3.4
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1354 */
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1355 private void handleHideProxy(Menu proxy) {
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1356 proxy.removeListener(DWT.Hide, getMenuCreatorListener());
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1357 proxy.getDisplay().asyncExec(dgRunnable( (Menu proxy_) {
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1358 if (!proxy_.isDisposed()) {
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1359 MenuItem parentItem = proxy_.getParentItem();
4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1360 proxy_.dispose();
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1361 parentItem.setMenu(holdMenu);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1362 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1363 if (holdMenu !is null && !holdMenu.isDisposed()) {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1364 holdMenu.notifyListeners(DWT.Hide, null);
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1365 }
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1366 holdMenu = null;
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1367 }, proxy ));
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1368 }
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1369
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1370 /**
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1371 * Return the widget associated with this contribution item. It should not
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1372 * be cached, as it can be disposed and re-created by its containing
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1373 * ContributionManager, which controls all of the widgets lifecycle methods.
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1374 * <p>
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1375 * This can be used to set layout data on the widget if appropriate. The
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1376 * actual type of the widget can be any valid control for this
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1377 * ContributionItem's current ContributionManager.
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1378 * </p>
71 4878bef4a38e Some fixing Frank Benoit <benoit@tionex.de> parents: 70 diff changeset	1379 *
70 46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1380 * @return the widget, or <code>null</code> depending on the lifecycle.
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1381 * @since 3.4
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1382 */
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1383 public Widget getWidget() {
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1384 return widget;
46a6e0e6ccd4 Merge with d-fied sources of 3.4M7 Frank Benoit <benoit@tionex.de> parents: 43 diff changeset	1385 }
16 e0f0aaf75edd PopupDialog, bindings and actions Frank Benoit <benoit@tionex.de> parents: diff changeset	1386 }

Mercurial > projects > dwt-addons

annotate dwtx/jface/action/ActionContributionItem.d @ 192:c3583c6ec027