Code review comment for lp:~mcintire-evan/ubuntu-terminal-app/disable-copy

Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thank you for the patch! The code looks good to me and I assume Stefano tested it already.

However, I'm sorry that I am _that_ guy again. :p

For comments we usually put a whitespace after the two leading slashes. Additionally, we only put semi-colons after QML properties when there are two in the same line.

Again, sorry for that, but I think it makes sense to agree on a common code style. Nevertheless, good work! :)

review: Needs Fixing

« Back to merge proposal