| Summary: | [ActionSets] Action set ordering is not stable | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Randy Giffen <Randy_Giffen> |
| Component: | UI | Assignee: | Simon Arsenault <simon_arsenault> |
| Status: | VERIFIED FIXED | QA Contact: | |
| Severity: | major | ||
| Priority: | P2 | CC: | n.a.edgar |
| Version: | 2.0 | ||
| Target Milestone: | 2.1 M4 | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Whiteboard: | |||
|
Description
Randy Giffen
We try to process them in order of id when initially added, but this does not get maintained for incremental updates. Changed PluginActionSetBuilder (and a little PluginActionBuilder) to do the following: 1) Menu group markers/separators are sorted based on the action set id to keep these group in a consistent order. We use to just add to the end of the menu. 2) When scanning thru the actions of a menu group, the search is terminated if another group is found. Therefore the action will be added to the end of the group. It was possible beforehand to scan past the end of the group and end up adding an action into a different group. Plugin developers can now depend on the action set id to properly order their actions. We know this is not ideal but is sufficient for the moment until we can properly address bug 16282 Verified in I20021216. |