Scott Lawrence wrote:
> On Tue, 2008-07-01 at 10:08 -0400, Martin Steinmann wrote:
>> It would be good to add a help text that explains how the process works.
>> Also, the country field needs to be explained and the UI needs to test
>> that the admin enters a valid country code (Upper case and 2 letter
>> valid country code).
> 
> I wouldn't worry about whether or not it's 'valid'; the CA will take
> care of that.  It does have to be exactly two upper case letters or the
> CSR generation will fail.
> 

Especially if one generates certificates for Dune inhabitants ;-)

But no harm in having a simple verifier here. Tapestry has masked edit 
fields BTW, that we are yet to use...

http://tapestry.apache.org/tapestry4.1/tapestry-contrib/componentreference/maskedit.html


The menu entry should not have the word "Manage" (unless we want to add it 
to each and every menu entry - j/k).
The button should be called just "Import" and "Generate" not "Import CRT". 
"Generate CSR"

You probably want to shoot for a shorter tab name too 'Generate CSR and 
private key' is probably too long: again Import and Generate might be OK.

Quick help should explain what one is generating, but - then again - if 
someone actually has a need to generate CSR, they probably know what they 
are doing already. And if they don't, no amount of help text will explain it...

Nice UI.
D.

_______________________________________________
sipx-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipx-dev
Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev

Reply via email to