Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 363742

Summary: Don't show onWidgetLoad in the Properties view
Product: z_Archived Reporter: Will Smythe <smythew>
Component: EDTAssignee: lu lu <lulu>
Status: CLOSED WONTFIX QA Contact:
Severity: enhancement    
Priority: P3 CC: hjiyong, svihovec
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Will Smythe CLA 2011-11-14 15:09:05 EST
onWidgetLoad seems like a pretty low-level event -- if so, it shouldn't be in the Properties view. At a minimum, it should not be the first event type in the Events tab table.
Comment 1 lu lu CLA 2012-01-10 01:28:31 EST
We think that 'onWidgetLoad' shouldn't be removed out of the event list because it maybe used sometimes, and if change the location of 'onWidgetLoad' in event list, we need to redefine IR, so we think it should be better to keep current implementation. Thanks.
Comment 2 Will Smythe CLA 2012-01-10 01:41:22 EST
Can we not sort the list of events in the Properties view? I don't think onWidgetLoad should be listed first .. and sorting would solve this problem.
Comment 3 Huang Ji Yong CLA 2012-01-10 02:26:41 EST
(In reply to comment #2)
> Can we not sort the list of events in the Properties view? I don't think
> onWidgetLoad should be listed first .. and sorting would solve this problem.

Hi Will,
The order of event list depends on their definition orders. We don't have any facility to sort the event right now.
onWidgetLoad is defined in DojoBase while the other events are defined in the parent Widget "class". So the onWidgetLoad event is listed as the 1st one.
I think it is hard to change the process order so that there is no easy way to move the event down. If we want to change the order, we have to do the change in IR or VE, add the sorting mechanism. I think it is a lot of effort and may not have many benefits.
At the same time, I think the event is useful for developers so that we should not remove it.
Comment 4 Brian Svihovec CLA 2012-01-10 10:03:23 EST
I have opened Bug 368258 to cover sorting the events in the future.  We should close this as 'won't fix'.
Comment 5 lu lu CLA 2012-01-10 21:30:56 EST
(In reply to comment #4)
> I have opened Bug 368258 to cover sorting the events in the future.  We should
> close this as 'won't fix'.

Thanks, Brian. Close this enhancement as won't fix.