Re: [platform-ui-dev] MenuManager modification to get rid of IMenuCreato

[Randy Hudson <hudsonr@xxxxxxxxxx>]

I just encountered problems with this
interface last week, and a recent clarification was made by Lynne Kues
on the newsgroup.

I don't think it would be acceptable
for the MenuManager's Action to be ignored in the submenu case, which is
the intended usage.  This is why I fave suggested ToolBarDropDownMenuManager,
in bug:
http://bugs.eclipse.org/bugs/show_bug.cgi?id=25910

Please "vote" for this feature
by adding any comments to the *bugzilla* and CCing yourself.
-Randy

"Joseph Khalil" <joseph_khalil@xxxxxxxxxxx> Sent by: platform-ui-dev-admin@xxxxxxxxxxx 11/11/2002 05:09 AM Please respond to platform-ui-dev

To:        <platform-ui-dev@xxxxxxxxxxx>         cc:                 Subject:        [platform-ui-dev] MenuManager modification to get rid of IMenuCreator

I have an opinion in IMenuCreator...
i really hate it coz the problem is solved in MenuManager by adding a
MenuManager to another MenuManager so we can get the DROP_DOWN menu..
we can make a similar solution for a toolbar by making some modification
in
MenuManager

Since a MenuManager is a IContributionItem
we can make the MenuManager takes an IAction in its constructor.
so the Text and id is taken directly from the IAction instead of the text
and id taken in its
constructor now.. also the image can be taken from the IAction so the sub
menu
can have image not text only as now..
note that the run() method of IAction is ignored in the sub menu case..
now what about toolBar case..
when you want to add a toolItem with DROP_DOWN menu , you add a MenuManager
with it's IAction , so the image is taken from the IAction, when you click
on
it the run() method runs , when you click the arrow the MenuManager shows
its
menu (that contains the MenuManager  child IContribution Items )

I guess this will be enough and consistent...
_______________________________________________
platform-ui-dev mailing list
platform-ui-dev@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/platform-ui-dev