Matt,

Keep up the good work.  A couple comments below.

INSTALL_DISK_PANEL.txt:

        Did you intend to add a bunch of <CR>s at the end of the file?

INSTALL_LANGUAGE_PANEL.txt:

        "...by using the Options button in the dialog box."

        I think that you need to be more specific here.  It almost
        sounds like you can go back to the installer and change the
        locale there for the running system.  So perhaps saying
        something about login or GDM or ...

INSTALL_TIMEZONE_PANEL.txt:

        "the dropdown selections automatically populate with the time zone..."

        You might change "populate" to "update".

WELCOME_PANEL.txt:
        'To view the online help for this installer, select the "?" button on 
an installer panel.'

        It seems kind of silly to explain how to get online help from
        within the online help.  Or is this the text that is show on
        the Welcome panel itself?

        Appears to also have an extra <CR> at the end of the file.

Thanks,

John

Matt Keenan wrote:
> Sort of Code review for Bug :
>   13559 -  Integrate New Help Files for 2010.03 Release
>   http://defect.opensolaris.org/bz/show_bug.cgi?id=13559
> 
> Webrev :
>   http://cr.opensolaris.org/~mattman/bug_13559/
> 
> 
> Updated help docs for GUI installer containing edits pertinent to 
> 2010.03 release. No actual code just txt file updates.
> 
> cheers
> 
> Matt
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to