Code review comment for lp:~sergiomeneses/ubuntu-manual-tests/gnome-terminal

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

> I'd change the setup instructions to make it independent of Unity:
>
> -<dt> Open Terminal using the dash, and typing 'terminal'</dt>
> +<dt> Open the Terminal application</dt>
>

I agree

> I'm not sure why you modified:
>
> 8 - <dt> Modify the colours to a different set</dt>
> 9 + <dt> Modify the "Colors" to a different set</dt>
>
> It seems more natural to me, dt> Modify the colours to a different set</dt>,
> same applies for:

Perhaps you can say something like:
<dt>Click on Edit, select Profiles, select Edit and select "Colors"</dt>
<dt>Using the options contained in the "Colors" tab modify the color palette, choosing some non-default colors or a different color scheme</dt>
<dd>The colors in your terminal match your modifications</dd>

What do you think?

>
> 11 - <dt> Set a Background Image</dt>
> 12 + <dt> Set a "Background" Image</dt>
>
> I agree with writing commands between ""

me too

>
> 22 + <dt> Type "touch text1.txt"</dt>
> 23 + <dt> type "ls"</dt>
>
> + <dd>txt should now green?</dd> should be -> <dd>text1.txt is enlighten in
> green?</dd> ?
>
> Greetings

Also, consider giving some more detail for these steps
<dt> Set a Background Image</dt>
<dt> Change the Zoom Level</dt>

review: Needs Fixing

« Back to merge proposal