[STUDENTS] More visual effects updates

[Stephanie Mardell]

Hi Phil,

Updated patch as requested.

Stephanie

On Sat, 2008-09-13 at 00:10 +0100, Phil Bull wrote:
> 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
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diffname.txt
Type: text/x-patch
Size: 1588 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-doc/attachments/20080913/bb93a7c2/attachment.bin>