Community
Participate
Working Groups
Wizards, especially the JavaLiteProjectWizard, open with too much blank space. A cursory review shows that the implementation just hides all the various groups that make up the original wizard. I assume that these hidden groups continue to occupy space. This may be an opportunity to just build our own simplified wizards since we really only need to capture a name.
We have an opportunity here to "do things right". I think that our best track here is to build separate UI and model components (the class that JavaLiteProjectWizard wizard is based on merges UI and model into one). That is, we should create a separate and completely non-UI "wizard" that--given sufficient configuration--can do the project creation for us. Our "UI" classes can then simply do what they do best: render a UI that drives the wizard. Using the word "wizard" in this context will likely be confusing, so we'll need to find a different word. Our first implementation should provide only what we need. Ultimately, though, it's probably something that we can evolve to the point that it's a worthy contribution to the JDT project.
I'd like to see this done soon. I have exchanged emails with several people about this bug, but do not see any of their names on the CC list. If somebody wants to do this bug, please indicate your interest here. I intend to look at this in the second week of July if nobody else has stepped up.
As a first cut at resolving this bug, I am removing the unused sections of code.
I believe that I've been able to eliminate the the NewJavaLiteProjectWizardPageTwo class and much of the NewJavaLiteProjectWizardPageOne class. This significantly reduces the amount of duplicate code and use of non-API. Will attach as patch for verification by others.
Created attachment 142147 [details] Patch that eliminates quite a lot of duplicate code This patch removes NewJavaLiteProjectWizardPageTwo and significantly reduces the amount of code duplication in NewJavaLiteProjectWizardPageOne. Please verify that it functions as required.
Created attachment 142148 [details] mylyn/context/zip
Marking bug as helpwanted; need somebody to verify that the patch is valid. Improvements welcome.
Patch committed in HEAD.
