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
