[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