Shawn Walker wrote:
Padraig O'Briain wrote:


On 12/07/09 18:36, Shawn Walker wrote:
Padraig O'Briain wrote:


On 12/05/09 21:42, Shawn Walker wrote:
Padraig O'Briain wrote:
Here is an updated webrev http://cr.opensolaris.org/~padraig/ips-12962-v1/

It does not make any changes to the logic but does make some layout changes:

1) Change heading in Manage Publishers dialog
2) Add icons to Up and Down buttons
3) Remove origins expander in Modify Publisher dialog
4) Some minor layout changes in Manage Publishers and Modify Publisher dialog.

This is beautiful. This is the absolute best the manage publishers stuff has ever looked.

However, I did notice that it allowed me to add SSL information to one of my publishers despite one of my origins being http.


Do you mean that the GUI seems to allow you to add SSL information or the GUI actually does add SSL information when one of your origins is http?

It allowed me to enter SSL information when one of my origins was http. I didn't check to see if it actually saved it when I hit OK, or if it errored once I did so.

It does not save the data.
I will log a bug that the GUI should prevent the data being entered if it does not intend using it.

Padraig
Cheers,

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

Reply via email to