John,

One very simple change could help in the dialog:
" Please let the developers know about this problem by
  filing a bug together with the error details listed below at:

  http://defect.opensolaris.org";

   ^^^^^^^^^^^^^^^^^^
    This could be URI button


best
Michal

On 01/14/10 12:02 PM, Michal Pryc wrote:
LGTM

This also partialy fixes bug 13850. While switching to the publisher with exprider certificate, the user now doesn't end up having unusable PM. The error dialog is shown and PM closes when clicking on the Close button.


Michal


On 01/14/10 10:57 AM, Padraig O'Briain wrote:
This looks good to me.

Padraig

On 01/13/10 15:28, John Rice wrote:
Sorry posted work in progress webrev, correct pylinted one is at:
http://cr.opensolaris.org/~jmr/pm_13852_global_error_handler_13Jan_325pm/

JR

John Rice wrote:
Hi - this webrev adds a global error handler to PM to catch any unhandled exceptions and display them to the user.

http://cr.opensolaris.org/~jmr/pm_13852_global_error_handler_13Jan_320pm/ Code review: 13852 PM needs a global error handler for uncaught exceptions

I also intend to log caught ApiExceptions and make the Logs available from a View->Log menu, but that's for another webrev.

JR
_______________________________________________
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
_______________________________________________
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

Reply via email to