Evan Layton wrote:

>
> I think these look good.
>
> However the wireframe shown at
> http://xdesign.sfbay.sun.com/projects/solaris/subprojects/package_mngt/UI_specs/ui_spec_phase2/html-mockup/index.htm
>
> differs slightly and I have some nits for the wireframe.
>
> On item #1 of "Manage Your Boot Environments" it states "User can
> set any other BE as the default" but doesn't indicate that this is
> really the active on reboot and that for this change to take place
> a reboot is required.
>
> On item #1 of the confirmation dialog the note stating that changes
> will happen on reboot makes it sounds like the BE's being deleted
> won't be deleted until the system is rebooted. It needs to be clear
> that the reboot only appiles to seting the active (or default) BE
> and that the BE deletes will happen immediately.

Those things should be changes as tech writer will review and propose 
changes. I am adding Jenya, since she might know who is reviewing the text


>> The whole webrev is at:
>>
>> http://cr.opensolaris.org/~migi/18_09_be_management_3201_misc_v1/
>
> Looking at your newer version (v3) status_checkmark.png appears corrupted.

This is an image so the diff can't show it. Please open in firefox or so:
http://cr.opensolaris.org/~migi/22_09_be_management_3201_misc_v3/raw_files/new/src/gui/data/icons/status_checkmark.png



> I'm still somewhat inexperienced with python but as far as I can tell the
> rest looks OK.

I've already submited those changes to the gate this morning, so other 
changes can be submitted as an bug to the existing one.

best
Michal
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to