[Uml-devel] comments on CodeGen Wizard

[Luis De la Parra Blum]

> I'm not sure that there's a logical separation in what is in the dialogue.
> Everything seems to belong on the one page.
>
yeap... everything seems to belong together... but still....
once I read somewhere (I think it was some documentation for QWizard???) that 
its better to have more pages with less options in each one that a page with 
too many options in it. even if it belongs together... in that case maybe 
label the pages "code options (1)" and "code options (2)"
if you get too many options in one page is easy to loose focus and not see all 
options available.

>
> Well I really didn't understand the old labels and I don't think the user
> should have to work out that there's a `what's this' or even a tool
> tip.

it was pretty logical to me! =), but ok, of course it always seems logical to 
the person who writes something, if you think its confusing then ok (I also 
think the user should not have to find out through a tool tip or what's this 
something that should be obvious)

> I'm not sure that the active language makes much sence, it's just a
> configuration option so it should be with the other configuration options
> (on codeGenerationOptionsPage).  However the combo box is a good idea,
> I'll do that.

to me it makes sense. Ok, for now the only thing affected by the "active 
language" is the code generation, but later we could have the option "active 
language:"
- Pure UML
- C++
- C++ with QT extensions
- Java
- (etc)

and based on this active language generate code, use profiles, etc...
but that is going to be later.

so.., if you want to have the option there, then just make it a combo box. 
After that we can see how it looks and deide if we leave like that or if we 
need to split the page into two.
>
> Thanks for keeping an eye on me Luis.

no problem =))

luis
ps: if you decide to leave the combo box there, then please just make sure you 
ask what umbrello what the current language is and set this lanugage as the 
default in the combo box (so that if the user already had choosen PHP, he 
doesnt have to change this everytime form C++ to PHP)