> Looks good.
> 
> [0]
> 
> 101     -                '<a class="previous"
> href="http://example.org/";>PreVious</span>' +
> 102     -                '<div id="client-listing"></div>' +
> 103     +                    '<a class="previous"
> href="http://example.org/";>PreVious</a>' +
> 104     +                    '<div id="client-listing"></div>' +
> 
> I've spotted the fix (s/span/a/) but the indentation seems off.

The indentation was done just to indicate that there's a nested html structure 
there. The bug here is not limiting to the parent node scope and just tried to 
bring out that there's a parent there. 
-- 
https://code.launchpad.net/~rharding/launchpad/reportbug/+merge/115562
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to