[STUDENTS] More visual effects updates
Phil Bull
philbull at gmail.com
Fri Sep 12 23:10:54 UTC 2008
Hi Stephanie,
On Thu, 2008-09-11 at 13:38 -0700, Stephanie Mardell wrote:
> Here is a minor update as requested. Let me know what you think. Also
> as this is my first patch please let me know if there is anything else
> that I need to do.
The patch looks really good, thanks for sending it so quickly! I came up
with a couple of small issues:
* Two instances of button text are currently in apostrophes, but
should use the <guibutton> tag, e.g. 'Keep Settings' =>
<guibutton>Keep Settings</guibutton>
* I'm not sure that the "40 seconds" information needs to be in a
note. I try to limit the number of note tags I use because they
tend to break the flow of the page and distract from the rest of
the information. What are your feelings on this?
* Is the timeout definitely still 40 seconds? It has a habit of
changing every once in a while, and I'm can't check because
compiz is broken on my computer...
Thanks,
Phil
--
Phil Bull
https://launchpad.net/people/philbull
More information about the ubuntu-doc
mailing list