Code review comment for lp:~mc-return/compiz/compiz.merge-fix1075600-workspacenames-default-appearance-values-not-ideal

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Two problems:

1. You're making two unrelated changes in one branch; changing strings and changing the default appearance.

2. We usually don't want to change the default appearance for any established compiz plugins. However I will test the branch and see if I agree it looks better after #1 is fixed.

review: Needs Fixing

« Back to merge proposal