Code review comment for lp:~nik90/ubuntu/precise/remmina/add_quicklist

Sebastien Bacher (seb128) wrote :

That looks great Nekhelesh, thanks for your work! We will make sure that's shipped in precise. One small comment, the OnlyShowIn should have have a trailing ";", i.e "OnlyShowIn=Unity;", the current way will probably work in practice but will trigger warning about the syntax when given to parser tools

« Back to merge proposal