Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 363742 - Don't show onWidgetLoad in the Properties view
Summary: Don't show onWidgetLoad in the Properties view
Status: CLOSED WONTFIX
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: lu lu CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-14 15:09 EST by Will Smythe CLA
Modified: 2017-02-23 14:18 EST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.