Bug 13083 - [Contributions] updating: ISV Toolbar manager not marked as dirty when items visibility changes
Summary: [Contributions] updating: ISV Toolbar manager not marked as dirty when items ...
Status: ASSIGNED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-04-09 12:30 EDT by Darin Swanson CLA
Modified: 2019-09-06 16:07 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Swanson CLA 2002-04-09 12:30:12 EDT
20020404

DebugActionGroups allow plugins to specify the visiblility of actions in the 
toolbar and popup menus of "debug" views.  

In DebugActionGroupsManager#updateDebugActionGroups, we change the visiblity of 
contributed items in the toolbar retrieved via: 
  IActionBars actionBars = viewPart.getViewSite().getActionBars();
  //the isvToolBarMgr
  IToolBarManager toolBarManager = actionBars.getToolBarManager();

It appears that the wrong toolBarManager is marked as dirty.
In ViewPane#updateActionBars(), the viewToolBarManager is marked as dirty
but the isvToolBarMgr is not.  Therefore the visibility changes are not shown 
in the toolbar.

To fix the problem, we must now explicitly mark the isvToolBarManager as dirty.
Previously this was not necessary (I believe in builds < 20020402).
Comment 1 Nick Edgar CLA 2002-04-09 16:33:50 EDT
Changing visibility on an item does not mark the manager as dirty.
Is it possible you were adding/removing items previously?
Comment 2 Darin Swanson CLA 2002-04-09 16:41:34 EDT
AFAIK, we have only ever set the visibility.
No problem...I will release fix in our code.
Comment 3 Nick Edgar CLA 2002-04-09 17:09:36 EDT
There have been some interesting changes in this area.
If you have the time to step through it, it would be helpful.
But ContributionItem.setVisible just sets the field, and this hasn't changed 
recently.
Comment 4 Simon Arsenault CLA 2002-09-04 12:04:55 EDT
Have you been able to step thru the code to determine what is going on? If so, 
can this be closed now or have you identified a problem area?
Comment 5 Darin Swanson CLA 2002-09-05 12:13:34 EDT
Simon, I will have to take a look at this next week. I am away from a computer 
for the next 4 days and I don't remember the specifics from April ;-)
Comment 6 Darin Swanson CLA 2002-09-12 13:41:43 EDT
I would propose this be changed to a feature request...changing the visibility 
of an contributed item would flag the toolbar manager as dirty.
Comment 7 Michael Van Meekeren CLA 2006-04-21 13:56:41 EDT
Moving Dougs bugs
Comment 8 Paul Webster CLA 2007-04-05 19:01:31 EDT
Assigning to component owner
PW
Comment 9 Eclipse Webmaster CLA 2019-09-06 16:07:16 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.