Hey John.

This is working nicely for me now. Thanks!

Take care.
--joanie

On Thu, 2010-01-14 at 12:47 +0000, John Rice wrote:
> Joanie - thanks for the feedback. I have fixed the issues below and this 
> now supports logging from PM, PM-webinstall and UM. I have reviewed the 
> code with Padraig offline.
> 
> I have put in #DEBUG for PM, WI and UM for ease of testing. I'll take 
> them out for final submission to the gate.
> 
> http://cr.opensolaris.org/~jmr/pm_13333_log_view_14Jan_1230pm/
> 13333 PM Provide user notification when errors written to log in GUI.
> 
> JR
> 
> 
> > Having uncommented those lines:
> >
> > 1. Nit: It seems that the accessible name for that icon is always
> > 'Remote search error' regardless if it is a search error or the result
> > of the log error. While you could change the AtkObject's name at the
> > same time you change the tooltip text, I'd imagine it would be a lot
> > simpler to just change the AtkObject's name to something more generic
> > like "Error". Would you mind doing so?
> >   
> Done
> > 2. Any chance you're making GUI changes outside of the main thread? If I
> > do the following:
> >
> > - Uncomment the lines above
> > - Enable AT support (logging out if it wasn't already enabled)
> > - Launch Accerciser
> > - Launch PM
> >
> >   
> I was making some gui calls mea culpa :) Tested with Orca and working 
> fine for me now.
> > I can regularly (though not 100% of the time) hang my desktop. Without
> > your patch I'm not seeing this issue. Sorry....
> >
> > Take care.
> > --joanie
> >
> >   
> 


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

Reply via email to