Code review comment for lp:~shrini/ubuntu-manual-tests/shrini-dia

Revision history for this message
shrini (shrini) wrote :

Thanks Elfy.

Fixed it and added into a new branch.
https://code.launchpad.net/~shrini/ubuntu-manual-tests/shrini-dia-modified

2014-07-24 21:36 GMT+05:30 Elfy <email address hidden>:
> Review: Needs Fixing
>
> Thanks for that - little bit needs to be done, otherwise it's good.
>
> If you can fix these I'll be able to approve and get it merged.
>
> We don't use this type of heading any longer - so if you can remove both please.
>
> Test-case name: Dia/dia-001
> Test-case name: Dia/dia-002
>
> <dt>Go to File > Open..., select an image from your Home directory and press Open</dt>
>
> Might be good to change that to not include Home.
>
> <dt>Go to File > Open..., select an image and press Open</dt>
>
> <dd>Dia creates a new image in the canvas</dd>
>
> No it doesn't - it creates and empty .dia file to work in, so reword that too please.
>
> Thanks :)
> --
> https://code.launchpad.net/~shrini/ubuntu-manual-tests/shrini-dia/+merge/228126
> You are the owner of lp:~shrini/ubuntu-manual-tests/shrini-dia.

--
Regards,
T.Shrinivasan

My Life with GNU/Linux : http://goinggnu.wordpress.com
Free E-Magazine on Free Open Source Software in Tamil : http://kaniyam.com

Get CollabNet Subversion Edge : http://www.collab.net/svnedge

« Back to merge proposal