I copied the diff according to the cover letter guidance https://dev.launchpad.net/DetailedCoverLetterTemplate
Since this was my first review I did not know the exact process.
I changed the pagetests narative.
I have also updated the pagetest style as Abel Deuring suggest this problem in another MP.
HTML style was updated... and got rid of that TAB
I need to increase the table size to accommodate the new column. The action column was wrapped when viewed by an admin.
All translation tests are OK. Lint is clean.
Thanks for the review.
« Back to merge proposal
I copied the diff according to the cover letter guidance https:/ /dev.launchpad. net/DetailedCov erLetterTemplat e
Since this was my first review I did not know the exact process.
I changed the pagetests narative.
I have also updated the pagetest style as Abel Deuring suggest this problem in another MP.
HTML style was updated... and got rid of that TAB
I need to increase the table size to accommodate the new column. The action column was wrapped when viewed by an admin.
All translation tests are OK.
Lint is clean.
Thanks for the review.