Padraig,
The webrev itself looks ok. I have prepared new webrev:
http://cr.opensolaris.org/~migi/ips-11243-v2/
Because you didn't apply cleanly. Just merging problem, no func changes.
Few comments below:
-> The preferences Help goes to the wrong help section.
Images:
Figure 1-1 Package Manager
Figure 1-2 Package Manager Start Page
Figure 1-3 Package Manager Main View
-> Those figures are fine.
1.7.2. How to Use WebInstall to Add Publishers and Install Packages
"You will see a window similar to the example window shown below. The
window shows a list of publishers that will be added and the packages
that will be installed. Review the information."
-> This figure is ok, but it doesn't have the border around with "Figure
1-4 " in it.
Figure 1-4 Update Manager Main Window
-> This is an old Update Manager screenshot with wrong UM icon. If we
will introduce the border in the 1.7.2, then the figure number should
change to "Figure 1-5".
best
Michal Pryc
On 12/15/09 09:40 AM, Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-11243-v1/,
11243 PM Help needs update with new Help tags
This webrev integrates the new Package Manager help text and ensures
that the help is invoked with the correct help tags.
Padraig
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
