[Merge] lp:~lukas-kde/qtubuntu/ubuntuIconsFonts into lp:qtubuntu

Daniel d'Andrada daniel.dandrada at canonical.com
Wed Mar 29 11:13:09 UTC 2017


Review: Needs Fixing

"""
private:
    QFont systemFont, fixedFont;
"""

Should follow the project's naming convention for member variables. Eg:
s/systemFont/mSystemFont
-- 
https://code.launchpad.net/~lukas-kde/qtubuntu/ubuntuIconsFonts/+merge/321178
Your team Ubuntu Phablet Team is subscribed to branch lp:qtubuntu.



More information about the Ubuntu-reviews mailing list