Community
Participate
Working Groups
When the user clicks on the "Browse..." button next to the Superclass text field in the New Java Class wizard they have to watch the progress bar go across the bottom of the wizard. I assume that it is building a list of classes that are visible from the current SourceFolder. It would be really nice if that list was built up in the background while the user is typing the name of the new class. If the user never clicks the browse button then the list can be thrown away, and if the user clicks the browse button before the list has completed then the progress bar can pickup where it currently is. I would imagine that most of the time, the list can be loaded before the user clicks on the browse button.
should be revisited during the performance work
Created attachment 2751 [details] new type wizard - Browse supertype button here's the trace
asking Martin for comment and further action btw, there's no busy cursor either
This dialog is using the same dialog that the all types dialog uses. There it's the same: classes are searched first, and then the dialog is opened. Busy cursor is there. Moving back to jdt.ui as this is a TypeSelectionDialog issue
there's no busy cursor - try it on Action (select Action and try creating a subclass of it)
I20030128 displays a busy cursor in this case. Regarding performance: the performace is comparable to the Open Type performance. This means that the first time the browse button is pressed, the type cache may be populated. If so, you see appropriate progress feedback in the progress monitor. Most time is spent in JavaSearchScope. Moving to JDT Core for comments.
Matt's original comment is still valid. Asking for some background computation of the list would indeed improve this situation. Computing the list of all types in the workbench has a cost, I doubt we could reduce it that much.
As I said the list is precomputed in 90% of the cases since it is shared with the open type dialog. It must only be recomputed if new types where added or existing types deleted. There are no plans to improve this for 2.1 by doing a computation of the list in the background.
As of now 'LATER' and 'REMIND' resolutions are no longer supported. Please reopen this bug if it is still valid for you.
