Quick follow-up: If I click on the error icon, the log window appears and the icon goes away; if I access the log through the View menu instead, the log window appears and the icon stays put.
Would it be worth adding a 'clear' (or whatever) button alongside the 'close' button? --joanie On Thu, 2010-01-14 at 09:19 -0500, Joanmarie Diggs wrote: > 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
