Comment 7 for bug 150641

Revision history for this message
Phil Bull (philbull) wrote :

The patch looks good, and seems comprehensive. It also validates.

I found a few minor issues, which are nitpicks more than anything:

 * "Open the <application>Visual Effects Preferences</application>"
The application isn't specifically called "Visual Effects Preferences" (the window is called "Appearance Preferences"), so it would probably be better to refer to it as "visual effects preferences", without the capitals and <application> tag.

* "Enable extra Visual Effects"
"Visual Effects" should probably be lower case here. This was also an issue with the unpatched docs (i.e. "Desktop Effects" should have been "desktop effects" in a few cases)

 * "not be able to use Visual effects."
"Visual effects" -> "visual effects"

Thanks for working on this!