Code review comment for lp:~jtatum/mago/gconf

Revision history for this message
Nagappan Alagappan (nagappan) wrote :

> Adding a gconf wrapper for mago. I needed to use gconf while writing some test
> cases. I believe there are uses for gconf beyond test cases, so I placed the
> wrapper on the same level as other global utility classes: in the mago
> directory itself. I also added a check to determine if accessibility is
> enabled. This check uses one of the gconf static convenience methods.

Nice work :)

The import text size is increasing, can we just import utils and use utils.function ?

« Back to merge proposal